2
0
mirror of https://github.com/fork-maintainers/iceraven-browser synced 2024-11-19 09:25:34 +00:00

[fenix] For https://github.com/mozilla-mobile/fenix/issues/16376: move Startup*Test to perf pkg under code owners.

Instead of listing many different files here, we should just put them in
/perf/ packages and code owner them.
This commit is contained in:
Michael Comella 2020-11-11 15:04:13 -08:00 committed by Michael Comella
parent d5f666cbf3
commit 39384786dd
2 changed files with 2 additions and 7 deletions

1
.github/CODEOWNERS vendored
View File

@ -51,7 +51,6 @@
*StrictMode*kt @mozilla-mobile/Performance *StrictMode*kt @mozilla-mobile/Performance
*ConstraintLayoutPerfDetector* @mozilla-mobile/Performance *ConstraintLayoutPerfDetector* @mozilla-mobile/Performance
*MozillaRunBlockingCheck.kt @mozilla-mobile/Performance *MozillaRunBlockingCheck.kt @mozilla-mobile/Performance
*StartupExcessiveResourceUseTest.kt @mozilla-mobile/Performance
# We want to be aware of new features behind flags as well as features # We want to be aware of new features behind flags as well as features
# about to be enabled. # about to be enabled.

View File

@ -2,7 +2,7 @@
* License, v. 2.0. If a copy of the MPL was not distributed with this * License, v. 2.0. If a copy of the MPL was not distributed with this
* file, You can obtain one at http://mozilla.org/MPL/2.0/. */ * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
package org.mozilla.fenix.ui package org.mozilla.fenix.perf
import android.util.Log import android.util.Log
import android.view.View import android.view.View
@ -16,10 +16,8 @@ import kotlinx.android.synthetic.main.activity_home.*
import org.junit.Assert.assertEquals import org.junit.Assert.assertEquals
import org.junit.Rule import org.junit.Rule
import org.junit.Test import org.junit.Test
import org.mozilla.fenix.perf.RunBlockingCounter
import org.mozilla.fenix.ext.components import org.mozilla.fenix.ext.components
import org.mozilla.fenix.helpers.HomeActivityTestRule import org.mozilla.fenix.helpers.HomeActivityTestRule
import org.mozilla.fenix.perf.ComponentInitCount
// BEFORE INCREASING THESE VALUES, PLEASE CONSULT WITH THE PERF TEAM. // BEFORE INCREASING THESE VALUES, PLEASE CONSULT WITH THE PERF TEAM.
private const val EXPECTED_SUPPRESSION_COUNT = 11 private const val EXPECTED_SUPPRESSION_COUNT = 11
@ -69,9 +67,7 @@ private val failureMsgRecyclerViewConstraintLayoutChildren = getErrorMessage(
* RunBlocking is mostly used to return values to a thread from a coroutine. However, if that * RunBlocking is mostly used to return values to a thread from a coroutine. However, if that
* coroutine takes too long, it can lead that thread to block every other operations. * coroutine takes too long, it can lead that thread to block every other operations.
* *
* The perf team is code owners for this file so they should be notified when the count is modified. * The perf team is code owners for this package so they should be notified when the counts are modified.
*
* IF YOU UPDATE THE TEST NAME, UPDATE CODE OWNERS.
*/ */
class StartupExcessiveResourceUseTest { class StartupExcessiveResourceUseTest {
@get:Rule @get:Rule