Commit Graph

5394 Commits (a197281092b1392c6a4917126fdff6bc5f58ad45)
 

Author SHA1 Message Date
MarcLeclair 78850c7e3d [fenix] 15278 detekt rule runblocking (https://github.com/mozilla-mobile/fenix/pull/15942)
* For https://github.com/mozilla-mobile/fenix/issues/15278: added CoroutineManager to count runBlocking calls

* For https://github.com/mozilla-mobile/fenix/issues/15278: Added actual detekt rule for runblocking and its config to the yaml

* For https://github.com/mozilla-mobile/fenix/issues/15278: Added unit test for RunblockingCounter

* For https://github.com/mozilla-mobile/fenix/issues/15278: renamed StrictModeStartupSuppressionCountTest.kt to PerformanceStartupTest.kt and added runBlockingCount test

* Lint fix

* For https://github.com/mozilla-mobile/fenix/issues/15278: made runblocking a Long to prevent overflow

* For https://github.com/mozilla-mobile/fenix/issues/15278: fixed MozRunblocking name, description and moved RunBlockingCounter to perf package

* For https://github.com/mozilla-mobile/fenix/issues/15278:Renamed MozillaRunblockingCheck to MozillaRunBlockingCheck

* For https://github.com/mozilla-mobile/fenix/issues/15278: Added setup for unit test, since it failed without restting counter

* For https://github.com/mozilla-mobile/fenix/issues/15278: Fixed naming for RunBlocking lint check

* For https://github.com/mozilla-mobile/fenix/issues/15278: removed changes made to test to use runBlockingIncrement

* For https://github.com/mozilla-mobile/fenix/issues/15728: added test exclusion for runBlocking check

* For https://github.com/mozilla-mobile/fenix/issues/15278: changed null check and added Synchronized to count setter

* For https://github.com/mozilla-mobile/fenix/issues/15278: fix for nits

* For https://github.com/mozilla-mobile/fenix/issues/15278: added StartupExcessiveResourceUseTest to CODEOWNERS

* For https://github.com/mozilla-mobile/fenix/issues/15278: fixed for nits

* For https://github.com/mozilla-mobile/fenix/issues/15278: Moved increment function to extension function and fixed indentation

* For https://github.com/mozilla-mobile/fenix/issues/15278: Added tests for Atomic Integer extension and nit fix
4 years ago
ekager e564dda990 [fenix] For https://github.com/mozilla-mobile/fenix/issues/6313 - Remove unneeded delay in BrowserAnimator 4 years ago
Roger Yang f60df4a976 [fenix] For https://github.com/mozilla-mobile/fenix/issues/16268 - Allowing multiple task while sharing (https://github.com/mozilla-mobile/fenix/pull/16275) 4 years ago
Jocelyne Abi Haidar 3139337d7f [fenix] For https://github.com/mozilla-mobile/fenix/issues/15931: Sort Downloads from newest to oldest (https://github.com/mozilla-mobile/fenix/pull/15939) 4 years ago
mcarare e682cb55cc [fenix] For https://github.com/mozilla-mobile/fenix/issues/15430: Fix StrictMode death on opening app from widget voice search. 4 years ago
mcarare c66ab231d5 [fenix] For https://github.com/mozilla-mobile/fenix/issues/15430: Fix StrictMode death on opening app from widget text search. 4 years ago
mcarare 87ce122d03 [fenix] For https://github.com/mozilla-mobile/fenix/issues/14552: Do not display back button when there is no tab history. 4 years ago
MickeyMoz 95d1497c25 [fenix] Update Android Components version to 65.0.20201101143039. 4 years ago
Mozilla L10n Automation Bot a94eca1063 [fenix] Import l10n. 4 years ago
MickeyMoz 95d897d7c1 [fenix] Update Android Components version to 65.0.20201031143038. 4 years ago
Michael Comella 133965917c [fenix] For https://github.com/mozilla-mobile/fenix/issues/15707 - post: add lint test dependencies to android-gradle-dependencies script.
`assemble` doesn't assemble the tests so we need to run `test`.
`testClasses` isn't good enough because, according to `--profile`, it
doesn't include dependencies for `testRuntimeClasspath`.
4 years ago
Michael Comella 7479db73fe [fenix] For https://github.com/mozilla-mobile/fenix/issues/15707: run lint rules tests in CI.
This works but it's imperfect: when the tests fail, it links to the lint
failure artifacts rather than the test failure artifacts.
4 years ago
Michael Comella 62a0f6b088 [fenix] For https://github.com/mozilla-mobile/fenix/issues/15707: add lint rules tests to pre-push hook. 4 years ago
Michael Comella 424af26340 [fenix] For https://github.com/mozilla-mobile/fenix/issues/15707: add MozillaStrictModeSuppressionTest. 4 years ago
Michael Comella 5ed416405e [fenix] For https://github.com/mozilla-mobile/fenix/issues/15707: correctly configure JUnit for mozilla-detekt-rules.
Without this, the tests would not run from the command line though they
could run in Android Studio.
4 years ago
Michael Comella ce90102a75 [fenix] For https://github.com/mozilla-mobile/fenix/issues/15707: add ConstraintLayoutPerfDetectorTest to code owners. 4 years ago
Michael Comella 9ddef29862 [fenix] For https://github.com/mozilla-mobile/fenix/issues/15707: add ConstraintLayoutPerfDetectorTest.
I had to ignore one of the tests because the linter test API seemed to
be broken. All of these APIs are beta so I didn't think it was worth
trying to force it to work.
4 years ago
Michael Comella cb07bf265d [fenix] For https://github.com/mozilla-mobile/fenix/issues/15707: remove super.visitElement for ConstraintLayout lint.
It caused a test to fail because super.visitElement asserted false
because it expected to be overriden. It was correctly overridden - we
were just still calling through to super anyway.
4 years ago
Michael Comella 3f31e4cc1b [fenix] For https://github.com/mozilla-mobile/fenix/issues/15707: clean up lint rules dependencies.
For mozilla-detekt-rules, the tests didn't compile at all so apparently
they haven't been running in testing.

mozilla-lint-rules worked but they were not clean.
4 years ago
Mugurell b85522198c [fenix] For https://github.com/mozilla-mobile/fenix/issues/15963 - Increase touch target for the close button in the tabs tray grid layout
We need to ensure it has at least 48dp while also not appearing too big.
4 years ago
Mugurell 58a22af103 [fenix] For https://github.com/mozilla-mobile/fenix/issues/15757 - Avoid the double spacing issues for grid items in tabs tray
Items should have a 16dp all around padding.
Have them have a 8dp padding that will add to 16 when placed together.
Have the parent have another 8dp that will add to 16 for when items are placed
to the edge of the available screen.
4 years ago
Mugurell 0dd114beb8 [fenix] For https://github.com/mozilla-mobile/fenix/issues/15968 - Show a border for the selected tab in the grid based tabs tray
The border for the grid item will now be updated in the same method that
updates the background for the list item.
4 years ago
Mugurell f90db55a2f [fenix] For https://github.com/mozilla-mobile/fenix/issues/15756 - Add a play/pause button in the tabs tray grid layout
Moved the previous layout in a new, bigger FrameLayout that would serve as a
buffer space for the play/pause button to be translated to the exterior.
Otherwise the button would be clipped.
4 years ago
Gabriel Luong 949d66da5f [fenix] For https://github.com/mozilla-mobile/fenix/issues/9548 - Ability to rename top sites (https://github.com/mozilla-mobile/fenix/pull/16285)
* Ability to rename top sites, closes https://github.com/mozilla-mobile/fenix/issues/9548

* Highlight the "Remove" top site action as destructive

With more menu items in the top site contextual menu, it makes sense to differentiate

* Added test for the top site renaming action

* Fixed lint check (wildcard imports, blank spaces)

* Applied suggestions from code review

* Apply suggestions from code review

Co-authored-by: Gabriel Luong <gabriel.luong@gmail.com>

* Implemented dialog click listener with manual dismiss/cancel

Co-authored-by: Lorenzo Stanco <lorenzo.stanco@gmail.com>
4 years ago
Mickey Moz ccb507dc60 [fenix] Update Android Components version to 65.0.20201030143049. (https://github.com/mozilla-mobile/fenix/pull/16286) 4 years ago
Gregory Mierzwinski 45aa002f94 [fenix] Bug 1674186 - Use shippable platforms for Browsertime tests. (https://github.com/mozilla-mobile/fenix/pull/16261)
* Use shippable platforms.

* Undo testing changes.
4 years ago
ekager c4d7e19da6 [fenix] No issue: Make sure we are cleaning up test coroutines 4 years ago
mozilla-l10n-automation-bot 0a9821fdbb [fenix] Import l10n. (https://github.com/mozilla-mobile/fenix/pull/16274) 4 years ago
ekager 986e28f100 [fenix] For https://github.com/mozilla-mobile/fenix/issues/16161 - Fixes Saved Login toolbar layout 4 years ago
Codrut Topliceanu 2d083e853a [fenix] For https://github.com/mozilla-mobile/fenix/issues/14637 - Fix sitePermissionsFeature breaking change + AC bump (https://github.com/mozilla-mobile/fenix/pull/16229)
* For https://github.com/mozilla-mobile/fenix/issues/14637 - Compatibility patch for AC - 8554

* Update Android Components version to 65.0.20201029143039

Co-authored-by: Arturo Mejia <arturomejiamarmol@gmail.com>
4 years ago
mozilla-l10n-automation-bot f15d1054bb [fenix] Import l10n. (https://github.com/mozilla-mobile/fenix/pull/16240) 4 years ago
mcarare 3047ea3601 [fenix] For https://github.com/mozilla-mobile/fenix/issues/16250: Disable remote USB debugging for SDK <23. 4 years ago
Mihai Adrian Carare 5fb5471df1 [fenix] For # 15929: Remove the search widget discoverability experiment. (https://github.com/mozilla-mobile/fenix/pull/16081)
* For https://github.com/mozilla-mobile/fenix/issues/15929: Remove SearchWidgetCFR telemetry.

* For https://github.com/mozilla-mobile/fenix/issues/15929: Remove SearchWidgetCFR and search widget experiment.

* For https://github.com/mozilla-mobile/fenix/issues/15929: Remove unit tests references to search widget experiment.
4 years ago
MickeyMoz 2fb65a44c7 [fenix] Update Android Components version to 64.0.20201027143116. 4 years ago
mcarare 51018be388 [fenix] For https://github.com/mozilla-mobile/fenix/issues/15413: Use proper url when bookmarking a page. 4 years ago
Oana Horvath 0f70a8d4fc [fenix] For https://github.com/mozilla-mobile/fenix/issues/10690: re-enable editBookmarkTest 4 years ago
Mozilla L10n Automation Bot 29e7f50cc3 [fenix] Import l10n. 4 years ago
Arturo Mejia df0da31c1f [fenix] Fix AddonsManagementFragment detekt error 4 years ago
Christian Sadilek 79ab29eff2 [fenix] No issue: Fix onConfigurationChange crashes in Debug build 4 years ago
Christian Sadilek 56c98b57b8 [fenix] Closes https://github.com/mozilla-mobile/fenix/issues/11285: Replace Session[Manager] observers in BaseBrowserFragment 4 years ago
Grisha Kruglov 268da65b3f [fenix] Closes https://github.com/mozilla-mobile/fenix/issues/15816 - Attempt to handle 'allowInPrivate..' pref with detached context 4 years ago
mcarare 047b712ffe [fenix] For https://github.com/mozilla-mobile/fenix/issues/15379: Use proper url when sharing. 4 years ago
ekager 647e73e851 [fenix] For https://github.com/mozilla-mobile/fenix/issues/16199 - Don't pass BrowserToolbarView in onStart when we just need parent 4 years ago
Arturo Mejia 001a446cf0 [fenix] Update Android Components version to 64.0.20201027134712. 4 years ago
Arturo Mejia d3faecd59c [fenix] Fix add-on translation braking changes 4 years ago
Aaron Train a224f0c546 [fenix] For https://github.com/mozilla-mobile/fenix/issues/8169 - Upgrade Mockwebserver (https://github.com/mozilla-mobile/fenix/pull/16186)
Upgrades Mockwebserver to 4.9.0
4 years ago
mozilla-l10n-automation-bot b28b69915e [fenix] Import l10n. (https://github.com/mozilla-mobile/fenix/pull/16198) 4 years ago
Lorenzo Stanco bd068b288a [fenix] For https://github.com/mozilla-mobile/fenix/issues/16122 - Add a dark theme color for the top sites pin icon (https://github.com/mozilla-mobile/fenix/pull/16180) 4 years ago
ekager 2ae46088f6 [fenix] No issue - Update lint detector tests to run 4 years ago
Christian Sadilek b82dd39018 [fenix] Update Android Components version to 64.0.20201026161129 (https://github.com/mozilla-mobile/fenix/pull/16188) 4 years ago