searching for reviewer(snorp)
3221db51a16c1d73a644881b5351576a6a13c2a7: Bug 1432949 - Return error if channel cannot be opened. Expand download blob test case r=geckoview-reviewers,snorp
owlishDeveloper <bugzeeeeee@gmail.com> - Tue, 09 Mar 2021 00:45:22 +0000 - rev 637528
Push 15212 by ffxbld-merge at Mon, 22 Mar 2021 14:40:41 +0000
Bug 1432949 - Return error if channel cannot be opened. Expand download blob test case r=geckoview-reviewers,snorp Differential Revision: https://phabricator.services.mozilla.com/D75687
077f29a531ec21a48e7b07815bfebad1afb2a287: Bug 1581971: Part 1 - Bump Android NDK requirement to 21d; r=snorp,firefox-build-system-reviewers,mhentges
Aaron Klotz <aklotz@mozilla.com> - Fri, 19 Feb 2021 21:07:18 +0000 - rev 635283
Push 15122 by ffxbld-merge at Mon, 22 Feb 2021 12:02:19 +0000
Bug 1581971: Part 1 - Bump Android NDK requirement to 21d; r=snorp,firefox-build-system-reviewers,mhentges NDK 21 includes `lldb-server`, which we need in order to support `./mach run --debug` with `lldb`. The Android SDK manager no longer includes a standalone `lldb` package; perhaps it was deprecated? Anyway, this appears to currently be the best way to get `lldb-server` into a location that is easy to find during build configuration. Differential Revision: https://phabricator.services.mozilla.com/D94379
373f29f3f07c6745d5f440a1199fbb8627afd3fd: Bug 1611554: Part 2 - Modify GeckoView build to generate manifest and service definitions from jinja files; r=nalexander,geckoview-reviewers,snorp
Aaron Klotz <aklotz@mozilla.com> - Wed, 13 Jan 2021 17:32:54 +0000 - rev 629876
Push 15035 by ffxbld-merge at Mon, 25 Jan 2021 15:18:06 +0000
Bug 1611554: Part 2 - Modify GeckoView build to generate manifest and service definitions from jinja files; r=nalexander,geckoview-reviewers,snorp * We add a config option for setting the number of content services; * We add a config option to indicate whether content services should be isolated. This one is just a `project_flag` since it doesn't really need the ability to be overridden; it's something whose default we would want to flip when the time comes; * We set a dependency so that mobile/android/base/pre-export is executed; * We add the `gen_from_jinja.py` script which is mostly just a dumb shim that takes the input template and the config arguments, instantiates jinja, generates the final output, and dumps it to the output fd; * We add the requisite `moz.build` statements to generate the manifest overlay and the service definitions; * We update `build.gradle` so that Gradle knows to look for the generated files when building the apk. Differential Revision: https://phabricator.services.mozilla.com/D82577
0d4317da10068a0a00d8b71ef5df3136c6eb57c9: Bug 1611554: Part 2 - Modify GeckoView build to generate manifest and service definitions from jinja files; r=nalexander,geckoview-reviewers,snorp
Aaron Klotz <aklotz@mozilla.com> - Wed, 13 Jan 2021 00:24:06 +0000 - rev 629792
Push 15035 by ffxbld-merge at Mon, 25 Jan 2021 15:18:06 +0000
Bug 1611554: Part 2 - Modify GeckoView build to generate manifest and service definitions from jinja files; r=nalexander,geckoview-reviewers,snorp * We add a config option for setting the number of content services; * We add a config option to indicate whether content services should be isolated. This one is just a `project_flag` since it doesn't really need the ability to be overridden; it's something whose default we would want to flip when the time comes; * We set a dependency so that mobile/android/base/pre-export is executed; * We add the `gen_from_jinja.py` script which is mostly just a dumb shim that takes the input template and the config arguments, instantiates jinja, generates the final output, and dumps it to the output fd; * We add the requisite `moz.build` statements to generate the manifest overlay and the service definitions; * We update `build.gradle` so that Gradle knows to look for the generated files when building the apk. Differential Revision: https://phabricator.services.mozilla.com/D82577
1c7597ab79cc8bfe4e5934b792c50128907b4c32: Bug 1685614 - Use GeckoResult in EventDispatcher. r=owlish,snorp
Agi Sferro <agi@sferro.dev> - Thu, 07 Jan 2021 22:24:46 +0000 - rev 629308
Push 15035 by ffxbld-merge at Mon, 25 Jan 2021 15:18:06 +0000
Bug 1685614 - Use GeckoResult in EventDispatcher. r=owlish,snorp This patch adds `queryString`, `queryBoolean`, `queryVoid` and `queryBundle` to `EventDispatcher` in place of `dispatch(type, message, callback)`. These `query*` methods returns a `GeckoResult` which can be manipulated using `GeckoResult.map`. Differential Revision: https://phabricator.services.mozilla.com/D96951
87036d2ffe1e32e84ec8a7157453ecdbfd10f385: Bug 1625233 - Add EventCallback.resolveTo. r=snorp
Agi Sferro <agi@sferro.dev> - Fri, 04 Dec 2020 20:52:45 +0000 - rev 626260
Push 14929 by ffxbld-merge at Mon, 14 Dec 2020 15:19:11 +0000
Bug 1625233 - Add EventCallback.resolveTo. r=snorp Differential Revision: https://phabricator.services.mozilla.com/D96150
3c07e6f1f530e778a2dfcdfd097dce3ea7f56189: Bug 1519636 - Reformat with clang-format-11 to the Google coding style r=andi,sg,geckoview-reviewers,snorp
Sylvestre Ledru <sledru@mozilla.com> - Wed, 18 Nov 2020 09:05:59 +0000 - rev 624482
Push 14929 by ffxbld-merge at Mon, 14 Dec 2020 15:19:11 +0000
Bug 1519636 - Reformat with clang-format-11 to the Google coding style r=andi,sg,geckoview-reviewers,snorp It is bringing some minor changes # ignore-this-changeset Differential Revision: https://phabricator.services.mozilla.com/D90795
97fd4f15e7e915310af2dd7d480d990f637b179a: Bug 1676771 - Resample touch events in Android widget before sending them to APZ. r=geckoview-reviewers,kats,snorp
Markus Stange <mstange.moz@gmail.com> - Wed, 18 Nov 2020 02:59:29 +0000 - rev 624433
Push 14929 by ffxbld-merge at Mon, 14 Dec 2020 15:19:11 +0000
Bug 1676771 - Resample touch events in Android widget before sending them to APZ. r=geckoview-reviewers,kats,snorp This behavior can be toggled with the (live) pref android.touch_resampling.enabled. Differential Revision: https://phabricator.services.mozilla.com/D96798
9c8f524f9c43a672fedf962bf1e98532e12ec696: Bug 1676771 - Break HandleMotionEvent into two parts. r=geckoview-reviewers,kats,snorp
Markus Stange <mstange.moz@gmail.com> - Wed, 18 Nov 2020 02:59:32 +0000 - rev 624432
Push 14929 by ffxbld-merge at Mon, 14 Dec 2020 15:19:11 +0000
Bug 1676771 - Break HandleMotionEvent into two parts. r=geckoview-reviewers,kats,snorp Differential Revision: https://phabricator.services.mozilla.com/D96796
c01032316a7271c40fe91d8cbecdc68c9e9eb83f: Bug 1676771 - Resample touch events in Android widget before sending them to APZ. r=geckoview-reviewers,kats,snorp
Markus Stange <mstange.moz@gmail.com> - Tue, 17 Nov 2020 23:15:34 +0000 - rev 624386
Push 14929 by ffxbld-merge at Mon, 14 Dec 2020 15:19:11 +0000
Bug 1676771 - Resample touch events in Android widget before sending them to APZ. r=geckoview-reviewers,kats,snorp This behavior can be toggled with the (live) pref android.touch_resampling.enabled. Differential Revision: https://phabricator.services.mozilla.com/D96798
45736914b0633e30f42340c008c73d03d8c8ecb8: Bug 1676771 - Break HandleMotionEvent into two parts. r=geckoview-reviewers,kats,snorp
Markus Stange <mstange.moz@gmail.com> - Tue, 17 Nov 2020 20:20:32 +0000 - rev 624385
Push 14929 by ffxbld-merge at Mon, 14 Dec 2020 15:19:11 +0000
Bug 1676771 - Break HandleMotionEvent into two parts. r=geckoview-reviewers,kats,snorp Differential Revision: https://phabricator.services.mozilla.com/D96796
88d4a2ac2cfcb9bc2509b89c19d7f571bb7b9309: Bug 1676771 - Resample touch events in Android widget before sending them to APZ. r=geckoview-reviewers,kats,snorp
Markus Stange <mstange.moz@gmail.com> - Tue, 17 Nov 2020 16:27:07 +0000 - rev 624312
Push 14929 by ffxbld-merge at Mon, 14 Dec 2020 15:19:11 +0000
Bug 1676771 - Resample touch events in Android widget before sending them to APZ. r=geckoview-reviewers,kats,snorp This behavior can be toggled with the (live) pref android.touch_resampling.enabled. Differential Revision: https://phabricator.services.mozilla.com/D96798
7aa9b58c7916bd2c91527e2f1c885e46019d163f: Bug 1676771 - Break HandleMotionEvent into two parts. r=geckoview-reviewers,kats,snorp
Markus Stange <mstange.moz@gmail.com> - Tue, 17 Nov 2020 18:55:53 +0000 - rev 624311
Push 14929 by ffxbld-merge at Mon, 14 Dec 2020 15:19:11 +0000
Bug 1676771 - Break HandleMotionEvent into two parts. r=geckoview-reviewers,kats,snorp Differential Revision: https://phabricator.services.mozilla.com/D96796
1a7948d7ac71c618f409bf68d7c0b924a27e0500: Bug 1671983 - Part 1: Remove now-unused process switching shouldLoadURI methods, r=annyG,geckoview-reviewers,snorp
Nika Layzell <nika@thelayzells.com> - Thu, 12 Nov 2020 18:00:53 +0000 - rev 624041
Push 14929 by ffxbld-merge at Mon, 14 Dec 2020 15:19:11 +0000
Bug 1671983 - Part 1: Remove now-unused process switching shouldLoadURI methods, r=annyG,geckoview-reviewers,snorp These methods are no longer necessary, as all loads which can trigger process switches now go through DocumentChannel. The shouldLoadURI methods on nsIWebBrowserChrome3 are unfortunately still necessary as they're used by the disabled-by-default "Single-Site Browser" feature. In the future this may be possible to clean-up. Differential Revision: https://phabricator.services.mozilla.com/D94638
670b6951b647f11d1e4e60b026f36e26358578e2: Bug 1676814 - Revert a2f0dae16c37 to stop crashing on fast fling etc. r=botond,geckoview-reviewers,snorp
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Fri, 13 Nov 2020 15:21:59 +0000 - rev 623441
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1676814 - Revert a2f0dae16c37 to stop crashing on fast fling etc. r=botond,geckoview-reviewers,snorp Differential Revision: https://phabricator.services.mozilla.com/D96965
b7f5466d0d3b66ab3e6031249fba09b141637e46: Bug 1676768 - Add some missing gfx:: namespace annotations in nsWindow.cpp. r=geckoview-reviewers,snorp
Markus Stange <mstange.moz@gmail.com> - Thu, 12 Nov 2020 16:11:50 +0000 - rev 623212
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1676768 - Add some missing gfx:: namespace annotations in nsWindow.cpp. r=geckoview-reviewers,snorp Depends on D96793 Differential Revision: https://phabricator.services.mozilla.com/D96794
fc6dfec1a09e9eba3ba7d9a3257e0e238ae985af: Bug 1676768 - Add some missing includes. r=geckoview-reviewers,snorp
Markus Stange <mstange.moz@gmail.com> - Thu, 12 Nov 2020 16:11:58 +0000 - rev 623211
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1676768 - Add some missing includes. r=geckoview-reviewers,snorp Depends on D96792 Differential Revision: https://phabricator.services.mozilla.com/D96793
89c8b4e18ab213de32cd5106fe8c714c9ed6b2a6: Bug 1676768 - Move WebAuthnTokenManager class definition into a header file, because nsAppShell.cpp calls WebAuthnTokenManager::Init(). r=geckoview-reviewers,snorp
Markus Stange <mstange.moz@gmail.com> - Thu, 12 Nov 2020 16:12:10 +0000 - rev 623210
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1676768 - Move WebAuthnTokenManager class definition into a header file, because nsAppShell.cpp calls WebAuthnTokenManager::Init(). r=geckoview-reviewers,snorp This only happened to work because unified build included WebAuthnTokenManager.cpp and nsAppShell.cpp into the same compilation unit. Differential Revision: https://phabricator.services.mozilla.com/D96792
11a9aa1e6305f8cd761a2b0d47d67b0a05f91072: Bug 1640416 - p1: input size check before creating video encoder. r=alwu,geckoview-reviewers,snorp
John Lin <jolin@mozilla.com> - Wed, 04 Nov 2020 21:57:36 +0000 - rev 622211
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1640416 - p1: input size check before creating video encoder. r=alwu,geckoview-reviewers,snorp On Android, software H.264 video encoder accepts some sizes when configuring but will crash later after frames are sent to the encoder. Use method provided in API 21+ to validate the input size and avoid using software encoder on earlier versions to prevent crashes. Differential Revision: https://phabricator.services.mozilla.com/D94461
2c215fa18600ef03fdc39d6813d514fbfcd9284a: Bug 1458653 - Do not prevent fling acceleration based on the time that has elapsed between two flings. r=botond,geckoview-reviewers,snorp
Markus Stange <mstange.moz@gmail.com> - Wed, 04 Nov 2020 23:32:38 +0000 - rev 622204
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1458653 - Do not prevent fling acceleration based on the time that has elapsed between two flings. r=botond,geckoview-reviewers,snorp The other fling acceleration restrictions make it so that there is no need for this additional restriction anymore. Removing this restriction allows a fling to be accelerated even if the previous fling has been "spinning" for a while (a second or more), provided that the page is still moving fast enough. This matches Chrome's behavior. Differential Revision: https://phabricator.services.mozilla.com/D95472
dfc2063b5c583ad2795b20c879642200def828cb: Bug 1663000 - Ensure the dynamic toolbar can be movied via scroll handoff from a subframe. r=botond,geckoview-reviewers,snorp
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 04 Nov 2020 03:21:14 +0000 - rev 622036
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1663000 - Ensure the dynamic toolbar can be movied via scroll handoff from a subframe. r=botond,geckoview-reviewers,snorp Differential Revision: https://phabricator.services.mozilla.com/D93919
3695289c1eb92da6750a4dc6c42841541e9c64c3: Bug 1663000 - Handle the event as if it's consumed in the root APZC in InputQueue::ReceiveTouchInput() if there's contents covered by the dynamic toolbar. r=geckoview-reviewers,botond,snorp
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 04 Nov 2020 03:16:17 +0000 - rev 622035
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1663000 - Handle the event as if it's consumed in the root APZC in InputQueue::ReceiveTouchInput() if there's contents covered by the dynamic toolbar. r=geckoview-reviewers,botond,snorp Differential Revision: https://phabricator.services.mozilla.com/D93917
f7f190aa7cb004818057fb2fd20ed469ffa85850: Bug 1663000 - Add a bunch of tests including non-scrollable contents but covered by the dynamic toolbar. r=botond,geckoview-reviewers,snorp
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 04 Nov 2020 03:16:15 +0000 - rev 622034
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1663000 - Add a bunch of tests including non-scrollable contents but covered by the dynamic toolbar. r=botond,geckoview-reviewers,snorp - The test cases cover with/without event handler, and with preventDefault in the event handler - The non-scrollable contents but covered by the dynamic toolbar cases don't pass without the event handler now (the cases with the event handler have been fixed by the previous change) - There is a test case which don't pass at all right now, that is a case (iframe_98vh_scrollable.html) that after scrolled at the bottom edge of an iframe Differential Revision: https://phabricator.services.mozilla.com/D93916
fbb2c1d1159ddb302d306c99fb70a793c06898f1: Bug 1663000 - Return UNHANDLED even if there's no event handler which uses preventDefault() and the root content is not covered by the dynamic toolbar. r=botond,geckoview-reviewers,snorp
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 04 Nov 2020 03:16:07 +0000 - rev 622033
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1663000 - Return UNHANDLED even if there's no event handler which uses preventDefault() and the root content is not covered by the dynamic toolbar. r=botond,geckoview-reviewers,snorp This change has a couple of tests but they are for scrollable/non-scrollable contents, they are not for non-scrollable contents but covered by the dynamic toolbar cases, e.g. 100vh body element. Those cases will be introduced in the subsequent change. Differential Revision: https://phabricator.services.mozilla.com/D93915
0a53caf5f3fd512466effe593eab99736a30c406: Bug 1663000 - Change 33vh value to 33% in touch.html and rename touchEventForResult to touchEventForResultWithStaticToolbar. r=botond,geckoview-reviewers,snorp
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 04 Nov 2020 03:15:57 +0000 - rev 622031
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1663000 - Change 33vh value to 33% in touch.html and rename touchEventForResult to touchEventForResultWithStaticToolbar. r=botond,geckoview-reviewers,snorp touchEventForResult is the only one test using touch.html and it doesn't use `setDynamicToolbarMaxHeight`, which means the test is supposed to work with static toolbar, so with this change it gets clearer what the test does. (and we are going to add a bunch of tests with dynamic toolbar) Differential Revision: https://phabricator.services.mozilla.com/D93913
a404f809f79d14f784843ee6102f224cee15bb67: Bug 1671983 - Part 1: Remove now-unused process switching shouldLoadURI methods, r=annyG,geckoview-reviewers,snorp
Nika Layzell <nika@thelayzells.com> - Wed, 04 Nov 2020 00:55:23 +0000 - rev 622007
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1671983 - Part 1: Remove now-unused process switching shouldLoadURI methods, r=annyG,geckoview-reviewers,snorp These methods are no longer necessary, as all loads which can trigger process switches now go through DocumentChannel. The shouldLoadURI methods on nsIWebBrowserChrome3 are unfortunately still necessary as they're used by the disabled-by-default "Single-Site Browser" feature. In the future this may be possible to clean-up. Differential Revision: https://phabricator.services.mozilla.com/D94638
a7b20ab818cdbaa392f6b21ee7a7fa1c0a4b5953: Bug 1663884 - <input type=number> should use TYPE_NUMBER_FLAG_DECIMAL too. r=m_kato,geckoview-reviewers,snorp
Bharath S <h20190017@goa.bits-pilani.ac.in> - Tue, 03 Nov 2020 22:54:40 +0000 - rev 621993
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1663884 - <input type=number> should use TYPE_NUMBER_FLAG_DECIMAL too. r=m_kato,geckoview-reviewers,snorp Differential Revision: https://phabricator.services.mozilla.com/D95773
6eea54efb322b0ba3dd4583454eda80844a2e256: Bug 1671460 - Add @DeprecationSchedule to @Deprecated APIs. r=snorp
Agi Sferro <agi@sferro.dev> - Mon, 02 Nov 2020 19:42:17 +0000 - rev 621747
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1671460 - Add @DeprecationSchedule to @Deprecated APIs. r=snorp Differential Revision: https://phabricator.services.mozilla.com/D95383
73ef4114f8c1d015bdb331c8dfb1d0d052cd8b98: Bug 1671460 - Update to apilint 0.4.0. r=snorp
Agi Sferro <agi@sferro.dev> - Mon, 02 Nov 2020 19:42:25 +0000 - rev 621746
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1671460 - Update to apilint 0.4.0. r=snorp There is an unfortunate-but-necessary ordering issue that needs to be fixed in the api.txt file which is causing a lot of one time updates. The ordering of annotations was not stable (and now it is). Differential Revision: https://phabricator.services.mozilla.com/D95382
a4674f09c3721d48e9cd9e3fd1cfb53cf4bc179f: Bug 1674498 - Remove references to `jarsigner` from `configure` and delete the `debug_sign_tool` r=geckoview-reviewers,nalexander,snorp
Ricky Stewart <rstewart@mozilla.com> - Mon, 02 Nov 2020 16:50:56 +0000 - rev 621720
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1674498 - Remove references to `jarsigner` from `configure` and delete the `debug_sign_tool` r=geckoview-reviewers,nalexander,snorp This is no longer used by Android devs. Differential Revision: https://phabricator.services.mozilla.com/D95417
3365e10c4d877d47eed782e8fcf70c6353306caa: Bug 1670873 - Ignore DebugConfig on SDK_INT < 21. r=snorp
Agi Sferro <agi@sferro.dev> - Thu, 29 Oct 2020 18:10:06 +0000 - rev 621407
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1670873 - Ignore DebugConfig on SDK_INT < 21. r=snorp Differential Revision: https://phabricator.services.mozilla.com/D94481
e8148ff3fdd39808395b1aff5dc01dbe6bf3bd57: Bug 1670873 - Ignore DebugConfig on SDK_INT < 21. r=snorp
Agi Sferro <agi@sferro.dev> - Thu, 29 Oct 2020 16:18:41 +0000 - rev 621393
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1670873 - Ignore DebugConfig on SDK_INT < 21. r=snorp Differential Revision: https://phabricator.services.mozilla.com/D94481
4ecb44816920e876206d556cf7fa0d766dcacf09: Bug 1500644 - Make GeckoView example toolbar dynamic at bottom by using CoordinatorLayout. r=geckoview-reviewers,snorp
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Tue, 27 Oct 2020 14:57:47 +0000 - rev 621104
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1500644 - Make GeckoView example toolbar dynamic at bottom by using CoordinatorLayout. r=geckoview-reviewers,snorp This way is basically how android-components provides the dynamic toolbar feature but there are a couple of caveats on this change as of now. 1) There is no option to switch the toolbar position at top 2) This change forces the toolbar to be dynamic, there is no way to make it static Each corresponding file in android-components is; NestedGeckoView.java <- NestedGeckoView.kt [1] ToolbarBottomBehavior.java <- BrowserToolbarBottomBehavior.kt [2] GeckoViewBottomBehavior <- EngineViewBottomBehavior.kt [3] [1] https://github.com/mozilla-mobile/android-components/blob/b6a5c6444140920768e1332e2618538efe29151e/components/browser/engine-gecko-nightly/src/main/java/mozilla/components/browser/engine/gecko/NestedGeckoView.kt [2] https://github.com/mozilla-mobile/android-components/blob/b6a5c6444140920768e1332e2618538efe29151e/components/browser/toolbar/src/main/java/mozilla/components/browser/toolbar/behavior/BrowserToolbarBottomBehavior.kt [3] https://github.com/mozilla-mobile/android-components/blob/b6a5c6444140920768e1332e2618538efe29151e/components/feature/session/src/main/java/mozilla/components/feature/session/behavior/EngineViewBottomBehavior.kt Differential Revision: https://phabricator.services.mozilla.com/D92357
6e193e6ff95d1ed585ff20566ed5e03b0a94fcf0: Bug 1673316 - Ensure docShellIsActive is preserved when switching process. r=snorp
Agi Sferro <agi@sferro.dev> - Tue, 27 Oct 2020 18:41:58 +0000 - rev 621049
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1673316 - Ensure docShellIsActive is preserved when switching process. r=snorp I'm not a fan of accessing private bits like `docShellIsActive` in tests like that but it's both: 1) very important, if we don't correctly activate docShell performance will be very poor 2) stable, this API is not likely to change, and if it changes it should be easy to get whatever the replacement is Differential Revision: https://phabricator.services.mozilla.com/D94874
0af7c44c8e72d3fea3793a83752c282dae2d43f7: Bug 1518999 - Trick contentful detection in some geckoview tests r=snorp,geckoview-reviewers,agi
Sean Feng <sefeng@mozilla.com> - Tue, 27 Oct 2020 16:13:23 +0000 - rev 621011
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1518999 - Trick contentful detection in some geckoview tests r=snorp,geckoview-reviewers,agi Some geckoview tests require gradient usage. Since background images are async, these tests would wait for a contentful paint to make sure the images are decoded before running the assertions. This causes an issue because gradient-only backgrounds aren't contentful anymore according to the latest spec. We fix the tests by adding a transparent gif to the background image list to trick the contentful detection. Differential Revision: https://phabricator.services.mozilla.com/D88230
822af3fb6462c53f36e10d76da7caddf6d644814: Bug 1648156 - Migrate GeckoViewSelectionActionChild to actor. r=snorp
Agi Sferro <agi@sferro.dev> - Tue, 27 Oct 2020 16:18:25 +0000 - rev 621005
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1648156 - Migrate GeckoViewSelectionActionChild to actor. r=snorp Differential Revision: https://phabricator.services.mozilla.com/D94648
d03053f89c391892c76babb155365782648c72a0: Bug 1672262 - Add workaround for duplicate START event. r=snorp
Agi Sferro <agi@sferro.dev> - Fri, 23 Oct 2020 21:08:10 +0000 - rev 620578
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1672262 - Add workaround for duplicate START event. r=snorp Differential Revision: https://phabricator.services.mozilla.com/D94371
3b266a09140c96472878746bb8a80b0a61a16189: Bug 1672262 - Disable modules on processSwitch only if they have frameScripts. r=snorp
Agi Sferro <agi@sferro.dev> - Fri, 23 Oct 2020 21:08:07 +0000 - rev 620577
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1672262 - Disable modules on processSwitch only if they have frameScripts. r=snorp This also fixes a bug where we send an extra `about:blank` event when switching process. Differential Revision: https://phabricator.services.mozilla.com/D94370
b6eb139c78a74c05109bf5e87f0f9d5ec110f3aa: Bug 1672262 - Removed unused onDestroyBrowser. r=snorp
Agi Sferro <agi@sferro.dev> - Fri, 23 Oct 2020 21:08:02 +0000 - rev 620575
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1672262 - Removed unused onDestroyBrowser. r=snorp Differential Revision: https://phabricator.services.mozilla.com/D94368
8b2ac5a6c98ae439d819fd5188c8bc3d8241e1b2: Bug 1518999 - Trick contentful detection in some geckoview tests r=snorp,geckoview-reviewers,agi
Sean Feng <sefeng@mozilla.com> - Fri, 23 Oct 2020 19:46:30 +0000 - rev 620565
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1518999 - Trick contentful detection in some geckoview tests r=snorp,geckoview-reviewers,agi Some geckoview tests require gradient usage. Since background images are async, these tests would wait for a contentful paint to make sure the images are decoded before running the assertions. This causes an issue because gradient-only backgrounds aren't contentful anymore according to the latest spec. We fix the tests by adding a transparent gif to the background image list to trick the contentful detection. Differential Revision: https://phabricator.services.mozilla.com/D88230
017e651826e58643b75f738180471ffd04af8f8f: Bug 1670841 - part2 : prevent tab with a main controller from being suspended. r=snorp,nika
alwu <alwu@mozilla.com> - Thu, 22 Oct 2020 21:31:18 +0000 - rev 620395
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1670841 - part2 : prevent tab with a main controller from being suspended. r=snorp,nika When a tab is owning a main controller, which means that the tab is either playing media, or still waiting for media keys in order to control its media again. In those situations, we should not suspend the tab so that its script can continue to run for playing media or handling the future task for media keys. Differential Revision: https://phabricator.services.mozilla.com/D93445
51a5a97203d1d0c6bf3696cd8d113512013e79ec: Bug 1646810 - Unlock fission.autostart on Android. r=snorp
Agi Sferro <agi@sferro.dev> - Mon, 19 Oct 2020 18:13:13 +0000 - rev 619999
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1646810 - Unlock fission.autostart on Android. r=snorp Differential Revision: https://phabricator.services.mozilla.com/D92482
f2409bbdc3f78b3d5f8e2174e0b1eb4db97e1143: Bug 1672247 - Pass correct flags in ProgessTracker's listener. r=snorp
Agi Sferro <agi@sferro.dev> - Tue, 20 Oct 2020 20:20:34 +0000 - rev 619969
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1672247 - Pass correct flags in ProgessTracker's listener. r=snorp Differential Revision: https://phabricator.services.mozilla.com/D94171
017ecd33c3b81bce91ae59f37732fbdf6c4fcf3c: Bug 1646810 - Add processes tab4-tab49. r=snorp
Agi Sferro <agi@sferro.dev> - Mon, 19 Oct 2020 18:29:02 +0000 - rev 619937
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1646810 - Add processes tab4-tab49. r=snorp Differential Revision: https://phabricator.services.mozilla.com/D93863
c2f0c2e26efff9d57526e2322c97d234393108b3: Bug 1665068 - Preserve layers for inactive GV windows. r=jnicol,snorp
Agi Sferro <agi@sferro.dev> - Mon, 19 Oct 2020 18:28:49 +0000 - rev 619729
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1665068 - Preserve layers for inactive GV windows. r=jnicol,snorp Differential Revision: https://phabricator.services.mozilla.com/D93816
af4968ec85fe0d87e6f753ddabbc0e49415040c1: Bug 1665068 - Set default clear color according to theme. r=snorp
Agi Sferro <agi@sferro.dev> - Mon, 19 Oct 2020 18:29:15 +0000 - rev 619728
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1665068 - Set default clear color according to theme. r=snorp This also makes it so we don't set a cover automatically anymore, this avoids a flash when an app sets a session immediately after removing the old session (AC does this). Additionally, this also helps with a startup flash, when it's too early for GeckoView to know whether the app's theme is light or dark. Differential Revision: https://phabricator.services.mozilla.com/D93815
4413bbc14dc86aa582a3e578a1dca2efe387a2f5: Bug 1665068 - Set a default background in SurfaceViewWrapper. r=snorp
Agi Sferro <agi@sferro.dev> - Mon, 19 Oct 2020 18:29:23 +0000 - rev 619727
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1665068 - Set a default background in SurfaceViewWrapper. r=snorp Differential Revision: https://phabricator.services.mozilla.com/D93814
55a5da81f312752af94773b856b566b569b4e629: Bug 1671578 - Overload Loader.equals. r=snorp
Agi Sferro <agi@sferro.dev> - Mon, 19 Oct 2020 15:46:56 +0000 - rev 619696
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1671578 - Overload Loader.equals. r=snorp Differential Revision: https://phabricator.services.mozilla.com/D93812
100f7a3b25f86290cd43a7f1cc10bc5af2c32b22: Bug 1671571 - Don't pass null headers in Loader. r=snorp
Agi Sferro <agi@sferro.dev> - Mon, 19 Oct 2020 15:45:17 +0000 - rev 619695
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1671571 - Don't pass null headers in Loader. r=snorp Differential Revision: https://phabricator.services.mozilla.com/D93811
b9a241e6495a2fbe33ed0237b9132ad11e714b28: Bug 1671849 - Move drag threshold setting to nsLookAndFeel; r=geckoview-reviewers,snorp
Edgar Chen <echen@mozilla.com> - Mon, 19 Oct 2020 15:16:49 +0000 - rev 619672
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1671849 - Move drag threshold setting to nsLookAndFeel; r=geckoview-reviewers,snorp Differential Revision: https://phabricator.services.mozilla.com/D93902