searching for reviewer(agi)
e3e9aa4186aa6767aad99e22d1304cb9de648923: WIP: Bug 1703707 - Replace cross-process OMXClient call with IPC to enable isolated process in Android r=geckoview-reviewers,aklotz,agi,jolin
owlishDeveloper <bugzeeeeee@gmail.com> - Fri, 16 Apr 2021 23:34:35 +0000 - rev 576449
Push 141348 by istorozhko@mozilla.com at Fri, 16 Apr 2021 23:36:57 +0000
WIP: Bug 1703707 - Replace cross-process OMXClient call with IPC to enable isolated process in Android r=geckoview-reviewers,aklotz,agi,jolin Differential Revision: https://phabricator.services.mozilla.com/D111200
e355ab90d444dd76c8a4f823c3520338e9a96246: Bug 1689369 - Use fallible version of StringParam in BoxString. r=geckoview-reviewers,agi
Makoto Kato <m_kato@ga2.so-net.ne.jp> - Fri, 16 Apr 2021 06:53:49 +0000 - rev 576344
Push 141255 by m_kato@ga2.so-net.ne.jp at Fri, 16 Apr 2021 06:56:15 +0000
Bug 1689369 - Use fallible version of StringParam in BoxString. r=geckoview-reviewers,agi If JavaScript string is UTF-16, we can return error when we cannot allocate Java string. But if it is Latin-1, GeckoView crashes due to using infallible version of StringParam. So we should use fallible version of StringParam instead even if Latin-1. Differential Revision: https://phabricator.services.mozilla.com/D112176
a290c08cb41e0617d7384c9e0f517fec32671e29: Bug 1700750 - Added setCookieBehaviorPrivateMode to ContentBlocking.Settings r=esawin,geckoview-reviewers,agi
Dylan Roeh <droeh@mozilla.com> - Wed, 14 Apr 2021 01:29:17 +0000 - rev 575812
Push 140970 by droeh@mozilla.com at Wed, 14 Apr 2021 01:31:40 +0000
Bug 1700750 - Added setCookieBehaviorPrivateMode to ContentBlocking.Settings r=esawin,geckoview-reviewers,agi Differential Revision: https://phabricator.services.mozilla.com/D110940
426ef1679498e4d2ab548ef4611bc2c95a45d5f7: Bug 1703968: Part 7 - Set GV's COOP+COEP processCount; r=geckoview-reviewers,agi
Aaron Klotz <aklotz@mozilla.com> - Mon, 12 Apr 2021 21:57:26 +0000 - rev 575567
Push 140805 by aklotz@mozilla.com at Mon, 12 Apr 2021 22:00:27 +0000
Bug 1703968: Part 7 - Set GV's COOP+COEP processCount; r=geckoview-reviewers,agi The default setting for the number of COOP+COEP processes is 1, which I believe is causing various test failures. While this is not the final configuration, I'd like to use this value (total content processes - number of e10s web content processes) to test this out. Differential Revision: https://phabricator.services.mozilla.com/D111729
e196ca586b933590d6a8058ad92bcc50a6a61fd9: Bug 1691819 - [4.5] Add credit card autocomplete tests. r=geckoview-reviewers,agi
Eugen Sawin <esawin@mozilla.com> - Mon, 12 Apr 2021 21:57:16 +0000 - rev 575566
Push 140804 by esawin@mozilla.com at Mon, 12 Apr 2021 21:59:52 +0000
Bug 1691819 - [4.5] Add credit card autocomplete tests. r=geckoview-reviewers,agi Differential Revision: https://phabricator.services.mozilla.com/D109135
22224a37e7d40957f6ab3c3a7ca92bc9ad56d3c5: Bug 1691819 - [2.1] Share required prefs to enable credit card and address form autocomplete for GeckoView. r=geckoview-reviewers,agi
Eugen Sawin <esawin@mozilla.com> - Mon, 12 Apr 2021 21:57:15 +0000 - rev 575564
Push 140804 by esawin@mozilla.com at Mon, 12 Apr 2021 21:59:52 +0000
Bug 1691819 - [2.1] Share required prefs to enable credit card and address form autocomplete for GeckoView. r=geckoview-reviewers,agi Differential Revision: https://phabricator.services.mozilla.com/D108521
82d6eabbbc9dfb451531dbda25453b4b3e90ebd1: Bug 1691819 - [1.7] Extend the Autocomplete API to support credit cards. r=geckoview-reviewers,agi
Eugen Sawin <esawin@mozilla.com> - Mon, 12 Apr 2021 21:57:15 +0000 - rev 575563
Push 140804 by esawin@mozilla.com at Mon, 12 Apr 2021 21:59:52 +0000
Bug 1691819 - [1.7] Extend the Autocomplete API to support credit cards. r=geckoview-reviewers,agi Differential Revision: https://phabricator.services.mozilla.com/D106695
dec61fda86750d64d5d94d01cb15f52750a3bebf: Bug 1691821 - [1.a.3] Extract common mechanics from the form autofill extension to toolkit to enable cross-platform reuse. r=zbraniecki,dimi,tgiles,geckoview-reviewers,agi
Eugen Sawin <esawin@mozilla.com> - Mon, 12 Apr 2021 17:33:15 +0000 - rev 575511
Push 140772 by esawin@mozilla.com at Mon, 12 Apr 2021 18:14:39 +0000
Bug 1691821 - [1.a.3] Extract common mechanics from the form autofill extension to toolkit to enable cross-platform reuse. r=zbraniecki,dimi,tgiles,geckoview-reviewers,agi Differential Revision: https://phabricator.services.mozilla.com/D108057
10ec0515d9ba462f0c385579a49ea79fc04ac5bf: Bug 1655896 -Use TextChangeData.MergeWith instead. r=geckoview-reviewers,agi
Makoto Kato <m_kato@ga2.so-net.ne.jp> - Mon, 12 Apr 2021 03:14:24 +0000 - rev 575424
Push 140709 by m_kato@ga2.so-net.ne.jp at Mon, 12 Apr 2021 03:17:03 +0000
Bug 1655896 -Use TextChangeData.MergeWith instead. r=geckoview-reviewers,agi deleteSurroundingText may create multiple text transactions. This test failure occurs when multiple text transactions are dispatched to Gecko. GeckoEditableSupport records text changes by Gecko then it notifies Java of text changes. And it merges old text changes with new text changes by AddIMETextChange to reduce JNI/Binder calls if possible. This issue is that AddIMETextChange creates incorrect text change range by merging text change ranges. Actually, TextChangeData already has merge function now, so we should use it instead of own merge function in GeckoEditableSupport. Differential Revision: https://phabricator.services.mozilla.com/D111207
2604a5dedf0ee2ff0476d0b3ac62173c9315c40c: Bug 1703968: Part 2 - Enable COOP+COEP prefs on GeckoView Nightly; r=geckoview-reviewers,agi
Aaron Klotz <aklotz@mozilla.com> - Fri, 09 Apr 2021 22:57:17 +0000 - rev 575341
Push 140669 by aklotz@mozilla.com at Fri, 09 Apr 2021 23:06:15 +0000
Bug 1703968: Part 2 - Enable COOP+COEP prefs on GeckoView Nightly; r=geckoview-reviewers,agi Differential Revision: https://phabricator.services.mozilla.com/D111444
356cc754720a221d572dbc5f17e1c7ba6389a78e: Bug 1703968: Part 1 - Set GeckoView's e10s web process count to 2 on Nightly; r=geckoview-reviewers,agi
Aaron Klotz <aklotz@mozilla.com> - Fri, 09 Apr 2021 22:57:16 +0000 - rev 575340
Push 140669 by aklotz@mozilla.com at Fri, 09 Apr 2021 23:06:15 +0000
Bug 1703968: Part 1 - Set GeckoView's e10s web process count to 2 on Nightly; r=geckoview-reviewers,agi Differential Revision: https://phabricator.services.mozilla.com/D111443
e955f7f5503cfd0474ba86d40df606c681b77849: Bug 1691821 - [1.a.2] Extract common mechanics from the form autofill extension to toolkit to enable cross-platform reuse. r=zbraniecki,dimi,tgiles,geckoview-reviewers,agi
Eugen Sawin <esawin@mozilla.com> - Fri, 09 Apr 2021 22:12:11 +0000 - rev 575329
Push 140665 by esawin@mozilla.com at Fri, 09 Apr 2021 23:01:23 +0000
Bug 1691821 - [1.a.2] Extract common mechanics from the form autofill extension to toolkit to enable cross-platform reuse. r=zbraniecki,dimi,tgiles,geckoview-reviewers,agi Differential Revision: https://phabricator.services.mozilla.com/D108057
cbe5389803cdd15b0de91c2a5def5104049f62b8: Bug 1703968: Part 2 - Enable COOP+COEP prefs on GeckoView Nightly; r=geckoview-reviewers,agi
Aaron Klotz <aklotz@mozilla.com> - Fri, 09 Apr 2021 16:45:54 +0000 - rev 575284
Push 140631 by aklotz@mozilla.com at Fri, 09 Apr 2021 17:50:38 +0000
Bug 1703968: Part 2 - Enable COOP+COEP prefs on GeckoView Nightly; r=geckoview-reviewers,agi Depends on D111443 Differential Revision: https://phabricator.services.mozilla.com/D111444
ac7dbdbff84d2bd066c842ede3f2ddb14feaf0b4: Bug 1703968: Part 1 - Set GeckoView's e10s web process count to 2 on Nightly; r=geckoview-reviewers,agi
Aaron Klotz <aklotz@mozilla.com> - Fri, 09 Apr 2021 16:45:53 +0000 - rev 575283
Push 140631 by aklotz@mozilla.com at Fri, 09 Apr 2021 17:50:38 +0000
Bug 1703968: Part 1 - Set GeckoView's e10s web process count to 2 on Nightly; r=geckoview-reviewers,agi Differential Revision: https://phabricator.services.mozilla.com/D111443
482fec13b96fc58c569f1e7d00635f7ac024e2b5: Bug 1700365 - SynthesizeNativeMouseEvent shouldn't set button state on ButtonUp. r=geckoview-reviewers,agi
Makoto Kato <m_kato@ga2.so-net.ne.jp> - Fri, 09 Apr 2021 09:25:35 +0000 - rev 575210
Push 140578 by m_kato@ga2.so-net.ne.jp at Fri, 09 Apr 2021 09:28:22 +0000
Bug 1700365 - SynthesizeNativeMouseEvent shouldn't set button state on ButtonUp. r=geckoview-reviewers,agi Button state is current button push state. So when on ButtonUp, this is 0. Differential Revision: https://phabricator.services.mozilla.com/D110197
406fb35b61ca6f612ea494d8ef2cdf2976fa4d7d: Bug 1668952: Part 2 - Fix handling of long toplevel data URIs in GeckoView; r=geckoview-reviewers,agi
Aaron Klotz <aklotz@mozilla.com> - Fri, 09 Apr 2021 06:01:58 +0000 - rev 575198
Push 140568 by aklotz@mozilla.com at Fri, 09 Apr 2021 06:04:38 +0000
Bug 1668952: Part 2 - Fix handling of long toplevel data URIs in GeckoView; r=geckoview-reviewers,agi While I evaluated multiple approaches to this problem, I selected two: The first is to handle the case in `GeckoSession.load` directly from within Java, as this allows us to avoid a Java to Native string conversion, as well as giving us better opportunities to report the exact nature of the failure back to the embedding app. We add a new error code specifically for this case. In the case where no `NavigationDelegate` is registered, I used `GeckoSession.load` throwing an exception as a crude way of notifying the app. In the case where the load request originates from within web content, we deny the request from within `nsWindow` to avoid requiring any conversion of the URI spec from native into Java. Differential Revision: https://phabricator.services.mozilla.com/D109427
d4b5e3c2ae1e20d702472b5e4af609a3e642c2b8: Bug 1561927 - Ignore weird motion event on Chrome OS. r=geckoview-reviewers,agi
Makoto Kato <m_kato@ga2.so-net.ne.jp> - Tue, 06 Apr 2021 22:50:03 +0000 - rev 574664
Push 140293 by m_kato@ga2.so-net.ne.jp at Tue, 06 Apr 2021 22:52:31 +0000
Bug 1561927 - Ignore weird motion event on Chrome OS. r=geckoview-reviewers,agi Android app on Chrome OS doesn't use scroll event on touch pad scroll. And this generate wired mouse event via it. So we should add a workaround like Blink (https://crbug.com/704051). Differential Revision: https://phabricator.services.mozilla.com/D110058
ad3936fb3593733fac9dcedd385f708ab0909b36: Bug 1701668, remove WebBrowserChromeChild actor, r=agi
Neil Deakin <neil@mozilla.com> - Tue, 06 Apr 2021 18:52:47 +0000 - rev 574602
Push 140268 by neil@mozilla.com at Tue, 06 Apr 2021 19:05:00 +0000
Bug 1701668, remove WebBrowserChromeChild actor, r=agi Differential Revision: https://phabricator.services.mozilla.com/D110876
da0d62a0534b60a964918d43f8c105c699157dd0: Bug 1701668, remove onBeforeLinkTraversal implementations and handle links in docshell, r=kmag,mixedpuppy,geckoview-reviewers,agi
Neil Deakin <neil@mozilla.com> - Tue, 06 Apr 2021 18:52:46 +0000 - rev 574600
Push 140268 by neil@mozilla.com at Tue, 06 Apr 2021 19:05:00 +0000
Bug 1701668, remove onBeforeLinkTraversal implementations and handle links in docshell, r=kmag,mixedpuppy,geckoview-reviewers,agi This also fixes links in top or in-process subframes and out-of-process subframes in extensions sidebars and panels so that they open in new tabs. Differential Revision: https://phabricator.services.mozilla.com/D110102
44faa7ca10bb0b1972eb6e50bfab829d3fc4a756: Bug 1703118 - Remove unneeded caching / parent-process checks in android lookandfeel. r=agi,geckoview-reviewers
Emilio Cobos Álvarez <emilio@crisal.io> - Tue, 06 Apr 2021 10:29:43 +0000 - rev 574480
Push 140208 by ealvarez@mozilla.com at Tue, 06 Apr 2021 10:32:10 +0000
Bug 1703118 - Remove unneeded caching / parent-process checks in android lookandfeel. r=agi,geckoview-reviewers After bug 1697607 and the subsequent pref removal this code only runs in the parent process. After bug 1699088 the caller takes care of caching. So there should be no need to cache stuff ourselves. Differential Revision: https://phabricator.services.mozilla.com/D110871
5682b0384df056930db2a330c9c8c8bb3e031b81: Bug 1701148 - Remove unused variables in AndroidBridge. r=geckoview-reviewers,agi
Makoto Kato <m_kato@ga2.so-net.ne.jp> - Mon, 29 Mar 2021 01:21:28 +0000 - rev 573531
Push 139670 by m_kato@ga2.so-net.ne.jp at Mon, 29 Mar 2021 01:23:53 +0000
Bug 1701148 - Remove unused variables in AndroidBridge. r=geckoview-reviewers,agi Depends on D109857 Differential Revision: https://phabricator.services.mozilla.com/D109858
ddffe5f2a69bbe8418de1b081d471b822f1d3929: Bug 1701148 - Get rid of AndroidBridge::GetAPIVersion(). r=geckoview-reviewers,agi
Makoto Kato <m_kato@ga2.so-net.ne.jp> - Mon, 29 Mar 2021 01:21:27 +0000 - rev 573530
Push 139670 by m_kato@ga2.so-net.ne.jp at Mon, 29 Mar 2021 01:23:53 +0000
Bug 1701148 - Get rid of AndroidBridge::GetAPIVersion(). r=geckoview-reviewers,agi Use `mozilla::jni::GetAPIVersion` instead. Differential Revision: https://phabricator.services.mozilla.com/D109857
789b7a6daf1835c571cdf66be712b8fa78bffab6: Bug 1697633 - Add tests for getPermissions and onLocationChange permission reporting. r=geckoview-reviewers,agi
Dylan Roeh <droeh@mozilla.com> - Thu, 25 Mar 2021 16:20:26 +0000 - rev 573121
Push 139477 by droeh@mozilla.com at Thu, 25 Mar 2021 16:23:40 +0000
Bug 1697633 - Add tests for getPermissions and onLocationChange permission reporting. r=geckoview-reviewers,agi Differential Revision: https://phabricator.services.mozilla.com/D108962
d7bbf31acfbce69bc6bdc7839499dc3d13228738: Bug 1697633 - Update onLocationChange to include permission reporting; add getAllPermissions and getPermissions to StorageController. r=agi,geckoview-reviewers,owlish
Dylan Roeh <droeh@mozilla.com> - Thu, 25 Mar 2021 16:20:25 +0000 - rev 573120
Push 139477 by droeh@mozilla.com at Thu, 25 Mar 2021 16:23:40 +0000
Bug 1697633 - Update onLocationChange to include permission reporting; add getAllPermissions and getPermissions to StorageController. r=agi,geckoview-reviewers,owlish Differential Revision: https://phabricator.services.mozilla.com/D107927
c34db229aed29014f676dabff036f25f22f8b32c: Bug 1699646 - Add Android device information to the profile meta object r=geckoview-reviewers,agi,gerald,julienw
Nazım Can Altınova <canaltinova@gmail.com> - Wed, 24 Mar 2021 16:56:12 +0000 - rev 572887
Push 139391 by canaltinova@gmail.com at Wed, 24 Mar 2021 16:58:36 +0000
Bug 1699646 - Add Android device information to the profile meta object r=geckoview-reviewers,agi,gerald,julienw Differential Revision: https://phabricator.services.mozilla.com/D109408
16e354847fa2d3eeddf7ae2e61025d0a1c3d5a8b: Bug 1698174: Remove support for MOZ_DEBUG_*_WAIT_FOR_JAVA_DEBUGGER environment variables; r=geckoview-reviewers,agi
Aaron Klotz <aklotz@mozilla.com> - Tue, 23 Mar 2021 18:59:54 +0000 - rev 572750
Push 139304 by aklotz@mozilla.com at Tue, 23 Mar 2021 21:14:15 +0000
Bug 1698174: Remove support for MOZ_DEBUG_*_WAIT_FOR_JAVA_DEBUGGER environment variables; r=geckoview-reviewers,agi This is in favour of using `adb shell am set-debug-app` instead. Differential Revision: https://phabricator.services.mozilla.com/D108977
b17d8c432b6b096cb0a09873701dfea5c13e375a: Bug 1680407 - [geckodriver] Remove --android-storage argument and always use external storage for profile folder. r=webdriver-reviewers,agi,jgraham
Henrik Skupin <mail@hskupin.info> - Tue, 23 Mar 2021 12:46:43 +0000 - rev 572632
Push 139259 by hskupin@mozilla.com at Tue, 23 Mar 2021 12:49:10 +0000
Bug 1680407 - [geckodriver] Remove --android-storage argument and always use external storage for profile folder. r=webdriver-reviewers,agi,jgraham Differential Revision: https://phabricator.services.mozilla.com/D109407
c5708498541f28d20104d746eabbc48637ff8ee0: Bug 1680407 - [geckodriver] Remove --android-storage argument and always use external storage for profile folder. r=webdriver-reviewers,agi,jgraham
Henrik Skupin <mail@hskupin.info> - Tue, 23 Mar 2021 12:05:19 +0000 - rev 572618
Push 139252 by hskupin@mozilla.com at Tue, 23 Mar 2021 12:08:26 +0000
Bug 1680407 - [geckodriver] Remove --android-storage argument and always use external storage for profile folder. r=webdriver-reviewers,agi,jgraham Differential Revision: https://phabricator.services.mozilla.com/D109407
e17c6f92b2bf41e464bbbea1afbff9c9e78db93b: Bug 1699707 - Remove DeviceProximityEvent and UserProximityEvent r=agi,annevk,smaug
Kagami Sascha Rosylight <krosylight@mozilla.com> - Mon, 22 Mar 2021 18:16:16 +0000 - rev 572451
Push 139173 by krosylight@mozilla.com at Mon, 22 Mar 2021 19:29:28 +0000
Bug 1699707 - Remove DeviceProximityEvent and UserProximityEvent r=agi,annevk,smaug Note that this removes `window.ondeviceproximity` and `window.onuserproximity` which unexpectedly have been exposed unconditionally. Differential Revision: https://phabricator.services.mozilla.com/D109160
d9efd0b6e97df876e866b4c7ecf0d05b2bec22fd: Bug 1697683 - Don't allow getAvailableLanguages to return null. r=geckoview-reviewers,agi
Eitan Isaacson <eitan@monotonous.org> - Mon, 22 Mar 2021 16:30:06 +0000 - rev 572415
Push 139153 by eisaacson@mozilla.com at Mon, 22 Mar 2021 17:13:29 +0000
Bug 1697683 - Don't allow getAvailableLanguages to return null. r=geckoview-reviewers,agi Differential Revision: https://phabricator.services.mozilla.com/D109111
cf692733d4f67919c32f01c6f7b1dc6e4678a0ec: Bug 1685486 - Annotate SessionState.fromString/toString appropriately, stop throwing JSONExceptions, and handle null values a bit more gracefully. r=agi,geckoview-reviewers,owlish
Dylan Roeh <droeh@mozilla.com> - Thu, 18 Mar 2021 17:49:23 +0000 - rev 571844
Push 138924 by droeh@mozilla.com at Thu, 18 Mar 2021 17:51:47 +0000
Bug 1685486 - Annotate SessionState.fromString/toString appropriately, stop throwing JSONExceptions, and handle null values a bit more gracefully. r=agi,geckoview-reviewers,owlish Differential Revision: https://phabricator.services.mozilla.com/D105385
2e49dd2821a9b85a65df37de480a665c3e1e2b83: Bug 1699022 - Set surrounding text on onCreateInputConnection. r=geckoview-reviewers,agi
Makoto Kato <m_kato@ga2.so-net.ne.jp> - Thu, 18 Mar 2021 03:18:37 +0000 - rev 571749
Push 138865 by m_kato@ga2.so-net.ne.jp at Thu, 18 Mar 2021 03:21:01 +0000
Bug 1699022 - Set surrounding text on onCreateInputConnection. r=geckoview-reviewers,agi Android R+ has new API to set initial surrounding text. According to Android commit log [*], it can improve IME start up performance when setting initial surrounding text that is new API from Android API 30. * https://android.googlesource.com/platform/frameworks/base/+/c486acc4d29a0e441ddcd05b3fc0c919aee7fbd0 Differential Revision: https://phabricator.services.mozilla.com/D108736
3c458ed737cd3f605229ae113c04cb98c8dbb90d: Bug 1697680 - Remove Presentation API implementation r=smaug,agi
Kagami Sascha Rosylight <krosylight@mozilla.com> - Tue, 16 Mar 2021 18:21:12 +0000 - rev 571480
Push 138691 by krosylight@mozilla.com at Tue, 16 Mar 2021 19:56:33 +0000
Bug 1697680 - Remove Presentation API implementation r=smaug,agi Differential Revision: https://phabricator.services.mozilla.com/D108574
a8ac10a9cf48cf24331b29d2702939e903c7780a: Bug 1573860: Always use the system principal as the requesting principal when loading user-initiated URIs; r=agi,ckerschb
Aaron Klotz <aklotz@mozilla.com> - Tue, 16 Mar 2021 15:26:44 +0000 - rev 571443
Push 138666 by aklotz@mozilla.com at Tue, 16 Mar 2021 16:02:55 +0000
Bug 1573860: Always use the system principal as the requesting principal when loading user-initiated URIs; r=agi,ckerschb As discussed in the bug, when the app calls `loadUri` without `LOAD_FLAGS_EXTERNAL` and there is no referrer session, we may always use the system principal as the requesting principal. I removed all the special-casing of the other various schemes under the expectation that, under the conditions described above, they are also subject to the same policy. Differential Revision: https://phabricator.services.mozilla.com/D107517
30a34933ceae974cd6abda707137bc698d70ac49: Bug 1523577 - Clean up dead fennec code r=geckoview-reviewers,agi
owlishDeveloper <bugzeeeeee@gmail.com> - Mon, 15 Mar 2021 22:31:51 +0000 - rev 571358
Push 138610 by istorozhko@mozilla.com at Mon, 15 Mar 2021 22:34:14 +0000
Bug 1523577 - Clean up dead fennec code r=geckoview-reviewers,agi Depends on D108267 Differential Revision: https://phabricator.services.mozilla.com/D108268
81c687e80813b73faf8e0a37231685e476344dac: Bug 1523639: Make GeckoLoader.setupGeckoEnvironment compatible with isolatedProcess; r=geckoview-reviewers,agi
Aaron Klotz <aklotz@mozilla.com> - Mon, 15 Mar 2021 20:45:46 +0000 - rev 571334
Push 138592 by aklotz@mozilla.com at Mon, 15 Mar 2021 20:48:15 +0000
Bug 1523639: Make GeckoLoader.setupGeckoEnvironment compatible with isolatedProcess; r=geckoview-reviewers,agi * We remove the reference to `MOZ_ANDROID_DATA_DIR`, since it appears to have no more references in mozilla-central. There is one reference in the xpcshell harness for which I have filed a follow-up bug. * Given that the following variables are not useful by content, we restrict their retrieval to the parent process: * `DOWNLOADS_DIRECTORY` * `UPDATES_DIRECTORY` * `EXTERNAL_STORAGE` * `CACHE_DIRECTORY` * `PUBLIC_STORAGE` * Initial prefs * `TMPDIR` is still used by the XPCOM directory service. Given that we can still retrieve this value safely under `isolatedProcess`, we leave this in for now. We can revisit this one in the future. * We restrict retrieval of the user serial number to the parent process. AFAICT we do send this value along with crash reports, so I modified `IChildProcess.start` to accept this value and propagate it into the child process's environment so that an isolated process crash may still include it. * The remaining envrionment variables remain safe to retrieve when isolated. Differential Revision: https://phabricator.services.mozilla.com/D108507
d77fccbf6d2a416f7b5d4a8dd5014ce4f29c516e: Bug 1697607 - Enable widget.remote-look-and-feel everywhere. r=cmartin,geckoview-reviewers,agi
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 11 Mar 2021 21:20:54 +0000 - rev 570764
Push 138341 by ealvarez@mozilla.com at Fri, 12 Mar 2021 02:00:16 +0000
Bug 1697607 - Enable widget.remote-look-and-feel everywhere. r=cmartin,geckoview-reviewers,agi (And add a missing include while at it) The biggest concern here is potential startup time penalty (and it shouldn't be much anyways, if at all). In exchange, we avoid doing a lot of this work in content processes. Let's keep an eye on for regressions, but this sticking allows us to simplify a lot of the lookandfeel code in follow-ups. Differential Revision: https://phabricator.services.mozilla.com/D107926
9456d2f7812fba6157c7d6f3aa5247fc8fbef634: Bug 1691101 - Migrate from jcenter in android builds r=geckoview-reviewers,nalexander,agi
owlishDeveloper <bugzeeeeee@gmail.com> - Thu, 11 Mar 2021 00:18:40 +0000 - rev 570625
Push 138243 by istorozhko@mozilla.com at Thu, 11 Mar 2021 00:33:50 +0000
Bug 1691101 - Migrate from jcenter in android builds r=geckoview-reviewers,nalexander,agi Differential Revision: https://phabricator.services.mozilla.com/D107573
4ecf37c0cdd7badb41551d6e0acf8ba61edb4631: Bug 1691799: Bump GeckoView Nightly's content process limit to 40; r=geckoview-reviewers,agi
Aaron Klotz <aklotz@mozilla.com> - Mon, 08 Mar 2021 20:51:34 +0000 - rev 570135
Push 137934 by aklotz@mozilla.com at Mon, 08 Mar 2021 20:53:59 +0000
Bug 1691799: Bump GeckoView Nightly's content process limit to 40; r=geckoview-reviewers,agi This is the same number that Chromium uses. Differential Revision: https://phabricator.services.mozilla.com/D107518
13872f781893ee84b9173bd46b2c0a52cf763f45: Bug 1693416: Rewrite reset processes in tests to be compatible with isolated process r=geckoview-reviewers,agi
owlishDeveloper <bugzeeeeee@gmail.com> - Mon, 08 Mar 2021 20:20:26 +0000 - rev 570133
Push 137932 by istorozhko@mozilla.com at Mon, 08 Mar 2021 20:22:50 +0000
Bug 1693416: Rewrite reset processes in tests to be compatible with isolated process r=geckoview-reviewers,agi Differential Revision: https://phabricator.services.mozilla.com/D106862
06094b85b709b490fe2136b36bf40ffe9b3f2b13: Bug 1658988 - Close tab in ContentDelegate#onCloseRequest instead of finishing the activity to keep GVE from going into background r=geckoview-reviewers,agi
owlishDeveloper <bugzeeeeee@gmail.com> - Mon, 08 Mar 2021 17:24:11 +0000 - rev 570112
Push 137917 by istorozhko@mozilla.com at Mon, 08 Mar 2021 17:26:37 +0000
Bug 1658988 - Close tab in ContentDelegate#onCloseRequest instead of finishing the activity to keep GVE from going into background r=geckoview-reviewers,agi Differential Revision: https://phabricator.services.mozilla.com/D107484
fc28d407da7f6052588af603129992fa0b7e8136: Bug 1693584 - [2.0] Update MediaElement tests so suppress deprecation warnings. r=geckoview-reviewers,agi
Eugen Sawin <esawin@mozilla.com> - Thu, 04 Mar 2021 18:27:03 +0000 - rev 569708
Push 137769 by esawin@mozilla.com at Thu, 04 Mar 2021 19:26:36 +0000
Bug 1693584 - [2.0] Update MediaElement tests so suppress deprecation warnings. r=geckoview-reviewers,agi Differential Revision: https://phabricator.services.mozilla.com/D107232
0a465508f8cf25c28ed96b41407b080666361707: Bug 1693584 - [1.1] Deprecate the MediaElement API. r=geckoview-reviewers,agi
Eugen Sawin <esawin@mozilla.com> - Thu, 04 Mar 2021 18:27:02 +0000 - rev 569707
Push 137769 by esawin@mozilla.com at Thu, 04 Mar 2021 19:26:36 +0000
Bug 1693584 - [1.1] Deprecate the MediaElement API. r=geckoview-reviewers,agi Differential Revision: https://phabricator.services.mozilla.com/D107124
9992c143ba703148f130f2e1d0aa5b88a7615823: Bug 1693477 - notifyPush needs messageId. r=geckoview-reviewers,agi
Makoto Kato <m_kato@ga2.so-net.ne.jp> - Thu, 04 Mar 2021 00:36:44 +0000 - rev 569579
Push 137700 by m_kato@ga2.so-net.ne.jp at Thu, 04 Mar 2021 00:39:11 +0000
Bug 1693477 - notifyPush needs messageId. r=geckoview-reviewers,agi This is simple typo when payload is nothing. Also, we should add unit tests for this since we don't run mochitest in dom/push/test. The mochitest requires mock's implementation, but we doesn't have it. Differential Revision: https://phabricator.services.mozilla.com/D107067
2a971a8fde5610690cbea1a5c413a9b515e8ce0b: Bug 1678505 - Add {PanZoomController, GeckoView}.onTouchEventForDetailResult. r=geckoview-reviewers,agi,owlish
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Tue, 02 Mar 2021 08:06:29 +0000 - rev 569231
Push 137489 by hikezoe.birchill@mozilla.com at Tue, 02 Mar 2021 08:09:18 +0000
Bug 1678505 - Add {PanZoomController, GeckoView}.onTouchEventForDetailResult. r=geckoview-reviewers,agi,owlish Note about junit tests for overscroll-behavior; With a single test html and changing overscroll-behavior style on the root scroll element (i.e. <html>) dynamically would introduce new intermittent failures because there is no handy way to make sure the style change has been reflected on the APZ side, so instead we use static html files respectively. Differential Revision: https://phabricator.services.mozilla.com/D103421
7f777566dd98d5e5275073643dead97b3738c716: Bug 1678505 - Expose ScrollableDirections and OverscrollDirecitons to GeckoView. r=botond,geckoview-reviewers,agi,aklotz
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Tue, 02 Mar 2021 08:06:28 +0000 - rev 569230
Push 137489 by hikezoe.birchill@mozilla.com at Tue, 02 Mar 2021 08:09:18 +0000
Bug 1678505 - Expose ScrollableDirections and OverscrollDirecitons to GeckoView. r=botond,geckoview-reviewers,agi,aklotz Differential Revision: https://phabricator.services.mozilla.com/D103420
56e4eb9e13dbd6ac8d6f99b231e53b98df575f38: Bug 1643736 - Adding appropriate check before passing cookieStoreId to GeckoViewTabBridge.createNewTab() r=robwu,agi
ankushduacodes <61025943+ankushduacodes@users.noreply.github.com> - Mon, 01 Mar 2021 20:49:07 +0000 - rev 569147
Push 137446 by asferro@mozilla.com at Tue, 02 Mar 2021 00:23:38 +0000
Bug 1643736 - Adding appropriate check before passing cookieStoreId to GeckoViewTabBridge.createNewTab() r=robwu,agi Differential Revision: https://phabricator.services.mozilla.com/D100151
4d91456f4232fa3d903fe87955a8f39a0c4483de: Bug 1694699 - Minimize access to tab.hasTabPermission r=rpl,geckoview-reviewers,agi
Rob Wu <rob@robwu.nl> - Fri, 26 Feb 2021 21:34:54 +0000 - rev 568999
Push 137338 by rob@robwu.nl at Sat, 27 Feb 2021 00:47:17 +0000
Bug 1694699 - Minimize access to tab.hasTabPermission r=rpl,geckoview-reviewers,agi tab.hasTabPermission indirectly triggers an access to browser.currentURI, which for lazy tab browsers causes an incorrect value to be cached. To avoid this, skip the call to hasTabPermission. Differential Revision: https://phabricator.services.mozilla.com/D106622
09f4c4093c073cfee53d08138d0d65de7184fa87: Bug 1678505 - Add {PanZoomController, GeckoView}.onTouchEventForDetailResult. r=geckoview-reviewers,agi,owlish
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Fri, 26 Feb 2021 04:15:19 +0000 - rev 568886
Push 137270 by hikezoe.birchill@mozilla.com at Fri, 26 Feb 2021 04:18:08 +0000
Bug 1678505 - Add {PanZoomController, GeckoView}.onTouchEventForDetailResult. r=geckoview-reviewers,agi,owlish Note about junit tests for overscroll-behavior; With a single test html and changing overscroll-behavior style on the root scroll element (i.e. <html>) dynamically would introduce new intermittent failures because there is no handy way to make sure the style change has been reflected on the APZ side, so instead we use static html files respectively. Differential Revision: https://phabricator.services.mozilla.com/D103421
76e3f391b7ddb5382fb6667e08f034f04df8439a: Bug 1678505 - Expose ScrollableDirections and OverscrollDirecitons to GeckoView. r=botond,geckoview-reviewers,agi,aklotz
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Fri, 26 Feb 2021 04:15:19 +0000 - rev 568885
Push 137270 by hikezoe.birchill@mozilla.com at Fri, 26 Feb 2021 04:18:08 +0000
Bug 1678505 - Expose ScrollableDirections and OverscrollDirecitons to GeckoView. r=botond,geckoview-reviewers,agi,aklotz Differential Revision: https://phabricator.services.mozilla.com/D103420