eb4b44b2eb026ce5a4c126a8f6a44af95ca3c7ed: Bug 1311347 - Enable eslint of browser/components/sessionstore/. Initial changes by Sourav, updated by Standard8. r=jaws
jordan9769 <souravgarg833@gmail.com> - Fri, 28 Oct 2016 02:48:50 +0530 - rev 347517
Push 31499 by kwierso@gmail.com at Tue, 14 Mar 2017 23:41:39 +0000
Bug 1311347 - Enable eslint of browser/components/sessionstore/. Initial changes by Sourav, updated by Standard8. r=jaws MozReview-Commit-ID: 4RFxoV8SkIa
dc5d8fd7a1e0696e5a6af843260f6d67e6e9aa02: Bug 1343330 - Use fast-forward and fast-rewind characters instead of skip-to-end and skip-to-start on Windows 7. r=Felipe
Dão Gottwald <dao@mozilla.com> - Tue, 14 Mar 2017 13:03:42 +0100 - rev 347516
Push 31499 by kwierso@gmail.com at Tue, 14 Mar 2017 23:41:39 +0000
Bug 1343330 - Use fast-forward and fast-rewind characters instead of skip-to-end and skip-to-start on Windows 7. r=Felipe MozReview-Commit-ID: LUfyeUcDkmC
bb33447e31878c5d623c7987b8a041ebc1137f79: servo: Merge #15847 - Remove link to servo from treeherder job detail to reduce noise (from shinglyu:treeherder-cleanup); r=Wafflespeanut
Shing Lyu <slyu@mozilla.com> - Tue, 14 Mar 2017 04:08:49 -0700 - rev 347515
Push 31499 by kwierso@gmail.com at Tue, 14 Mar 2017 23:41:39 +0000
servo: Merge #15847 - Remove link to servo from treeherder job detail to reduce noise (from shinglyu:treeherder-cleanup); r=Wafflespeanut <!-- Please describe your changes on the following line: --> (This PR was original created by wlach in the deprecated servo-perf repo: https://github.com/shinglyu/servo-perf/pull/27) Quoting his original message: > The servo repository is already linked to from the revision list, there is no need for each job to link to it as well. >This came up in: https://bugzilla.mozilla.org/show_bug.cgi?id=1342296 --- <!-- 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: --> - [ ] There are tests for these changes OR - [x] These changes do not require tests because need full treeherder setup to test <!-- 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: d3fa0a3dd61273b8216a1c6f1312ba418dc52c85
2baef2ffbaedb7354286726660ebd36e84b432f0: No bug, Automated HPKP preload list update from host bld-linux64-spot-032 - a=hpkp-update
ffxbld - Tue, 14 Mar 2017 08:14:06 -0700 - rev 347514
Push 31498 by ffxbld at Tue, 14 Mar 2017 15:14:11 +0000
No bug, Automated HPKP preload list update from host bld-linux64-spot-032 - a=hpkp-update
d3e4852a0494693ab6c48d283db7e93db173525e: No bug, Automated HSTS preload list update from host bld-linux64-spot-032 - a=hsts-update
ffxbld - Tue, 14 Mar 2017 08:14:03 -0700 - rev 347513
Push 31498 by ffxbld at Tue, 14 Mar 2017 15:14:11 +0000
No bug, Automated HSTS preload list update from host bld-linux64-spot-032 - a=hsts-update
08f709c14bf70434c153defd6049b29078acba9f: merge mozilla-inbound to mozilla-central a=merge
Carsten "Tomcat" Book <cbook@mozilla.com> - Tue, 14 Mar 2017 14:23:03 +0100 - rev 347512
Push 31497 by cbook@mozilla.com at Tue, 14 Mar 2017 13:23:16 +0000
merge mozilla-inbound to mozilla-central a=merge
ab115811036da8a7c5103acc3f5ed4bd856933a5: Bug 1301649: Use MOZ_ASSERT_IF instead. r=smaug
Yoshi Huang <allstars.chh@mozilla.com> - Thu, 02 Mar 2017 11:01:33 +0800 - rev 347511
Push 31497 by cbook@mozilla.com at Tue, 14 Mar 2017 13:23:16 +0000
Bug 1301649: Use MOZ_ASSERT_IF instead. r=smaug
ca31554d93a23aa505034a8a88608cf3014b9f4b: Bug 1300671 - set firstPartyDomain on about: pages. r=smaug
Yoshi Huang <allstars.chh@mozilla.com> - Wed, 01 Mar 2017 18:59:57 +0800 - rev 347510
Push 31497 by cbook@mozilla.com at Tue, 14 Mar 2017 13:23:16 +0000
Bug 1300671 - set firstPartyDomain on about: pages. r=smaug When we load about:blank in a remote tab, it will have LOAD_FLAGS_DISALLOW_INHERIT_PRINCIPAL flag set, which will make NullPrinicipal as its document principal. So we add NULL_PRINCIPAL_FIRST_PARTY_DOMAIN as its firstPartyDomain. So when we load data:, or javascript: URI in a remote tab, it will inherit the principal from about:blank, hence also inherit the origin attributes. There are also some about: pages will use codebase principal, so we also set ABOUT_URI_FIRST_PARTY_DOMAIN as firstPartyDomain on their principals.
1622f3e9dd547aee7c34df51206a2b8c4e7e2462: Bug 1339707 - Replace NS_DispatchToCurrentThread to DocGroup / TabGroup dispatch method and set nsITimer target. r=smaug
Samael Wang <freesamael@gmail.com> - Tue, 07 Mar 2017 16:58:59 +0800 - rev 347509
Push 31497 by cbook@mozilla.com at Tue, 14 Mar 2017 13:23:16 +0000
Bug 1339707 - Replace NS_DispatchToCurrentThread to DocGroup / TabGroup dispatch method and set nsITimer target. r=smaug
32e9e0aa9a5bf1b1f3442da50f772037834cd575: Bug 1338476 - Add "pdfium.enabled" as a pref for turning on/off mortar pdf. r=evelyn, r=Gijs
Ya-Chieh Wu <ywu@mozilla.com> - Mon, 13 Mar 2017 19:57:00 +0100 - rev 347508
Push 31497 by cbook@mozilla.com at Tue, 14 Mar 2017 13:23:16 +0000
Bug 1338476 - Add "pdfium.enabled" as a pref for turning on/off mortar pdf. r=evelyn, r=Gijs
1853515da1f517b902667ddfa43c0a036ab6ad3b: Bug 1315037 - Use NS_ENSURE_SUCCESS for nsresult diagnosis. r=erahm
Chih-Yi Leu <cleu@mozilla.com> - Mon, 13 Mar 2017 02:14:00 +0100 - rev 347507
Push 31497 by cbook@mozilla.com at Tue, 14 Mar 2017 13:23:16 +0000
Bug 1315037 - Use NS_ENSURE_SUCCESS for nsresult diagnosis. r=erahm
1697080a4726def0547486a6e7ef057cfccdc5a8: Bug 1310681 - fix the missed closing parenthesis. r=me
JerryShih <hshih@mozilla.com> - Mon, 13 Mar 2017 11:54:00 +0100 - rev 347506
Push 31497 by cbook@mozilla.com at Tue, 14 Mar 2017 13:23:16 +0000
Bug 1310681 - fix the missed closing parenthesis. r=me MozReview-Commit-ID: Crsog4XJhW0
778683798a9aad9b0801ebd2bf6d0eda4ce240f6: Bug 1346510. Don't allow the surface cache to unlock the animated frames of an animated image (when discarding of animated images is disabled). r=aosmond
Timothy Nikkel <tnikkel@gmail.com> - Tue, 14 Mar 2017 01:11:44 -0500 - rev 347505
Push 31497 by cbook@mozilla.com at Tue, 14 Mar 2017 13:23:16 +0000
Bug 1346510. Don't allow the surface cache to unlock the animated frames of an animated image (when discarding of animated images is disabled). r=aosmond The pref has never been enabled, so this is quite surprising! It is currently possible (and has been for quite a while) to discard animated images. All we need is the follow sequence of events. 1. Decode an animated image. 2. Move the animated image out of view (so it is not painted). 3. Call canvas.drawImage on the animated image (or anything else that asks for a first frame only decode). This creates a static entry in the surface cache for this first frame in addition to the animated entry. Because it is a static request we will also start a first frame decode. RasterImage::Decode calls SurfaceCache::UnlockEntries https://dxr.mozilla.org/mozilla-central/rev/4ceb9062ea8f4113bfd1b3536ace4a840a72faa7/image/RasterImage.cpp#1166 and bam, the animated frames are now unlocked (even though the RasterImage, and it's entry in the surface cache is still locked). 4. Switch tabs, open about:memory and minimize memory to actual throw away the animated frames. 5. Switch back to the image tab, scroll the image back into view, it will not animate, it will just show the last composited frame forever.
62dd2dbd728be2f3b4c0932aafce8a810d3a284f: Backed out changeset 985d9b83325f (bug 1337043) for causing bug 1337043
Carsten "Tomcat" Book <cbook@mozilla.com> - Tue, 14 Mar 2017 07:02:02 +0100 - rev 347504
Push 31497 by cbook@mozilla.com at Tue, 14 Mar 2017 13:23:16 +0000
Backed out changeset 985d9b83325f (bug 1337043) for causing bug 1337043
e6113871af3df68c43bcdaa0afaa759b7abc7c72: Bug 1305754 - Don't rebuild the whole path in the rule tree when resolving style with replacement. r=heycam
L. David Baron <dbaron@dbaron.org> - Mon, 13 Mar 2017 15:31:32 -0700 - rev 347503
Push 31497 by cbook@mozilla.com at Tue, 14 Mar 2017 13:23:16 +0000
Bug 1305754 - Don't rebuild the whole path in the rule tree when resolving style with replacement. r=heycam MozReview-Commit-ID: HdNhDBnOS3u
e4725643ead636a7cb9b8f2103f68a31fe278dbe: Bug 1346356 (part 4) - Clean up notification in the profiler. r=mstange.
Nicholas Nethercote <nnethercote@mozilla.com> - Tue, 14 Mar 2017 10:03:33 +1100 - rev 347502
Push 31497 by cbook@mozilla.com at Tue, 14 Mar 2017 13:23:16 +0000
Bug 1346356 (part 4) - Clean up notification in the profiler. r=mstange. This patch does the following. - Introduces NotifyObservers() for the simple notification cases in platform.cpp. - Removes profiler_lock() and profiler_unlock() because they do notifications that the profiler add-on no longer listens for.
cbfe870385c09f0a63c46b586eeadb113cbde08a: Bug 1346356 (part 3) - Do most NotifyObserver() calls when gPSMutex is unlocked. r=mstange.
Nicholas Nethercote <nnethercote@mozilla.com> - Tue, 14 Mar 2017 10:03:33 +1100 - rev 347501
Push 31497 by cbook@mozilla.com at Tue, 14 Mar 2017 13:23:16 +0000
Bug 1346356 (part 3) - Do most NotifyObserver() calls when gPSMutex is unlocked. r=mstange. Calling NotifyObserver() with gPSMutex locked is a bad idea; arbitrary code can run in observers, which can easily include code that calls other profiler functions that themselves lock gPSMutex, causing deadlock. This has been seen in practise with locked_profiler_stop(). This patch moves all but one of the NotifyObserver() calls in platform.cpp to after the sections where gPSMutex is locked. The remaining call (for the "profiler-subprocess") is harmless, because it just calls a simple callback implemented within platform.cpp, and hard to move. In the future we plan to allow profiler_start() and profiler_stop() to be called from different threads. When that happens, it will be possible for the "profiler-start" and "profiler-stop" notifications to arrive out of order. Unfortunately this seems difficult to avoid. (Well, recursive mutexes would make this problem much easier, but we don't have those...)
8b0bfd19d13b861440f99b973249ac70f53cd457: Bug 1346356 (part 2) - Merge SamplerThread::Join() into ~SamplerThread(). r=mstange.
Nicholas Nethercote <nnethercote@mozilla.com> - Tue, 14 Mar 2017 10:03:33 +1100 - rev 347500
Push 31497 by cbook@mozilla.com at Tue, 14 Mar 2017 13:23:16 +0000
Bug 1346356 (part 2) - Merge SamplerThread::Join() into ~SamplerThread(). r=mstange. SamplerThread::Join() is always called just before the SamplerThread object is deleted. Might as well combine them.
3ec4e9dbdcb3f2a47e0618b6c97a3f753f92f1c9: Bug 1346356 (part 1) - Fix SamplerThread leak in profiler_start(). r=mstange.
Nicholas Nethercote <nnethercote@mozilla.com> - Tue, 14 Mar 2017 10:03:33 +1100 - rev 347499
Push 31497 by cbook@mozilla.com at Tue, 14 Mar 2017 13:23:16 +0000
Bug 1346356 (part 1) - Fix SamplerThread leak in profiler_start(). r=mstange. profiler_start() can call locked_profiler_stop(). In that case it must then call SamplerThread::Join() in order to free the SamplerThread, but it currently doesn't. This patch adds such a call. It also marks locked_profiler_start() with MOZ_MUST_USE to make this mistake less likely in the future.
0819cd293127732aebd3341eec2d7e5b136f208b: Bug 1346692 - Clarify reftest documentation about combining <failure-type>s. No review.
L. David Baron <dbaron@dbaron.org> - Mon, 13 Mar 2017 19:08:28 -0700 - rev 347498
Push 31497 by cbook@mozilla.com at Tue, 14 Mar 2017 13:23:16 +0000
Bug 1346692 - Clarify reftest documentation about combining <failure-type>s. No review. This makes two changes: * adds "on the line" to clarify what last means * adds "when combining <failure-type> from the manifest include and the test line" to clarify that the parenthetical only applies to combining at different levels, and not within a line DONTBUILD
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip