96b69746b2b66e818601d38d9e09c891def08f8e: Bug 1655940 part 4 - Optimize Array constructor in CacheIR and Warp. r?anba! draft
Jan de Mooij <jdemooij@mozilla.com> - Sat, 01 Aug 2020 12:46:21 +0200 - rev 3103270
Push 578593 by jdemooij@mozilla.com at Tue, 04 Aug 2020 06:25:18 +0000
Bug 1655940 part 4 - Optimize Array constructor in CacheIR and Warp. r?anba! This optimizes the Array() and Array(n) cases. Differential Revision: https://phabricator.services.mozilla.com/D85722
4fbadba916635fe6f6193728682787c4fc317af0: try_task_config for https://phabricator.services.mozilla.com/D85722 draft
libmozevent <release-mgmt-analysis@mozilla.com> - Tue, 04 Aug 2020 06:21:39 +0000 - rev 3103269
Push 578592 by reviewbot at Tue, 04 Aug 2020 06:23:34 +0000
try_task_config for https://phabricator.services.mozilla.com/D85722 Differential Diff: PHID-DIFF-teejvhplceew2ce5lndl
03e64eb4edb6c1c2c211783d5676af8bea7b45d9: Bug 1655940 part 4 - Optimize Array constructor in CacheIR and Warp. r?anba! draft
Jan de Mooij <jdemooij@mozilla.com> - Tue, 04 Aug 2020 06:21:36 +0000 - rev 3103268
Push 578592 by reviewbot at Tue, 04 Aug 2020 06:23:34 +0000
Bug 1655940 part 4 - Optimize Array constructor in CacheIR and Warp. r?anba! Summary: This optimizes the Array() and Array(n) cases. Differential Revision: https://phabricator.services.mozilla.com/D85722 Test Plan: Reviewers: anba! Subscribers: Bug #: 1655940 Differential Diff: PHID-DIFF-teejvhplceew2ce5lndl
66ceb7c3069beefc41b57786cf4424ddacdbba15: try_task_config for https://phabricator.services.mozilla.com/D85815 draft
libmozevent <release-mgmt-analysis@mozilla.com> - Tue, 04 Aug 2020 06:17:50 +0000 - rev 3103267
Push 578591 by reviewbot at Tue, 04 Aug 2020 06:18:26 +0000
try_task_config for https://phabricator.services.mozilla.com/D85815 Differential Diff: PHID-DIFF-srm5g7nkd4mkevbh74tb
6a4b43d0af776b15d256f40da49cdbe32952a625: Bug 1656888 - Load ad-hoc console listener module in worker context. r=ochameau. draft
Nicolas Chevobbe <nchevobbe@mozilla.com> - Tue, 04 Aug 2020 06:17:46 +0000 - rev 3103266
Push 578591 by reviewbot at Tue, 04 Aug 2020 06:18:26 +0000
Bug 1656888 - Load ad-hoc console listener module in worker context. r=ochameau. Summary: Depends on D85814 Test Plan: Reviewers: ochameau Subscribers: Bug #: 1656888 Differential Diff: PHID-DIFF-srm5g7nkd4mkevbh74tb
0b16ef504638c80be60bf61c611ad4b500f882ae: Bug 1656888 - Make WorkerThread's mock parent a Pool so it's resource API ready. r=ochameau. draft
Nicolas Chevobbe <nchevobbe@mozilla.com> - Tue, 04 Aug 2020 06:17:43 +0000 - rev 3103265
Push 578591 by reviewbot at Tue, 04 Aug 2020 06:18:26 +0000
Bug 1656888 - Make WorkerThread's mock parent a Pool so it's resource API ready. r=ochameau. Summary: The `watchResources` function [1] takes as a first parameter either a watcher actor or a target actor, that is used to manage the created resources (e.g. the object actors of console messages). In order to be able to manage actors, it needs to extend the `Pool` class (which is the case for actors). In the worker cases, things are a bit different, since we create a DevToolServer which only have a WebConsole and a Thread actor. Those actors usually take a target actor as a parent, but in the Worker thread we simply have an object implementing the properties and methods it needs. This will be this "parent" that we're going to pass to `watchResources`, and in order to be able to do that, it needs to be able to manage actors. In regard of all this, this patch makes the "parent" a Pool, on which we then put the existing methods and properties. We take this opportunity to store the parent in the connections object we have, in order to be able to retrieve it later (e.g. when the WorkerTargetActor on the content process main thread will notify the Worker thread about resources to watch/unwatch). [1] https://searchfox.org/mozilla-central/rev/0e309417bdc73be33fc93f6bd7a0d4063d2c6e51/devtools/server/actors/resources/index.js#66 Depends on D85813 Test Plan: Reviewers: ochameau Subscribers: Bug #: 1656888 Differential Diff: PHID-DIFF-5k7njzxj7sthqm6a5ivy
8d0b7b87a069abb16001351317263f803a5af437: Bug 1656888 - Add test cases to browser_target_list_workers. r=jdescottes. draft
Nicolas Chevobbe <nchevobbe@mozilla.com> - Tue, 04 Aug 2020 06:17:40 +0000 - rev 3103264
Push 578591 by reviewbot at Tue, 04 Aug 2020 06:18:26 +0000
Bug 1656888 - Add test cases to browser_target_list_workers. r=jdescottes. Summary: Numerous test cases are added to check behaviour of the targetList when workers: - already exists - are spawned - are terminated in same document or remote iframe. The test is tagged as fail-if when fission is enabled since we don't get the workers for remote iframes. Since the iframe now has a service worker (on example.org), we need to unregister it in browser_target_list_service_workers_navigation as it's not the expected service worker. Depends on D85812 Test Plan: Reviewers: jdescottes Subscribers: Bug #: 1656888 Differential Diff: PHID-DIFF-g3b73bjgbm6ts2ivp7hx
8d717b1c4e2dcc13bc86c6589d2e4dfe1faacf1b: Bug 1656888 - Make the iframe in fission_document.html a remote iframe. r=jdescottes. draft
Nicolas Chevobbe <nchevobbe@mozilla.com> - Tue, 04 Aug 2020 06:17:38 +0000 - rev 3103263
Push 578591 by reviewbot at Tue, 04 Aug 2020 06:18:26 +0000
Bug 1656888 - Make the iframe in fission_document.html a remote iframe. r=jdescottes. Summary: fission_document.html's iframe was on example.com while the top document is always on example.com as well, so we weren't exercising fission in it. This patch puts the iframe on example.org, which highlighted some TargetList test failures that this patch addresses. Test Plan: Reviewers: jdescottes Subscribers: Bug #: 1656888 Differential Diff: PHID-DIFF-uyelufyfqxfzz3n2ihkf
8f93f57b9160811393ae05305911e4a1616d672f: Try Chooser Enhanced (79 tasks selected) draft
alwu <alwu@mozilla.com> - Mon, 03 Aug 2020 23:17:10 -0700 - rev 3103262
Push 578590 by alwu@mozilla.com at Tue, 04 Aug 2020 06:18:07 +0000
Try Chooser Enhanced (79 tasks selected) Pushed via `mach try again`
14c5303d8e3b62745b1017b0f3a0024dcddd2e4c: Bug XXX - part3 : implement a new wakelock listener with newer API. draft
alwu <alwu@mozilla.com> - Mon, 03 Aug 2020 23:16:35 -0700 - rev 3103261
Push 578590 by alwu@mozilla.com at Tue, 04 Aug 2020 06:18:07 +0000
Bug XXX - part3 : implement a new wakelock listener with newer API.
5fd9e3f3abe8f12f4dc285ac9747845629c45e79: Bug XXX - part2 : rename WinWakeLockListener to LegacyWinWakeLockListener. draft
alwu <alwu@mozilla.com> - Mon, 03 Aug 2020 22:56:21 -0700 - rev 3103260
Push 578590 by alwu@mozilla.com at Tue, 04 Aug 2020 06:18:07 +0000
Bug XXX - part2 : rename WinWakeLockListener to LegacyWinWakeLockListener.
90b72d93ddad45be60bd331d8930bf5d3aa81695: Only one wakelock listener. draft
alwu <alwu@mozilla.com> - Mon, 03 Aug 2020 21:34:28 -0700 - rev 3103259
Push 578590 by alwu@mozilla.com at Tue, 04 Aug 2020 06:18:07 +0000
Only one wakelock listener.
1fda0a69166a6995a95f17b8f034c20489d3105a: try_task_config for https://phabricator.services.mozilla.com/D85814 draft
libmozevent <release-mgmt-analysis@mozilla.com> - Tue, 04 Aug 2020 06:16:56 +0000 - rev 3103258
Push 578589 by reviewbot at Tue, 04 Aug 2020 06:17:22 +0000
try_task_config for https://phabricator.services.mozilla.com/D85814 Differential Diff: PHID-DIFF-5k7njzxj7sthqm6a5ivy
a41dc937f7868e6463e557a29986b22040521524: Bug 1656888 - Make WorkerThread's mock parent a Pool so it's resource API ready. r=ochameau. draft
Nicolas Chevobbe <nchevobbe@mozilla.com> - Tue, 04 Aug 2020 06:16:52 +0000 - rev 3103257
Push 578589 by reviewbot at Tue, 04 Aug 2020 06:17:22 +0000
Bug 1656888 - Make WorkerThread's mock parent a Pool so it's resource API ready. r=ochameau. Summary: The `watchResources` function [1] takes as a first parameter either a watcher actor or a target actor, that is used to manage the created resources (e.g. the object actors of console messages). In order to be able to manage actors, it needs to extend the `Pool` class (which is the case for actors). In the worker cases, things are a bit different, since we create a DevToolServer which only have a WebConsole and a Thread actor. Those actors usually take a target actor as a parent, but in the Worker thread we simply have an object implementing the properties and methods it needs. This will be this "parent" that we're going to pass to `watchResources`, and in order to be able to do that, it needs to be able to manage actors. In regard of all this, this patch makes the "parent" a Pool, on which we then put the existing methods and properties. We take this opportunity to store the parent in the connections object we have, in order to be able to retrieve it later (e.g. when the WorkerTargetActor on the content process main thread will notify the Worker thread about resources to watch/unwatch). [1] https://searchfox.org/mozilla-central/rev/0e309417bdc73be33fc93f6bd7a0d4063d2c6e51/devtools/server/actors/resources/index.js#66 Depends on D85813 Test Plan: Reviewers: ochameau Subscribers: Bug #: 1656888 Differential Diff: PHID-DIFF-5k7njzxj7sthqm6a5ivy
ae2e05c27a2e47696a6de1eb63329554e7777e5c: Bug 1656888 - Add test cases to browser_target_list_workers. r=jdescottes. draft
Nicolas Chevobbe <nchevobbe@mozilla.com> - Tue, 04 Aug 2020 06:16:49 +0000 - rev 3103256
Push 578589 by reviewbot at Tue, 04 Aug 2020 06:17:22 +0000
Bug 1656888 - Add test cases to browser_target_list_workers. r=jdescottes. Summary: Numerous test cases are added to check behaviour of the targetList when workers: - already exists - are spawned - are terminated in same document or remote iframe. The test is tagged as fail-if when fission is enabled since we don't get the workers for remote iframes. Since the iframe now has a service worker (on example.org), we need to unregister it in browser_target_list_service_workers_navigation as it's not the expected service worker. Depends on D85812 Test Plan: Reviewers: jdescottes Subscribers: Bug #: 1656888 Differential Diff: PHID-DIFF-g3b73bjgbm6ts2ivp7hx
b7884ee4c88fa5216d2781eb0453071cfe83f03d: Bug 1656888 - Make the iframe in fission_document.html a remote iframe. r=jdescottes. draft
Nicolas Chevobbe <nchevobbe@mozilla.com> - Tue, 04 Aug 2020 06:16:44 +0000 - rev 3103255
Push 578589 by reviewbot at Tue, 04 Aug 2020 06:17:22 +0000
Bug 1656888 - Make the iframe in fission_document.html a remote iframe. r=jdescottes. Summary: fission_document.html's iframe was on example.com while the top document is always on example.com as well, so we weren't exercising fission in it. This patch puts the iframe on example.org, which highlighted some TargetList test failures that this patch addresses. Test Plan: Reviewers: jdescottes Subscribers: Bug #: 1656888 Differential Diff: PHID-DIFF-uyelufyfqxfzz3n2ihkf
4ef1a321e7510d04b1a639581927c9f3eb37f7c1: Try Chooser Enhanced (108 tasks selected) draft
Masayuki Nakano <masayuki@d-toybox.com> - Tue, 04 Aug 2020 15:16:52 +0900 - rev 3103254
Push 578588 by masayuki@d-toybox.com at Tue, 04 Aug 2020 06:17:14 +0000
Try Chooser Enhanced (108 tasks selected) Pushed via `mach try chooser`
ab92c3994c75a5fbcbb0e23f9ffc9a814b5f9ba1: Bug 1656799 - part 3: Make `WhiteSpaceVisibilityKeeper::PrepareToDeleteRange()` not track `EditorDOMPoint`s r=m_kato draft
Masayuki Nakano <masayuki@d-toybox.com> - Tue, 04 Aug 2020 15:15:58 +0900 - rev 3103253
Push 578588 by masayuki@d-toybox.com at Tue, 04 Aug 2020 06:17:14 +0000
Bug 1656799 - part 3: Make `WhiteSpaceVisibilityKeeper::PrepareToDeleteRange()` not track `EditorDOMPoint`s r=m_kato It may be faster to use `AutoTrackDOMRange` directly. Therefore, current `WhiteSpaceVisibilityKeeper::PrepareToDeleteRange()` should be renamed to `WhiteSpaceVisibilityKeeper::PrepareToDeleteRangeAndTrackPoints()` and we should make `HTMLEditor::HandleDeleteNonCollapsedSelection()` track the range by itself.
1475cabde5aa946157467c5eb0504b7a8f8aedc0: Bug 1656799 - part 2: Create range tracker r=m_kato draft
Masayuki Nakano <masayuki@d-toybox.com> - Mon, 03 Aug 2020 11:07:48 +0900 - rev 3103252
Push 578588 by masayuki@d-toybox.com at Tue, 04 Aug 2020 06:17:14 +0000
Bug 1656799 - part 2: Create range tracker r=m_kato There is only `AutoTrackDOMPoint`, but `HTMLEditor::HandleDeleteNonCollapsedSelection()` requires to tack pairs of DOM points, i.e., requires to track ranges. For making the code simpler, we should have range tracker too.
ec6cf0a1827bdcf4f68fedd8a24c9103092c18a3: Bug 1656799 - part 1: Make `HTMLEditor::HandleDeleteNonCollapsedSelection()` store selection ranges with `AutoRangeArray` first r=m_kato draft
Masayuki Nakano <masayuki@d-toybox.com> - Mon, 03 Aug 2020 10:04:37 +0900 - rev 3103251
Push 578588 by masayuki@d-toybox.com at Tue, 04 Aug 2020 06:17:14 +0000
Bug 1656799 - part 1: Make `HTMLEditor::HandleDeleteNonCollapsedSelection()` store selection ranges with `AutoRangeArray` first r=m_kato
(0) -3000000 -1000000 -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 +300000 +1000000 tip