searching for reviewer(jchen)
e19d324e5970979c11c697c600d60fc366314ed8: Bug 1493317 Part 2 - Enable AccessibleCaret in unit tests. r=jchen
Ting-Yu Lin <tlin@mozilla.com> - Sat, 16 Mar 2019 07:38:34 +0000 - rev 522371
Push 10871 by cbrindusan@mozilla.com at Mon, 18 Mar 2019 15:49:32 +0000
Bug 1493317 Part 2 - Enable AccessibleCaret in unit tests. r=jchen And use correct AccessibleCaret preference to disable it individually in tests. Differential Revision: https://phabricator.services.mozilla.com/D10299
0bb898d00ed7b9d1bdaf3b4320cbc2ac5bca00fc: Bug 1485045 - Make Java parts of GeckoView independent from build ABI r=jchen
James Willcox <snorp@snorp.net> - Fri, 23 Nov 2018 20:32:40 +0000 - rev 504326
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1485045 - Make Java parts of GeckoView independent from build ABI r=jchen This allows us to use the same Java code for any native platform, enabling a "fat" AAR. Differential Revision: https://phabricator.services.mozilla.com/D11497
bae43dfd3700460a55f3559eabbc888975f3a3c0: Bug 1485045 - Make Java parts of GeckoView independent from build ABI r=jchen
James Willcox <snorp@snorp.net> - Wed, 21 Nov 2018 15:22:06 +0000 - rev 503911
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1485045 - Make Java parts of GeckoView independent from build ABI r=jchen This allows us to use the same Java code for any native platform, enabling a "fat" AAR. Differential Revision: https://phabricator.services.mozilla.com/D11497
900cd4bf995f4b6a9651da7b9245f66b324e5847: Bug 1494713 - Add `HistoryDelegate` and wire up the Java history handlers. r=jchen,snorp
Lina Cambridge <lina@yakshaving.ninja> - Fri, 16 Nov 2018 17:32:04 +0000 - rev 503236
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1494713 - Add `HistoryDelegate` and wire up the Java history handlers. r=jchen,snorp Differential Revision: https://phabricator.services.mozilla.com/D11291
a1fe5c7791a7fec71e6c3a6b7c362e5ec00ba49f: Bug 1494713 - Implement a `GeckoViewHistory` backend. r=jchen,snorp,smaug
Lina Cambridge <lina@yakshaving.ninja> - Fri, 16 Nov 2018 17:32:02 +0000 - rev 503235
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1494713 - Implement a `GeckoViewHistory` backend. r=jchen,snorp,smaug Differential Revision: https://phabricator.services.mozilla.com/D11290
6644b6e14bde969270ccb646bb501a11221e3041: Bug 1494713 - Add `nsWindow::From`. r=jchen
Lina Cambridge <lina@yakshaving.ninja> - Fri, 16 Nov 2018 17:31:58 +0000 - rev 503234
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1494713 - Add `nsWindow::From`. r=jchen Differential Revision: https://phabricator.services.mozilla.com/D11289
7174015fbbd584f9afa8783c1ac119ab3ac8bbf4: Bug 1485045 - Make Java parts of GeckoView independent from build ABI r=jchen
James Willcox <snorp@snorp.net> - Wed, 14 Nov 2018 17:48:58 +0000 - rev 502647
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1485045 - Make Java parts of GeckoView independent from build ABI r=jchen This allows us to use the same Java code for any native platform, enabling a "fat" AAR. Differential Revision: https://phabricator.services.mozilla.com/D11497
24e87b02707bee36e1e98eb37c94fbaf3834e898: Bug 1499418 - [3.2] Refactor and move Fennec's telemetry classes to geckoview/. r=jchen
Eugen Sawin <esawin@mozilla.com> - Tue, 13 Nov 2018 15:29:21 +0100 - rev 502472
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1499418 - [3.2] Refactor and move Fennec's telemetry classes to geckoview/. r=jchen
8bc653b02e2f7aa97fc460cfa1dc1cd7e5e16302: Bug 1499418 - [1.3] Add GeckoView page load and startup performance telemetry probes. r=snorp,chutten,jchen
Eugen Sawin <esawin@mozilla.com> - Tue, 13 Nov 2018 15:39:11 +0100 - rev 502471
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1499418 - [1.3] Add GeckoView page load and startup performance telemetry probes. r=snorp,chutten,jchen
46ddb51e149152ab2feb664e101474c3f406d4a8: Bug 1436754 - Merge LayerSession into GeckoSession; r=snorp,jchen
Agi Sferro <agi@mozilla.com> - Fri, 09 Nov 2018 17:04:38 +0000 - rev 501886
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1436754 - Merge LayerSession into GeckoSession; r=snorp,jchen Depends on D10990 Differential Revision: https://phabricator.services.mozilla.com/D10991
4c759f171cefc121537bfdcc2213a04effc72057: Bug 1498246 - Add GeckoView Media API r=rbarker,snorp,esawin,jchen,cvan
Imanol Fernandez <mortimergoro@gmail.com> - Fri, 09 Nov 2018 11:59:28 +0000 - rev 501855
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1498246 - Add GeckoView Media API r=rbarker,snorp,esawin,jchen,cvan Add GeckoView Media API which provides a way to listen to HTMLMediaElement events in a GeckoSession and control the playback externally Differential Revision: https://phabricator.services.mozilla.com/D9026
a6b390c7b9791dfa760f9b09248ff84bd2a4c3e6: Bug 1498246 - Add GeckoView Media API r=rbarker,snorp,esawin,jchen,cvan
Imanol Fernandez <mortimergoro@gmail.com> - Thu, 08 Nov 2018 22:11:36 +0000 - rev 501648
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1498246 - Add GeckoView Media API r=rbarker,snorp,esawin,jchen,cvan Add GeckoView Media API which provides a way to listen to HTMLMediaElement events in a GeckoSession and control the playback externally Differential Revision: https://phabricator.services.mozilla.com/D9026
4f8d41b9bb67403c5021a9829ff9fae13ccc8b5c: Bug 1501648 - Prevent crashes for unregistered receivers from MmaDelegate and GeckoMediaControlAgent; r=jchen,JanH
Petru Lingurar <petru.lingurar@softvision.ro> - Thu, 08 Nov 2018 12:08:40 +0000 - rev 501554
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1501648 - Prevent crashes for unregistered receivers from MmaDelegate and GeckoMediaControlAgent; r=jchen,JanH Although this two receivers are guarded by checks for if they were initialized (and so registered) there are reports of crashes because of trying to unregister them without them actually being registered. The underlying issue will be investigated further in bug 1505685 but for the moment wrapping the unregister operations in a try-catch saves the users from a crash and because the unregister is done when the app is closed (for the MmaDelegate receiver) or when the app finished playing media (for the GeckoMediaControlAgent receiver) the user doesn't loose any functionality going forward. Differential Revision: https://phabricator.services.mozilla.com/D11177
60659171e4140a2e38b05d7c85cb2747a83e2964: Bug 1498246 - Add GeckoView Media API r=rbarker,snorp,esawin,jchen,cvan
Imanol Fernandez <mortimergoro@gmail.com> - Thu, 08 Nov 2018 01:37:56 +0000 - rev 501552
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1498246 - Add GeckoView Media API r=rbarker,snorp,esawin,jchen,cvan Add GeckoView Media API which provides a way to listen to HTMLMediaElement events in a GeckoSession and control the playback externally Differential Revision: https://phabricator.services.mozilla.com/D9026
fe3f1d254a1ffcb5f074bca1ec63f358e9f26118: Bug 1502187 - Implement Java part of viewport tree caching. r=jchen
Eitan Isaacson <eitan@monotonous.org> - Mon, 05 Nov 2018 22:35:35 +0000 - rev 501132
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1502187 - Implement Java part of viewport tree caching. r=jchen Depends on D9866 Differential Revision: https://phabricator.services.mozilla.com/D9867
70def5ae1d1128924c54f104fff71c5fbf337f9f: Bug 1502187 - Add FULL_ACCESSIBILITY_TREE to GeckoSessionSettings. r=jchen
Eitan Isaacson <eitan@monotonous.org> - Tue, 06 Nov 2018 17:04:12 +0000 - rev 501130
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1502187 - Add FULL_ACCESSIBILITY_TREE to GeckoSessionSettings. r=jchen Defaults now to 'true', when caching is good enough I'll switch it off. Generally, this would be used for apps/cases when full accessible tree is needed like UIAutomator. Depends on D9864 Differential Revision: https://phabricator.services.mozilla.com/D9865
1fafb01078dbb3602bcdac29d40f91f67f71fba3: Bug 1498233 - Stub out getBrowserForOuterWindowID for GeckoViewTab. r=jchen
Mike Conley <mconley@mozilla.com> - Tue, 06 Nov 2018 04:35:00 +0000 - rev 501109
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1498233 - Stub out getBrowserForOuterWindowID for GeckoViewTab. r=jchen GeckoViewTab stubs out gBrowser on the root window for WebExtension compatibility. LoginManagerParent also looks for gBrowser on windows that are handling username and password form fills, and expects it to implement getBrowserForOuterWindowID. This patch makes the stub implement getBrowserForOuterWindowID, which just returns the lone GeckoViewTab browser. Differential Revision: https://phabricator.services.mozilla.com/D10750
91f7f603954c1b7265bb8746dfd2d656e376981a: Bug 1405081 - Add WebRequest, WebResponse, and GeckoWebExecutor to GeckoView r=jchen,esawin
James Willcox <snorp@snorp.net> - Thu, 18 Oct 2018 08:28:09 -0500 - rev 500413
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1405081 - Add WebRequest, WebResponse, and GeckoWebExecutor to GeckoView r=jchen,esawin This exposes Gecko networking to GeckoView apps. It includes speculative connections, name resolution, and a Fetch-like HTTP API. Differential Revision: https://phabricator.services.mozilla.com/D7799 squash to executor
e24c0262f83ecdeb93ec0571df7528a377a4779e: Bug 1405081 - Move default timeout logic into Environment utility class r=jchen
James Willcox <snorp@snorp.net> - Tue, 16 Oct 2018 13:23:05 -0500 - rev 500411
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1405081 - Move default timeout logic into Environment utility class r=jchen Differential Revision: https://phabricator.services.mozilla.com/D8781
778ef7a62d5d321ce67b0c2ed06b79006e308e8b: Bug 1405081 - Move GeckoSessionTestRule.waitForResult() into UiThreadUtils r=jchen
James Willcox <snorp@snorp.net> - Tue, 16 Oct 2018 13:22:26 -0500 - rev 500410
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1405081 - Move GeckoSessionTestRule.waitForResult() into UiThreadUtils r=jchen Differential Revision: https://phabricator.services.mozilla.com/D7798
5fda5eb5fd01b8c4d77ed10279d931665c4e86f7: Bug 1405081 - Add WebRequestError and use it in NavigationDelegate.onLoadError r=esawin,jchen
James Willcox <snorp@snorp.net> - Tue, 16 Oct 2018 13:21:49 -0500 - rev 500409
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1405081 - Add WebRequestError and use it in NavigationDelegate.onLoadError r=esawin,jchen Differential Revision: https://phabricator.services.mozilla.com/D7797
017c51559d4afdbe3478b38d2585ec5566787965: Bug 1405081 - Move GeckoRuntime creation out of GeckoSesionTestRule r=jchen
James Willcox <snorp@snorp.net> - Thu, 04 Oct 2018 17:47:14 -0500 - rev 500408
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1405081 - Move GeckoRuntime creation out of GeckoSesionTestRule r=jchen This allows us to easily share a GeckoRuntime with other tests that don't use GeckoSessionTestRule. Differential Revision: https://phabricator.services.mozilla.com/D7796
9cfb0bf7cb66ab1b53492550d88832821b74be54: Bug 1485718 - Enable setting of javascript on GeckoView sessions through `GeckoSessionSettings.` r=jchen
Emily Toop <etoop@mozilla.com> - Thu, 01 Nov 2018 12:45:37 +0000 - rev 500350
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1485718 - Enable setting of javascript on GeckoView sessions through `GeckoSessionSettings.` r=jchen This is a resubmission due to issues landing old one. Not sure if this needs a re-review? Old review: https://phabricator.services.mozilla.com/D8093. This has now been closed so I cannot update it. Differential Revision: https://phabricator.services.mozilla.com/D10199
3992a5ddbac3421ba1ecba0a95e7649644db2b38: Bug 1502435 - 3 - Update tests to include checks for canary clientId; r=jchen
Petru Lingurar <petru.lingurar@softvision.ro> - Wed, 31 Oct 2018 01:14:57 +0200 - rev 500041
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1502435 - 3 - Update tests to include checks for canary clientId; r=jchen
5cb4e28e1eb88a03a8a707f53f97cbf4cfe45b12: Bug 1502435 - 2 - Include new `bug_1501329_affected` property in the core ping; r=jchen
Petru Lingurar <petru.lingurar@softvision.ro> - Wed, 31 Oct 2018 01:13:57 +0200 - rev 500040
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1502435 - 2 - Include new `bug_1501329_affected` property in the core ping; r=jchen Summary: Depends on D10202 Reviewers: jchen Reviewed By: jchen Subscribers: jchen, janerik Bug #: 1502435 Differential Revision: https://phabricator.services.mozilla.com/D10205
e0a9b2e68b8f8554bd1c410f3def294b2084eeea: Bug 1502435 - 1 - Set valid clientId and save if previous was canary clientId; r=jchen
Petru Lingurar <petru.lingurar@softvision.ro> - Tue, 30 Oct 2018 16:28:02 +0000 - rev 500018
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1502435 - 1 - Set valid clientId and save if previous was canary clientId; r=jchen Differential Revision: https://phabricator.services.mozilla.com/D10202
ef5037b7a3cdabb2bee19adcc36186e9d279ea8e: Bug 1501571 - Update <Product and feature tips> setting summary when disabled; r=jchen
Petru Lingurar <petru.lingurar@softvision.ro> - Mon, 29 Oct 2018 16:24:10 +0000 - rev 499796
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1501571 - Update <Product and feature tips> setting summary when disabled; r=jchen Needed to allow localizers more control over what needs to be translated in a String that contains another already translated String that should be bolded. Differential Revision: https://phabricator.services.mozilla.com/D9629
8d38810fdc7309de2520fb9b36abbb09767a4bee: Bug 1497526 - TransactionTooLargeException when downloading from Data URLs; r=jchen
Petru Lingurar <petru.lingurar@softvision.ro> - Fri, 26 Oct 2018 07:39:01 +0000 - rev 499536
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1497526 - TransactionTooLargeException when downloading from Data URLs; r=jchen The issue stems from putting download's source url as `cookie` in the pending intent for when the download notification is clicked. This cookie functions like a tag, allowing to identify a particular download. The problem is that a download's url can be a Data URL for which Mozilla imposes no size limit. When an Android Intent with that payload is used it counts against Binder's 1MB limit which can result in the TransactionTooLargeException. If we are to truncate download's source url to an arbitrary length of 100 we should still get a unique `cookie` for that download and at the same time avoid cluttering Notification's contentIntent. Differential Revision: https://phabricator.services.mozilla.com/D9666
73cfb79f183ac786b56dd97e63e969642ed0919f: Bug 1500906 - Suppress FileUriExposedExceptions when launching helper apps. r=jchen
Jan Henning <jh+bugzilla@buttercookie.de> - Wed, 24 Oct 2018 16:00:06 +0000 - rev 499261
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1500906 - Suppress FileUriExposedExceptions when launching helper apps. r=jchen Sharing tabs with file:// URIs is not possible, but users can still send them to other apps via the helper app system in the URL bar/context menu. "Intent:Open" and "Intent:OpenForResult" are both sent from Gecko by HelperApps.jsm. The same reasoning as in bug 1450449 applies as to why for publicly accessible files content:// URIs are more trouble than they're worth. Differential Revision: https://phabricator.services.mozilla.com/D9420
a263875f2a4c7c1b02445f9b90cb4ed9c3875c8a: Bug 1500511 - Fix javadoc issue in GeckoRuntimeSettings. r=jchen.
Agi Sferro <agi@mozilla.com> - Wed, 24 Oct 2018 15:58:39 +0000 - rev 499238
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1500511 - Fix javadoc issue in GeckoRuntimeSettings. r=jchen. Differential Revision: https://phabricator.services.mozilla.com/D9295
307ddcb2187bd1a1f95e537d745c7dfc4de11d24: Bug 1485718 - Enable setting of javascript on GeckoView sessions through `GeckoSessionSettings.` r=jchen
Emily Toop <etoop@mozilla.com> - Tue, 23 Oct 2018 15:40:30 +0000 - rev 499025
Push 10290 by ffxbld-merge at Mon, 03 Dec 2018 16:23:23 +0000
Bug 1485718 - Enable setting of javascript on GeckoView sessions through `GeckoSessionSettings.` r=jchen Differential Revision: https://phabricator.services.mozilla.com/D8093
26a874969a1e6efe27fbdc617f791bfdf141730d: Bug 1502187 - Implement Java part of viewport tree caching. r=jchen a=jcristau
Eitan Isaacson <eitan@monotonous.org> - Mon, 05 Nov 2018 22:35:35 +0000 - rev 498634
Push 10248 by jcristau@mozilla.com at Thu, 22 Nov 2018 18:20:00 +0000
Bug 1502187 - Implement Java part of viewport tree caching. r=jchen a=jcristau Depends on D9866 Differential Revision: https://phabricator.services.mozilla.com/D9867
f75f849333f5d4f2616e483cae97a3956083fb27: Bug 1502187 - Add FULL_ACCESSIBILITY_TREE to GeckoSessionSettings. r=jchen a=jcristau
Eitan Isaacson <eitan@monotonous.org> - Tue, 06 Nov 2018 17:04:12 +0000 - rev 498632
Push 10248 by jcristau@mozilla.com at Thu, 22 Nov 2018 18:20:00 +0000
Bug 1502187 - Add FULL_ACCESSIBILITY_TREE to GeckoSessionSettings. r=jchen a=jcristau Defaults now to 'true', when caching is good enough I'll switch it off. Generally, this would be used for apps/cases when full accessible tree is needed like UIAutomator. Depends on D9864 Differential Revision: https://phabricator.services.mozilla.com/D9865
7e437c76c2707ccadc0fcab0f2d24e7868c2a4c1: Bug 1501648 - Prevent crashes for unregistered receivers from MmaDelegate and GeckoMediaControlAgent. r=jchen,JanH, a=jcristau
Petru Lingurar <petru.lingurar@softvision.ro> - Thu, 08 Nov 2018 12:08:40 +0000 - rev 498473
Push 10179 by ryanvm@gmail.com at Tue, 13 Nov 2018 15:48:17 +0000
Bug 1501648 - Prevent crashes for unregistered receivers from MmaDelegate and GeckoMediaControlAgent. r=jchen,JanH, a=jcristau Although this two receivers are guarded by checks for if they were initialized (and so registered) there are reports of crashes because of trying to unregister them without them actually being registered. The underlying issue will be investigated further in bug 1505685 but for the moment wrapping the unregister operations in a try-catch saves the users from a crash and because the unregister is done when the app is closed (for the MmaDelegate receiver) or when the app finished playing media (for the GeckoMediaControlAgent receiver) the user doesn't loose any functionality going forward. Differential Revision: https://phabricator.services.mozilla.com/D11177
410f4e7ae2feb52dc23db8892829e1b767477849: Bug 1498233 - Stub out getBrowserForOuterWindowID for GeckoViewTab. r=jchen a=jcristau
Mike Conley <mconley@mozilla.com> - Tue, 06 Nov 2018 04:35:00 +0000 - rev 498397
Push 10138 by ebalazs@mozilla.com at Thu, 08 Nov 2018 12:27:01 +0000
Bug 1498233 - Stub out getBrowserForOuterWindowID for GeckoViewTab. r=jchen a=jcristau GeckoViewTab stubs out gBrowser on the root window for WebExtension compatibility. LoginManagerParent also looks for gBrowser on windows that are handling username and password form fills, and expects it to implement getBrowserForOuterWindowID. This patch makes the stub implement getBrowserForOuterWindowID, which just returns the lone GeckoViewTab browser. Differential Revision: https://phabricator.services.mozilla.com/D10750
cb358118e54de761923e4b29bee9d34c1560b98a: Bug 1502435 - Update tests to include checks for canary clientId. r=jchen, a=RyanVM
Petru Lingurar <petru.lingurar@softvision.ro> - Wed, 31 Oct 2018 01:14:57 +0200 - rev 498245
Push 10079 by ryanvm@gmail.com at Wed, 31 Oct 2018 19:35:01 +0000
Bug 1502435 - Update tests to include checks for canary clientId. r=jchen, a=RyanVM
81b5e0d8a72630490c41ccd34698c08413e2866b: Bug 1502435 - Include new `bug_1501329_affected` property in the core ping. r=jchen, a=RyanVM
Petru Lingurar <petru.lingurar@softvision.ro> - Wed, 31 Oct 2018 01:13:57 +0200 - rev 498244
Push 10079 by ryanvm@gmail.com at Wed, 31 Oct 2018 19:35:01 +0000
Bug 1502435 - Include new `bug_1501329_affected` property in the core ping. r=jchen, a=RyanVM Summary: Depends on D10202 Reviewers: jchen Reviewed By: jchen Subscribers: jchen, janerik Bug #: 1502435 Differential Revision: https://phabricator.services.mozilla.com/D10205
02cbbb3db3b576d57ad3f6f74916ea6d4b4f3306: Bug 1502435 - Set valid clientId and save if previous was canary clientId. r=jchen, a=RyanVM
Petru Lingurar <petru.lingurar@softvision.ro> - Tue, 30 Oct 2018 16:28:02 +0000 - rev 498243
Push 10079 by ryanvm@gmail.com at Wed, 31 Oct 2018 19:35:01 +0000
Bug 1502435 - Set valid clientId and save if previous was canary clientId. r=jchen, a=RyanVM Differential Revision: https://phabricator.services.mozilla.com/D10202
f9da620994ae1a1cf533882625c1777af07f56c3: Bug 1497526 - TransactionTooLargeException when downloading from Data URLs. r=jchen, a=RyanVM
Petru Lingurar <petru.lingurar@softvision.ro> - Fri, 26 Oct 2018 07:39:01 +0000 - rev 498224
Push 10076 by ryanvm@gmail.com at Wed, 31 Oct 2018 15:37:36 +0000
Bug 1497526 - TransactionTooLargeException when downloading from Data URLs. r=jchen, a=RyanVM The issue stems from putting download's source url as `cookie` in the pending intent for when the download notification is clicked. This cookie functions like a tag, allowing to identify a particular download. The problem is that a download's url can be a Data URL for which Mozilla imposes no size limit. When an Android Intent with that payload is used it counts against Binder's 1MB limit which can result in the TransactionTooLargeException. If we are to truncate download's source url to an arbitrary length of 100 we should still get a unique `cookie` for that download and at the same time avoid cluttering Notification's contentIntent. Differential Revision: https://phabricator.services.mozilla.com/D9666
2cbb3d6f52b09d55770daf277c509e087d88846d: Bug 1500906 - Suppress FileUriExposedExceptions when launching helper apps. r=jchen, a=RyanVM
Jan Henning <jh+bugzilla@buttercookie.de> - Wed, 24 Oct 2018 16:00:06 +0000 - rev 498120
Push 10031 by ryanvm@gmail.com at Thu, 25 Oct 2018 00:29:17 +0000
Bug 1500906 - Suppress FileUriExposedExceptions when launching helper apps. r=jchen, a=RyanVM Sharing tabs with file:// URIs is not possible, but users can still send them to other apps via the helper app system in the URL bar/context menu. "Intent:Open" and "Intent:OpenForResult" are both sent from Gecko by HelperApps.jsm. The same reasoning as in bug 1450449 applies as to why for publicly accessible files content:// URIs are more trouble than they're worth. Differential Revision: https://phabricator.services.mozilla.com/D9420
6eaed80ae36ffeaffe0bad1e969e5c92e1c9922c: Bug 1459864 - Modify <Product and feature tips> setting summary when disabled; r=jchen
Petru Lingurar <petru.lingurar@softvision.ro> - Fri, 19 Oct 2018 14:25:57 +0000 - rev 497968
Push 10016 by rgurzau@mozilla.com at Mon, 22 Oct 2018 13:58:55 +0000
Bug 1459864 - Modify <Product and feature tips> setting summary when disabled; r=jchen Inform the user about why is this feature disabled and what to do to enable it. Differential Revision: https://phabricator.services.mozilla.com/D8545
b9614de52765a2da6de9eb067c2431c9517b95cd: Bug 1499895 - [2.1] Update onLoadRequest tests. r=jchen
Eugen Sawin <esawin@mozilla.com> - Thu, 18 Oct 2018 23:51:56 +0200 - rev 497848
Push 10002 by archaeopteryx@coole-files.de at Fri, 19 Oct 2018 23:09:29 +0000
Bug 1499895 - [2.1] Update onLoadRequest tests. r=jchen
45e3efb5e07666bd7e6e7bb0c0db34aa2cd75b4a: Bug 1499895 - [1.3] Add triggering URI to onLoadRequest. r=snorp,jchen
Eugen Sawin <esawin@mozilla.com> - Wed, 17 Oct 2018 10:06:42 +0200 - rev 497847
Push 10002 by archaeopteryx@coole-files.de at Fri, 19 Oct 2018 23:09:29 +0000
Bug 1499895 - [1.3] Add triggering URI to onLoadRequest. r=snorp,jchen
740c3623a44aed64203d830a03682e14f90917f9: Bug 1496745 - Allow GeckoResult to be used without a Looper r=agi,jchen
James Willcox <snorp@snorp.net> - Thu, 18 Oct 2018 18:01:18 +0000 - rev 497730
Push 10002 by archaeopteryx@coole-files.de at Fri, 19 Oct 2018 23:09:29 +0000
Bug 1496745 - Allow GeckoResult to be used without a Looper r=agi,jchen GeckoResult can now be created on a thread with no Looper present. You can use `then` as before after creating a derived GeckoResult with a Handler via `withHandler`, or poll for the value via the new `poll` method. Differential Revision: https://phabricator.services.mozilla.com/D7896
50cc3c94b20d7ba104fc24b887e15064cad69d41: Bug 1499188 - Make accessibility entry input type an int in bundle message. r=jchen,Jamie
Eitan Isaacson <eitan@monotonous.org> - Tue, 16 Oct 2018 16:43:16 +0000 - rev 497729
Push 10002 by archaeopteryx@coole-files.de at Fri, 19 Oct 2018 23:09:29 +0000
Bug 1499188 - Make accessibility entry input type an int in bundle message. r=jchen,Jamie Differential Revision: https://phabricator.services.mozilla.com/D8871
222a23f1044ac44b3c2defb9b816bb80b90fd0b6: Bug 1499182 - Make Android role class names enums. r=jchen,Jamie
Eitan Isaacson <eitan@monotonous.org> - Wed, 17 Oct 2018 20:43:49 +0000 - rev 497588
Push 9996 by archaeopteryx@coole-files.de at Thu, 18 Oct 2018 18:37:15 +0000
Bug 1499182 - Make Android role class names enums. r=jchen,Jamie Differential Revision: https://phabricator.services.mozilla.com/D8764
2fee8f9b283dac85340edf7815fd62b93352bd33: Bug 1496745 - Allow GeckoResult to be used without a Looper r=agi,jchen
James Willcox <snorp@snorp.net> - Tue, 16 Oct 2018 18:21:40 +0000 - rev 497446
Push 9996 by archaeopteryx@coole-files.de at Thu, 18 Oct 2018 18:37:15 +0000
Bug 1496745 - Allow GeckoResult to be used without a Looper r=agi,jchen GeckoResult can now be created on a thread with no Looper present. You can use `then` as before after creating a derived GeckoResult with a Handler via `withHandler`, or poll for the value via the new `poll` method. Differential Revision: https://phabricator.services.mozilla.com/D7896
af27cdf736f592782bb9f6d0a0bff62222a55e3e: Bug 1495786 - Replace ambiguous Booleans with AllowOrDeny, an enum with clearly named values. r=snorp,jchen
Dylan Roeh <droeh@mozilla.com> - Tue, 16 Oct 2018 12:23:42 -0500 - rev 497280
Push 9996 by archaeopteryx@coole-files.de at Thu, 18 Oct 2018 18:37:15 +0000
Bug 1495786 - Replace ambiguous Booleans with AllowOrDeny, an enum with clearly named values. r=snorp,jchen
7413b2c7cdb4f45da50d2130a9403978977a36ed: Bug 1498854 - Rework dismissing of TabHistoryFragment. r=jchen
Jan Henning <jh+bugzilla@buttercookie.de> - Tue, 16 Oct 2018 16:17:18 +0000 - rev 497243
Push 9996 by archaeopteryx@coole-files.de at Thu, 18 Oct 2018 18:37:15 +0000
Bug 1498854 - Rework dismissing of TabHistoryFragment. r=jchen 1. The patch from bug 1476710 was nonsense and had the same effect as simply deleting that line, because the ChildFragmentManager is only of interest if the TabHistoryFragment loaded further Fragments itself. 2. The issue at hand is that under some circumstances the TabHistoryFragment will be trying to dismiss itself while its responsible FragmentManager is already busy transacting some Fragment state changes. More precisely, the fact that the Fragment is calling popBackStack*Immediately*, which isn't allowed if the FragmentManager is already handling some other transaction. 3. The dismiss() calls in response to the onClick() handlers are unproblematic, because they won't trigger any FragmentManager activity through any route other than the dismiss() call itself. 4. The dismiss() calls in onPause() *are* problematic because the Fragment- Manager will already be busy pausing the TabHistoryFragment, so triggering a further synchronous transaction is not allowed (and originally caused bug 1476710). 5. If the onPause() call happened because some external entity was attempting to remove the fragment (either BrowserApp directly, or indirectly by forwarding a back button press to the FragmentManager), then the Fragment trying to additionally remove itself is unnecessary. 6. If the onPause() call happens because the containing activity itself is being paused, then the Fragment being dismissed is the desired outcome (see bug 1093209), however the Fragment won't be able to remove itself because a) A synchronous transaction is illegal at that point in time. b) An async transaction would be possible, but might not complete until after onSaveInstanceState() had subsequently already been called, which again would be illegal because of state loss. c) An async transaction allowing state loss would succeed in any case, but would mean that if BrowserApp was subsequently destroyed while in back- ground and then later recreated from the savedInstanceState, the Tab- HistoryFragment would be recreated as well, which is undesired. 7. Therefore, the only way to dismiss the TabHistoryFragment when the containing activity is pausing is to synchronously dismiss the Fragment from inside the activity, *before* the onPause() call is forwarded to the FragmentManager. 8. Calling dismiss() in response to onDestroy() is unnecessary, because the Fragment is already irrevocably being removed as soon as we hit onPause(). 9. Because it doesn't make sense to have multiple TabHistoryFragments active at the same time, we also change the logic such that any attempt to show a new TabHistoryFragment will now replace the previous fragment. This is also useful in view of the fact that in order to close the Fragment, BrowserApp retrieves it by calling findFragmentByTag(), which simply returns the first matching Fragment, i.e. wouldn't properly handle things if we ever accidentally ended up with multiple Fragment instances active at the same time. Differential Revision: https://phabricator.services.mozilla.com/D8680
8eaeb557ba9f438afb0d6a7bd3bfc19f7b6df098: Bug 1498849 - Check for SDK compatibility on CollectionInfo.obtain(). r=jchen
Eitan Isaacson <eitan@monotonous.org> - Tue, 16 Oct 2018 16:00:15 +0000 - rev 497240
Push 9996 by archaeopteryx@coole-files.de at Thu, 18 Oct 2018 18:37:15 +0000
Bug 1498849 - Check for SDK compatibility on CollectionInfo.obtain(). r=jchen One method signature was introduced in API 19, and another, with selectionMode was introduced in 21. Differential Revision: https://phabricator.services.mozilla.com/D8788