b48486fff5210dea14d87804ab57c6dc5353ee56: Bug 1386445 - Early about:newtab pages are missing message APIs because RemotePages registers too late. r=mossop
Ed Lee <edilee@mozilla.com> - Tue, 01 Aug 2017 12:30:33 -0700 - rev 422479
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1386445 - Early about:newtab pages are missing message APIs because RemotePages registers too late. r=mossop This moves AboutNewTab.init from nsBrowserGlue.js handling of "browser-delayed-startup-finished" into aboutNewTabService.js so that when the service is loaded once from the main thread probably by browser.js towards the beginning of _delayedStartup just before potentially calling gBrowser.loadTabs, the service triggers the attaching of RemotePages(about:newtab) before any about:newtab pages load. Additionally even when RemotePages starts early enough, Activity Stream might not borrow the RemotePages instance early enough to catch the RemotePage:Load message, so to simulate that, RemotePages now remembers when a port has been loaded for consumers to check. Adds tests to confirm the expected properties on the port and value of loaded at the various RemotePage:* messages. MozReview-Commit-ID: IXJLvFCgbEH
2f39e4fb4ff15c395d6e3af74843070e17933e89: Bug 1376651 - Pass the nsIScriptElement instead of allocating a string every time r=ckerschb
Kate McKinley <kmckinley@mozilla.com> - Thu, 27 Jul 2017 11:01:24 -0700 - rev 422478
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1376651 - Pass the nsIScriptElement instead of allocating a string every time r=ckerschb Change the interface to GetAlowsInline to take an nsISupports* instead of a string, and pass the nsIScriptElement directly. If we don't have an element, then pass nullptr or the mock string created as an nsISupportsString. MozReview-Commit-ID: pgIMxtplsi
ec588b600d329b06ce2fc8955699127b571d67ee: bug 1356623 - remove now-unnecessary CNNIC certificate whitelist r=jcj
David Keeler <dkeeler@mozilla.com> - Thu, 03 Aug 2017 16:17:11 -0700 - rev 422477
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
bug 1356623 - remove now-unnecessary CNNIC certificate whitelist r=jcj As a result of CNNIC issuing an unconstrained intermediate certificate that misissued an end-entity certificate for google.com (see bug 1146026 and bug 1177209), we implemented a system that would in theory enable Firefox to continue to trust certificates that were valid at the time but not newly issued certificates. This consisted of a whitelist added in bug 1151512. The CNNIC roots have since been removed from NSS in bug 1380868. We can now remove the whitelist in Firefox. MozReview-Commit-ID: 7VXOuvwzbct
ca0419d7845fdc7cd922459cdd0f334eb56a2dfc: Bug 1385864 - Drop invalid optional_permissions from manifests r=kmag
Tomislav Jovanovic <tomica@gmail.com> - Fri, 04 Aug 2017 00:26:50 +0200 - rev 422476
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1385864 - Drop invalid optional_permissions from manifests r=kmag MozReview-Commit-ID: Df2wjNOTVkF
693f52e6fb26721453819c012a8134e694ade503: Bug 379797 - Various dump() methods should check browser.dom.window.dump.enabled. r=krizsa
Andrew McCreight <continuation@gmail.com> - Wed, 02 Aug 2017 11:56:33 -0700 - rev 422475
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 379797 - Various dump() methods should check browser.dom.window.dump.enabled. r=krizsa Checking this pref to avoid log spam in opt builds, in sandboxes, JS components, and whatever uses nsFrameMessageManager's dump method. This does mean that on Windows in an opt build when a debugger is present a debug string will no longer be printed unless the pref is set, but I think that is consistent with the non-Windows behavior. MozReview-Commit-ID: FWLAzBRVhlx
cd62bc7fffe4a0e01621872eee968fe656d04224: Bug 1352075 - Implement new animation for opening/closing the arrow panels. r=dao,mossop
Sam Foster <sfoster@mozilla.com> - Thu, 27 Jul 2017 11:07:23 -0700 - rev 422474
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1352075 - Implement new animation for opening/closing the arrow panels. r=dao,mossop * Use new panel animation when opening arrow-panels (including bookmarks menu) to fade in and drop into position * Linux/GTK is (still) excluded * New animation is non-directional (i.e. LTR vs. RTL) This was landed then backed out due to test failures. New since last review: * Make opacity & transform transition durations equal - ensuring popup is not still moving when popupshown is fired * Fix missing comma in transition-duration values * Add animating attribute to the arrowpanel binding to disable pointer-events during the opening transition (via :jaws) * Wait for popupshown rather than transitionend in bookmark reparenting test * Fix specificity of CSS rules for panels/bookmarks-menu on edges other than the top (via :jaws) MozReview-Commit-ID: DTnvyMryf5Y
9e8f2f0c7730a682427b8c11d44a166de43a2225: merge mozilla-central to autoland. r=merge a=merge
Sebastian Hengst <archaeopteryx@coole-files.de> - Fri, 04 Aug 2017 19:31:08 +0200 - rev 422473
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
merge mozilla-central to autoland. r=merge a=merge
f9b9ffb6fea52259bbcf00eca45285a930f0ca59: Bug 1383242 - Properly compare node to traversal range under different modes; r=smaug
Jim Chen <nchen@mozilla.com> - Wed, 02 Aug 2017 13:48:09 -0400 - rev 422472
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1383242 - Properly compare node to traversal range under different modes; r=smaug When the node borders one of the range bounds, `NodeIsInTraversalRange` should return different results depending on whether it's in pre mode or not. > <div><br></div> > \__/ In this pre mode example, the node <br> is within the range, and the node position (which is at the start of the node in pre mode) and the start bound are both (<div>, 0). Therefore, it shows the start bound should be inclusive in pre mode. > <div><br></div> > \___/ In this pre mode example, the node <br> is outside of the range, yet the node position and the end bound are both (<div>, 0). Therefore, it shows the end bound should be exclusive in pre mode. > <div><br></div> > \____/ in this post mode example, the node <br> is outside of the range, yet the node position (which is at the end of the node in post mode) and the start bound are both (<div>, 1). Therefore, it shows the start bound should be exclusive in post mode. > <div><br></div> > \__/ In this post mode example, the node <br> is within the range, and the node position and the end bound are both (<div>, 1). Therefore, it shows the end bound should be inclusive in post mode. In summary, the correct pre mode bound check is `start <= node < end`, and the correct post mode bound check is `start < node <= end`. This patch fixes `NodeIsInTraversalRange` to have the correct bounds check. MozReview-Commit-ID: IjJN1ua6jQ9
274953221bdfcec279aa53c02d1aa96d441a4359: Bug 1387219 - Remove code WakeLockListener support on XP and Vista. r=aklotz
Chris Pearce <cpearce@mozilla.com> - Fri, 04 Aug 2017 11:44:29 +1200 - rev 422471
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1387219 - Remove code WakeLockListener support on XP and Vista. r=aklotz billm reports that the runnable in this code is firing a lot, and since we don't support XP/Vista we're not benefiting from this overhead. MozReview-Commit-ID: Bpw1E9DxPpD
ebaccd4031d2e487300a24155b8ca57767d028b3: Bug 1383473 - Minor Styling fixes in the hamburger menu. r=Gijs
Erica Wright <ewright@mozilla.com> - Wed, 02 Aug 2017 17:03:49 -0400 - rev 422470
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1383473 - Minor Styling fixes in the hamburger menu. r=Gijs MozReview-Commit-ID: 2XJau2Yc63m
00b9e03ce554a5a7af93893b26ff5454c0337e8c: servo: Merge #17923 - Fix fixed position items with parents with CSS clips (from mrobinson:fixed-position-css-clip); r=emilio
Martin Robinson <mrobinson@igalia.com> - Fri, 04 Aug 2017 11:17:24 -0500 - rev 422469
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
servo: Merge #17923 - Fix fixed position items with parents with CSS clips (from mrobinson:fixed-position-css-clip); r=emilio In order to properly handle CSS clipping, we need to keep track of what the different kinds of clips that we have. On one hand, clipping due to overflow rules should respect the containing block hierarchy, while CSS clipping should respect the flow tree hierarchy. In order to represent the complexity of items that are scrolled via one clip/scroll frame and clipped by another we keep track of that status with a ClipAndScrollInfo. <!-- Please describe your changes on the following line: --> --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [x] `./mach build -d` does not report any errors - [x] `./mach test-tidy` does not report any errors - [ ] These changes fix #__ (github issue number if applicable). <!-- Either: --> - [x] There are tests for these changes OR - [ ] These changes do not require tests because _____ <!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.--> <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> Source-Repo: https://github.com/servo/servo Source-Revision: 49615284d0f45646da917f7dda22a1103d12974d
95a0b91cbd3432b0b8e8b23b21c2b9628ebd7dba: Bug 1387359 - add developer toggle to CustomizableUI even if devtools already loaded;r=nchevobbe
Julian Descottes <jdescottes@mozilla.com> - Fri, 04 Aug 2017 11:09:56 +0200 - rev 422468
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1387359 - add developer toggle to CustomizableUI even if devtools already loaded;r=nchevobbe After Bug 1359855, the developer toggle is added to CustomizableUI via devtools-startup. It is added after browser-delayed-startup-finished is received. However when starting devtools with --jsconsole, devtools will be initialized before we receive the event and the toggle will never be added to the UI. Introducing a new flag here dedicated to the developer toggle and adding a few comments to the devtools-startup:hookWindow() method which becomes a bit complex now. MozReview-Commit-ID: IhPFznt0O83
1256610b4f380b2b1b064252ccda925f097b3575: Bug 1387495 - dt-addon: use window object in webconsole/net/main to load stylesheets;r=jryans
Julian Descottes <jdescottes@mozilla.com> - Wed, 12 Jul 2017 19:26:33 +0200 - rev 422467
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1387495 - dt-addon: use window object in webconsole/net/main to load stylesheets;r=jryans From console/net/main.js we are calling loadSheet from sdk/stylesheet/utils. This API needs a real window object to work, but now that devtools are loaded as a system addon, by default the global object is a sandbox wrapper. Use the window object which points to the actual Window instead. MozReview-Commit-ID: LxDNfDiOso3
2d15b3c8b58caa1beed80c036d7c43d2f74b7a5a: Bug 1387026 - Fix Android browserAction corrupting legacy Addon menu items on overlapping menu item id. r=sebastian
Luca Greco <lgreco@mozilla.com> - Wed, 02 Aug 2017 21:29:18 +0200 - rev 422466
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1387026 - Fix Android browserAction corrupting legacy Addon menu items on overlapping menu item id. r=sebastian MozReview-Commit-ID: FWkJdAWu5Td
3af71fa1f3dd1cf6083f84612c2fb99ed765d2fb: Bug 1386777 - Interpret autoscroll anchor coordinates correctly when there is a device scale. r=kats
Botond Ballo <botond@mozilla.com> - Wed, 02 Aug 2017 20:11:07 -0400 - rev 422465
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1386777 - Interpret autoscroll anchor coordinates correctly when there is a device scale. r=kats MozReview-Commit-ID: 6caXxWjuqxm
044a5b82b056e9fc39c77c4356687343c8777d16: Bug 1207579 - Make PinSiteDialog public to ensure it can be accessed from v4.app.fragment r=snorp
Michael Kaply <mozilla@kaply.com> - Thu, 03 Aug 2017 15:16:40 -0500 - rev 422464
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1207579 - Make PinSiteDialog public to ensure it can be accessed from v4.app.fragment r=snorp MozReview-Commit-ID: 9HRASAnZkNn
ec99e3ba4e7a9ccb960937ac1eb39947a1060cc6: Bug 1355451 - Speculative connect to websites on mousedown on awesomebar item. r=mak
Evelyn Hung <jj.evelyn@gmail.com> - Fri, 04 Aug 2017 22:27:21 +0800 - rev 422463
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1355451 - Speculative connect to websites on mousedown on awesomebar item. r=mak We preconnect to an explicit URL or the url in moz-action:remotetab on the popup. MozReview-Commit-ID: Itgie3BMehn
989cea1f38209bc964d0dda2b9b9d0485cff6045: Bug 1386161 - [Mac] Remove IOAudioControl Rules. r=Alex_Gaynor
Haik Aftandilian <haftandilian@mozilla.com> - Sun, 30 Jul 2017 22:26:06 -0700 - rev 422462
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1386161 - [Mac] Remove IOAudioControl Rules. r=Alex_Gaynor MozReview-Commit-ID: 3cLUCJDoWlh
4cfb674227051e22bab651e5759f3de503a50560: Bug 1364821 - Fix merge bustage. a=RyanVM
Ryan VanderMeulen <ryanvm@gmail.com> - Fri, 04 Aug 2017 15:33:53 -0400 - rev 422461
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1364821 - Fix merge bustage. a=RyanVM A line was accidentally dropped when resolving merge conflicts between inbound and autoland. CI wasn't amused. This just adds it back.
bc829569880635c52d6e3d54f51cd7d3df180186: merge mozilla-inbound to mozilla-central. r=merge a=merge
Sebastian Hengst <archaeopteryx@coole-files.de> - Fri, 04 Aug 2017 19:28:27 +0200 - rev 422460
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
merge mozilla-inbound to mozilla-central. r=merge a=merge MozReview-Commit-ID: JhesEg4Bxjy
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip