2a91df8739c98e2106dd570759e099a503535302: Bug 1468953: Use node bounds in viewport instead of offsets from offsetParent. r=jdescottes
Daisuke Akatsuka <dakatsuka@mozilla.com> - Tue, 26 Jun 2018 10:51:40 +0900 - rev 478693
Push 9719 by ffxbld-merge at Fri, 24 Aug 2018 17:49:46 +0000
Bug 1468953: Use node bounds in viewport instead of offsets from offsetParent. r=jdescottes MozReview-Commit-ID: nqMLetevCZ
4cee97f375da6da94f2bb45d1930aefe1191d8c4: Bug 1470346 - Gesture activate all documents in tab, even across origins, upon user interaction. r=smaug
Chris Pearce <cpearce@mozilla.com> - Fri, 22 Jun 2018 11:52:20 +1200 - rev 478692
Push 9719 by ffxbld-merge at Fri, 24 Aug 2018 17:49:46 +0000
Bug 1470346 - Gesture activate all documents in tab, even across origins, upon user interaction. r=smaug Sometimes when video is playing, a preroll ad plays, and that may be in a cross origin iframe. If autoplay media is disabled, we require a user gesture in a document before playback in that document is permitted, and we require each origin to be gesture activated separately. So in the cross origin preroll video add case, then the user will have to click once to unblock playback for the cross origin ad, and then once the preroll ad finishes, the user will have to click again to activate playback of the same origin content video. This is a bad user experience. So we should instead make gesture activation propagate up the doc tree irrespective of crossing origins. This way, when the user clicks to activate, all documents in that tab are also also effectively gesture activated, and so can autoplay. MozReview-Commit-ID: 1HZQ5zkubR
0292160da4fdfaa8df3eda505921e194214871b4: Backed out changeset b5e89c1b4e8c (bug 1470073) for failing browser chrome tests on browser_test_io_activity.js on a CLOSED TREE
arthur.iakab <aiakab@mozilla.com> - Tue, 26 Jun 2018 04:10:45 +0300 - rev 478691
Push 9719 by ffxbld-merge at Fri, 24 Aug 2018 17:49:46 +0000
Backed out changeset b5e89c1b4e8c (bug 1470073) for failing browser chrome tests on browser_test_io_activity.js on a CLOSED TREE
dafeb03fc8c7ddc104eabdb2255ec689066a1ef3: Backed out 5 changesets (bug 1340498) for build bustages on SelectionChangeListener.h on a CLOSED TREE
arthur.iakab <aiakab@mozilla.com> - Tue, 26 Jun 2018 03:53:21 +0300 - rev 478690
Push 9719 by ffxbld-merge at Fri, 24 Aug 2018 17:49:46 +0000
Backed out 5 changesets (bug 1340498) for build bustages on SelectionChangeListener.h on a CLOSED TREE Backed out changeset fae677707059 (bug 1340498) Backed out changeset 2adde1d1742a (bug 1340498) Backed out changeset a444ab9cefde (bug 1340498) Backed out changeset e2ac49ef2034 (bug 1340498) Backed out changeset 6465310b3de9 (bug 1340498)
d50204fd6e1f343bd945d236f039e0d18061c85f: Bug 1451601 Do not buffer output in TaskCluster job output r=gps
Tom Ritter <tom@mozilla.com> - Mon, 25 Jun 2018 15:28:06 -0500 - rev 478689
Push 9719 by ffxbld-merge at Fri, 24 Aug 2018 17:49:46 +0000
Bug 1451601 Do not buffer output in TaskCluster job output r=gps MozReview-Commit-ID: 4XOcqSXGhHi
fdd8115f07f5b327183a9d78c7a69d59dfa10dec: Bug 1466574 - make generate_builtin_addons handle races better, r=ted
Dan Mosedale <dmose@meer.net> - Thu, 31 May 2018 09:56:20 -0700 - rev 478688
Push 9719 by ffxbld-merge at Fri, 24 Aug 2018 17:49:46 +0000
Bug 1466574 - make generate_builtin_addons handle races better, r=ted MozReview-Commit-ID: D9irkni3gQv
fae67770705996f4871eabaf2f56f6a9884407de: Bug 1340498 - Fix unified sources build errors r=mrbkap
Doug Thayer <dothayer@mozilla.com> - Fri, 13 Apr 2018 11:04:47 -0700 - rev 478687
Push 9719 by ffxbld-merge at Fri, 24 Aug 2018 17:49:46 +0000
Bug 1340498 - Fix unified sources build errors r=mrbkap Adding the Places* files into unified sources pushed the unified sources into a situation that exposed a strangely large number of errors. This seems to be the minimum set of changes I could make to resolve all of the issues. MozReview-Commit-ID: C2H9ce8FmE4
2adde1d1742a2224147afb2ee1e6d008d8d12857: Bug 1340498 - Add new globals to lint config r=standard8
Doug Thayer <dothayer@mozilla.com> - Mon, 29 Jan 2018 11:07:28 -0800 - rev 478686
Push 9719 by ffxbld-merge at Fri, 24 Aug 2018 17:49:46 +0000
Bug 1340498 - Add new globals to lint config r=standard8 MozReview-Commit-ID: 360gup8cWvi
a444ab9cefdeb8a7e442a78ce227aea9dc8a99d9: Bug 1340498 - Update onVisits tests to use 'page-visited' r=mak
Doug Thayer <dothayer@mozilla.com> - Wed, 14 Feb 2018 09:17:41 -0800 - rev 478685
Push 9719 by ffxbld-merge at Fri, 24 Aug 2018 17:49:46 +0000
Bug 1340498 - Update onVisits tests to use 'page-visited' r=mak MozReview-Commit-ID: FxC3gcUF9hl
e2ac49ef203437f4a489ef5ea5ee328a874d04ef: Bug 1340498 - Update onVisits uses to 'page-visited' r=mak
Doug Thayer <dothayer@mozilla.com> - Wed, 14 Feb 2018 09:11:49 -0800 - rev 478684
Push 9719 by ffxbld-merge at Fri, 24 Aug 2018 17:49:46 +0000
Bug 1340498 - Update onVisits uses to 'page-visited' r=mak Consuming the new 'page-visited' notification was fairly trivial, since it was already brought over to onVisits. There's not much to say about this other than that I'm a little bit uncertain about all the hoops we have to jump through to get a JSContext and GlobalObject from History.cpp (which is discussed in the earlier commit in the series). MozReview-Commit-ID: LHaBWSylyLI
6465310b3de950d0989b6e056f966fb2c7a1a337: Bug 1340498 - Implement new Places Observers interface r=mrbkap
Doug Thayer <dothayer@mozilla.com> - Wed, 14 Feb 2018 09:06:15 -0800 - rev 478683
Push 9719 by ffxbld-merge at Fri, 24 Aug 2018 17:49:46 +0000
Bug 1340498 - Implement new Places Observers interface r=mrbkap See the design doc[1] for further info. We would like to redesign the places observer system to be more performant and more friendly to consume. WebIDL was recommended as it simplifies creating simple dictionary payloads while allowing dynamic typing with `any`. There were some difficulties with WebIDL though, most of which revolved around allowing consumers to be weakly referenced, from both C++ and JS. The simplest solution I could come up with was to create a simple native interface for the C++ case, and a WebIDL wrapper for a JS callback in the JS case. Suggestions for simpler alternatives are very welcome though. [1] https://docs.google.com/document/d/1G45vfd6RXFXwNz7i4FV40lDCU0ao-JX_bZdgJV4tLjk/edit?usp=sharing MozReview-Commit-ID: ACnAEfa5WxO
31ac168c31d6adce9a4f3d0fde8c21979dd0ab96: Backed out changeset 48f880cffaed (bug 1448096) for devtools mass failure on a CLOSED TREE
arthur.iakab <aiakab@mozilla.com> - Tue, 26 Jun 2018 03:10:03 +0300 - rev 478682
Push 9719 by ffxbld-merge at Fri, 24 Aug 2018 17:49:46 +0000
Backed out changeset 48f880cffaed (bug 1448096) for devtools mass failure on a CLOSED TREE
7e18d20c263f5ac0f99c6aa80bfb2e69d310805a: Bug 1438499 - show 'close multiple tabs' warning dialog when quitting, r=jaws,whimboo
Gijs Kruitbosch <gijskruitbosch@gmail.com> - Wed, 13 Jun 2018 09:34:52 -0700 - rev 478681
Push 9719 by ffxbld-merge at Fri, 24 Aug 2018 17:49:46 +0000
Bug 1438499 - show 'close multiple tabs' warning dialog when quitting, r=jaws,whimboo MozReview-Commit-ID: J2gi9do8HK9
48f880cffaed8d9432df7c48e599838228e515f2: Bug 1448096 - use promiseDocumentFlushed to avoid sync reflows when resizing devtools windows, r=bgrins,mconley
Gijs Kruitbosch <gijskruitbosch@gmail.com> - Tue, 19 Jun 2018 18:12:42 +0100 - rev 478680
Push 9719 by ffxbld-merge at Fri, 24 Aug 2018 17:49:46 +0000
Bug 1448096 - use promiseDocumentFlushed to avoid sync reflows when resizing devtools windows, r=bgrins,mconley MozReview-Commit-ID: HATxzjdbQDj
0a78d0ffe366ec03ec16dea37f16f0bc3c590a41: Bug 1470002 - remove pageshow message passing, too, r=adw,jryans
Gijs Kruitbosch <gijskruitbosch@gmail.com> - Wed, 20 Jun 2018 22:53:40 +0100 - rev 478679
Push 9719 by ffxbld-merge at Fri, 24 Aug 2018 17:49:46 +0000
Bug 1470002 - remove pageshow message passing, too, r=adw,jryans MozReview-Commit-ID: 27v79lVOODG
6eda16d8031434d57ea0bdb869e35dd8e9ff0276: Bug 1470002 - remove pagehide message passing from browser code, r=mixedpuppy
Gijs Kruitbosch <gijskruitbosch@gmail.com> - Wed, 20 Jun 2018 22:41:27 +0100 - rev 478678
Push 9719 by ffxbld-merge at Fri, 24 Aug 2018 17:49:46 +0000
Bug 1470002 - remove pagehide message passing from browser code, r=mixedpuppy MozReview-Commit-ID: 18hsSwHsWzz
fec9f43850d3de3861ae37be4318179c17f820ec: Bug 1470100 - use a vector instead of an array for the list of prefs, r=ato
Gijs Kruitbosch <gijskruitbosch@gmail.com> - Thu, 21 Jun 2018 12:02:00 +0100 - rev 478677
Push 9719 by ffxbld-merge at Fri, 24 Aug 2018 17:49:46 +0000
Bug 1470100 - use a vector instead of an array for the list of prefs, r=ato MozReview-Commit-ID: 5JI85Rif28M
803d0242675c05b32609c19066a7731c0b7cf357: Bug 1469999 - Use yaml.safe_load() in telemetry test; r=chutten
Gregory Szorc <gps@mozilla.com> - Wed, 20 Jun 2018 14:34:15 -0700 - rev 478676
Push 9719 by ffxbld-merge at Fri, 24 Aug 2018 17:49:46 +0000
Bug 1469999 - Use yaml.safe_load() in telemetry test; r=chutten yaml.load() can lead to arbitrary code execution because it isn't secure by default and allows special YAML syntax. While it shouldn't be a problem here, I'm trying to get rid of all yaml.load() calls so we can add a lint to ban the practice. Differential Revision: https://phabricator.services.mozilla.com/D1740
a94c62b1e41a0602d78a043790bfe39c2be17abf: Bug 1469999 - Use yaml.safe_load() for loading shaders manifest; r=chmanchester
Gregory Szorc <gps@mozilla.com> - Wed, 20 Jun 2018 14:32:46 -0700 - rev 478675
Push 9719 by ffxbld-merge at Fri, 24 Aug 2018 17:49:46 +0000
Bug 1469999 - Use yaml.safe_load() for loading shaders manifest; r=chmanchester yaml.load() isn't safe and can lead to arbitrary code execution for untrusted input. While probably not an issue here, I'm trying to rid the tree of all yaml.load() instances so we can add a lint to ban its usage. Differential Revision: https://phabricator.services.mozilla.com/D1739
2452079eb42f9a3d2a3e624d7248fa024325742d: Bug 1469999 - Use yaml.safe_load() for loading linter config file; r=ahal
Gregory Szorc <gps@mozilla.com> - Wed, 20 Jun 2018 14:29:27 -0700 - rev 478674
Push 9719 by ffxbld-merge at Fri, 24 Aug 2018 17:49:46 +0000
Bug 1469999 - Use yaml.safe_load() for loading linter config file; r=ahal yaml.load() is unsafe and can lead to arbitrary code execution via syntax like `!!python/object/apply:os.system`. yaml.safe_load() is more reasonable. Differential Revision: https://phabricator.services.mozilla.com/D1738
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip