ae841c23486e561fb9ce36539149c7b5e9a5ffd3: Bug 1291715 - Add RTCDTMFSender and RTCDTMFToneChangeEvent to webidl; r=smaug
draft
Dan Minor <dminor@mozilla.com> - Wed, 14 Sep 2016 08:53:23 -0400 - rev 413578
Push
29449 by dminor@mozilla.com at Wed, 14 Sep 2016 12:58:37 +0000
Bug 1291715 - Add RTCDTMFSender and RTCDTMFToneChangeEvent to webidl; r=smaug
MozReview-Commit-ID: FvAp6DQex02
d2b7bce2df8a4ad68b7078b2731aedeb81639cc8: Bug 1156427 - Fix test_set_location_through_execute_script by waiting for the page to be loaded. r?ato
draft
Henrik Skupin <mail@hskupin.info> - Thu, 08 Sep 2016 15:32:59 +0200 - rev 413577
Push
29448 by bmo:hskupin@gmail.com at Wed, 14 Sep 2016 12:46:00 +0000
Bug 1156427 - Fix test_set_location_through_execute_script by waiting for the page to be loaded. r?ato
10ebec133080afa4378a4ec67f7cfcd7d081253f: Bug 1260552 - Fix clipboard and context menu tests;r=honza
draft
Julian Descottes <jdescottes@mozilla.com> - Wed, 14 Sep 2016 11:12:21 +0200 - rev 413576
Push
29447 by jdescottes@mozilla.com at Wed, 14 Sep 2016 12:34:56 +0000
Bug 1260552 - Fix clipboard and context menu tests;r=honza
MozReview-Commit-ID: CxG2pVikq3g
6115e8fdf46187342126e10bdb6643f58980d0be: Bug 1260552 - Fixing test failures; r=pbro
draft
Jan Odvarko <odvarko@gmail.com> - Tue, 13 Sep 2016 15:32:20 +0200 - rev 413575
Push
29447 by jdescottes@mozilla.com at Wed, 14 Sep 2016 12:34:56 +0000
Bug 1260552 - Fixing test failures; r=pbro
MozReview-Commit-ID: G4WQq9FPzaO
3604472f097a482459cf8f7092c7d398963ff875: Bug 1260552 - Replace xul:spliter by html:splitter; r=pbro
draft
Jan Odvarko <odvarko@gmail.com> - Tue, 13 Sep 2016 15:32:00 +0200 - rev 413574
Push
29447 by jdescottes@mozilla.com at Wed, 14 Sep 2016 12:34:56 +0000
Bug 1260552 - Replace xul:spliter by html:splitter; r=pbro
MozReview-Commit-ID: LlWKoZGT5Z0
7522444ff5fabf7102ce1e73fc0cf182c91a94d1: Bug 1260552 - Implement Splitter React component; r=jlongster, pbro
draft
Jan Odvarko <odvarko@gmail.com> - Tue, 13 Sep 2016 14:58:22 +0200 - rev 413573
Push
29447 by jdescottes@mozilla.com at Wed, 14 Sep 2016 12:34:56 +0000
Bug 1260552 - Implement Splitter React component; r=jlongster, pbro
MozReview-Commit-ID: LeMSMNO5UnN
460ed2d483bf1f90e3cec2506348298ace1151cd: Bug 1263458 - Force a full-page snapshot to make sure the scrollbars get included as well. r?botond
draft
Kartikaya Gupta <kgupta@mozilla.com> - Wed, 14 Sep 2016 08:11:27 -0400 - rev 413572
Push
29446 by kgupta@mozilla.com at Wed, 14 Sep 2016 12:12:15 +0000
Bug 1263458 - Force a full-page snapshot to make sure the scrollbars get included as well. r?botond
The reftest-no-flush avoids some issues on Fennec where the reftest harness
gets into an infinite loop of snapshotting because the pending-paint flag never
gets cleared. The flag seems to be set as a result of the APZ flush and I'm
unclear on why it doesn't get cleared normally, but this bypasses the problem.
MozReview-Commit-ID: 8SGoSkRo5nm
155f13a40d0f775674959f3aeb64078db9f1d65a: Bug 1263458 - Force a full-page snapshot to make sure the scrollbars get included as well. r?botond
draft
Kartikaya Gupta <kgupta@mozilla.com> - Wed, 14 Sep 2016 08:07:23 -0400 - rev 413571
Push
29445 by kgupta@mozilla.com at Wed, 14 Sep 2016 12:09:36 +0000
Bug 1263458 - Force a full-page snapshot to make sure the scrollbars get included as well. r?botond
The reftest-no-flush avoids some issues on Fennec where the reftest harness
gets into an infinite loop of snapshotting because the pending-paint flag never
gets cleared. The flag seems to be set as a result of the APZ flush and I'm
unclear on why it doesn't get cleared normally, but this bypasses the problem.
MozReview-Commit-ID: 9H23iylOAm3
ea432f4e420602bc8c53fbde58e9cc2641cf1099: Bug 1287007 - Move native messaging to child process
draft
Rob Wu <rob@robwu.nl> - Wed, 14 Sep 2016 03:52:35 -0700 - rev 413570
Push
29444 by bmo:rob@robwu.nl at Wed, 14 Sep 2016 11:43:49 +0000
Bug 1287007 - Move native messaging to child process
Move `runtime.connectNative` and `runtime.sendNativeMessage` to
`addon_child`. Note: This does not change the behavior for launching the
native app, it is still launched from the main process.
Some other significant changes:
ExtensionUtils.Port:
- Refactor Port in ExtensionUtils to allow using Port without wrappers
via `registerOnMessage` and `registerOnDisconnect`.
- Add "port" parameter to Port's `onMessage` method (Chrome compat).
- Do not clone the message received by Port's `onMessage` (per & Chrome
compat).
ExtensionUtils.Messenger
- Add connectGetRawPort to allow `postMessage` with already-sanitized
values without having to use `cloneInto` with `context.cloneScope`.
Native messaging:
- Do not throw an error when `disconnect()` is called on an
already-disconnected port (Chrome compat).
- Do not dispatch `onDisconnect` when `.disconnect()` was called (Chrome
compat).
- Use `context.extension` instead of passing the extension separately.
- Serialize the message in the child instead of the main process.
MozReview-Commit-ID: FJUJYSb3A81
08e7c80bf057e7bdf25bd65f25419ca381870d34: Bug 1287007 - Make window.close in extension pages async
draft
Rob Wu <rob@robwu.nl> - Tue, 13 Sep 2016 20:26:18 -0700 - rev 413569
Push
29444 by bmo:rob@robwu.nl at Wed, 14 Sep 2016 11:43:49 +0000
Bug 1287007 - Make window.close in extension pages async
Test coverage by tabs.onRemoved + window.close() in:
toolkit/components/extensions/test/mochitest/test_ext_tab_teardown.html
MozReview-Commit-ID: 7asg2XGrTaQ
be1ba7b98adef1c913257e1fb1111d8ec4bf6ebd: Bug 1287007 - Remove .contentWindow from ProxyContext
draft
Rob Wu <rob@robwu.nl> - Tue, 13 Sep 2016 16:02:03 -0700 - rev 413568
Push
29444 by bmo:rob@robwu.nl at Wed, 14 Sep 2016 11:43:49 +0000
Bug 1287007 - Remove .contentWindow from ProxyContext
MozReview-Commit-ID: ASNigrM07yz
e9a477358808ec61aba623bb75463df0505e2a03: Bug 1287007 - Fix test_ext_management_uninstall_self.js
draft
Rob Wu <rob@robwu.nl> - Tue, 13 Sep 2016 13:59:37 -0700 - rev 413567
Push
29444 by bmo:rob@robwu.nl at Wed, 14 Sep 2016 11:43:49 +0000
Bug 1287007 - Fix test_ext_management_uninstall_self.js
The test logic was broken by design: Two tests uninstall the addon,
but only one uninstall observer was used. Consequently, the second
test resumes the test before the addon was actually uninstalled.
It is probably sheer luck that the test worked before.
MozReview-Commit-ID: DcT48ZQ2bRp
85697f18b4eabd1bae3d11b6d6e61fb51a1baf54: Bug 1287007 - Enable proxying of most APIs.
draft
Rob Wu <rob@robwu.nl> - Mon, 12 Sep 2016 21:19:53 -0700 - rev 413566
Push
29444 by bmo:rob@robwu.nl at Wed, 14 Sep 2016 11:43:49 +0000
Bug 1287007 - Enable proxying of most APIs.
MozReview-Commit-ID: KrI42XrsVVG
b934fc30e5acfd4ceb03b6cfc4c212b85b98d9ee: Bug 1287007 - Fix timing issue in test_ext_storage.js
draft
Rob Wu <rob@robwu.nl> - Tue, 13 Sep 2016 00:45:00 -0700 - rev 413565
Push
29444 by bmo:rob@robwu.nl at Wed, 14 Sep 2016 11:43:49 +0000
Bug 1287007 - Fix timing issue in test_ext_storage.js
MozReview-Commit-ID: 63Ra7ThuHuf
eb98a6b4f6a513f7eea7cf3b40960011d2d1c783: Bug 1287007 - Fix "onclick" in contextMenus, to child.
draft
Rob Wu <rob@robwu.nl> - Mon, 12 Sep 2016 18:26:03 -0700 - rev 413564
Push
29444 by bmo:rob@robwu.nl at Wed, 14 Sep 2016 11:43:49 +0000
Bug 1287007 - Fix "onclick" in contextMenus, to child.
Main thing: Making contextMenus implementation webext-oop compatible.
Preparation:
- Add getParentEvent to ChildAPIManager to allow use of remote events.
- Introduce `addon_parent_only` "restriction" to only generate a schema
API in the main process.
- Do not fill in `null` for missing keys if the schema declares a key as
`"optional": "omit-key-if-missing"`. This is needed for the second
point in the next list.
Drive-by fixes:
- Ensure that the "onclick" handler is erased when a context closes.
- Do not clear the "onclick" handler in `contextMenus.update` if the
onclick key has been omitted (parity with Chrome).
- Remove some unnecessary `Promise.resolve()`
- Add extensive set of tests that check the behavior of the contextMenus
APIs with regards to the onclick attribute in various scenarios.
MozReview-Commit-ID: A5f3AUQzU8T
26b2bf4a56bbd1e6715119028c5800c2e0e78fd1: Bug 1287007 - Move native messaging to child process
draft
Rob Wu <rob@robwu.nl> - Wed, 14 Sep 2016 03:52:35 -0700 - rev 413563
Push
29443 by bmo:rob@robwu.nl at Wed, 14 Sep 2016 11:38:34 +0000
Bug 1287007 - Move native messaging to child process
Move `runtime.connectNative` and `runtime.sendNativeMessage` to
`addon_child`. Note: This does not change the behavior for launching the
native app, it is still launched from the main process.
Some other significant changes:
ExtensionUtils.Port:
- Refactor Port in ExtensionUtils to allow using Port without wrappers
via `registerOnMessage` and `registerOnDisconnect`.
- Add "port" parameter to Port's `onMessage` method (Chrome compat).
- Do not clone the message received by Port's `onMessage` (per & Chrome
compat).
ExtensionUtils.Messenger
- Add connectGetRawPort to allow `postMessage` with already-sanitized
values without having to use `cloneInto` with `context.cloneScope`.
Native messaging:
- Do not throw an error when `disconnect()` is called on an
already-disconnected port (Chrome compat).
- Do not dispatch `onDisconnect` when `.disconnect()` was called (Chrome
compat).
- Use `context.extension` instead of passing the extension separately.
- Serialize the message in the child instead of the main process.
MozReview-Commit-ID: FJUJYSb3A81
03c55f22c43d242e447d27bf3aa3860964e3ea7e: Bug 1287007 - Make window.close in extension pages async
draft
Rob Wu <rob@robwu.nl> - Tue, 13 Sep 2016 20:26:18 -0700 - rev 413562
Push
29443 by bmo:rob@robwu.nl at Wed, 14 Sep 2016 11:38:34 +0000
Bug 1287007 - Make window.close in extension pages async
Test coverage by tabs.onRemoved + window.close() in:
toolkit/components/extensions/test/mochitest/test_ext_tab_teardown.html
MozReview-Commit-ID: 7asg2XGrTaQ
171975f459f452d1ab18f9cb9782cf7d2186cba9: Bug 1287007 - Remove .contentWindow from ProxyContext
draft
Rob Wu <rob@robwu.nl> - Tue, 13 Sep 2016 16:02:03 -0700 - rev 413561
Push
29443 by bmo:rob@robwu.nl at Wed, 14 Sep 2016 11:38:34 +0000
Bug 1287007 - Remove .contentWindow from ProxyContext
MozReview-Commit-ID: ASNigrM07yz
4b6292446a899f1c743dae0d4a9c988db8ec9097: Bug 1287007 - Fix test_ext_management_uninstall_self.js
draft
Rob Wu <rob@robwu.nl> - Tue, 13 Sep 2016 13:59:37 -0700 - rev 413560
Push
29443 by bmo:rob@robwu.nl at Wed, 14 Sep 2016 11:38:34 +0000
Bug 1287007 - Fix test_ext_management_uninstall_self.js
The test logic was broken by design: Two tests uninstall the addon,
but only one uninstall observer was used. Consequently, the second
test resumes the test before the addon was actually uninstalled.
It is probably sheer luck that the test worked before.
MozReview-Commit-ID: DcT48ZQ2bRp
ab9b008af32878cb7191b49a74b026519a6723b4: Bug 1287007 - Enable proxying of most APIs.
draft
Rob Wu <rob@robwu.nl> - Mon, 12 Sep 2016 21:19:53 -0700 - rev 413559
Push
29443 by bmo:rob@robwu.nl at Wed, 14 Sep 2016 11:38:34 +0000
Bug 1287007 - Enable proxying of most APIs.
MozReview-Commit-ID: KrI42XrsVVG