8b602649e6737c56519eb2753122696020b8096b: Bug 1365786 - Avoid missing --help dep error when a @depends function uses builtins. r=chmanchester
Mike Hommey <mh+mozilla@glandium.org> - Thu, 18 May 2017 09:32:39 +0900 - rev 359098
Push 31849 by ryanvm@gmail.com at Fri, 19 May 2017 15:38:10 +0000
Bug 1365786 - Avoid missing --help dep error when a @depends function uses builtins. r=chmanchester
d2e6bfe37885afbbc7d9d85b3d839d73fb5120b3: Bug 1365477 - Ensure all callers of CommandLineHandler() pass a complete argv. r=chmanchester
Mike Hommey <mh+mozilla@glandium.org> - Wed, 17 May 2017 12:10:46 +0900 - rev 359097
Push 31849 by ryanvm@gmail.com at Fri, 19 May 2017 15:38:10 +0000
Bug 1365477 - Ensure all callers of CommandLineHandler() pass a complete argv. r=chmanchester CommandLineHandler() expects argv to be like sys.argv, containing the command name in argv[0], but various tests weren't doing that, in some cases even leading to ignored arguments passed as argv[0]. In turn, that made lint.py only test browser/moz.configure instead of all the project moz.configures as intended.
813c34daa4f9a3235b205613c348414e51b3baeb: Bug 1365477 - Use default=is_nightly instead of default=delayed_getattr(...). r=chmanchester
Mike Hommey <mh+mozilla@glandium.org> - Wed, 17 May 2017 12:08:33 +0900 - rev 359096
Push 31849 by ryanvm@gmail.com at Fri, 19 May 2017 15:38:10 +0000
Bug 1365477 - Use default=is_nightly instead of default=delayed_getattr(...). r=chmanchester Per bug 1363811 comment 1, it's actually a bug that the linter doesn't complain about it like it did in bug 1358215.
ae9d1af14c2481fc695461b4cd3fd488006acc84: Backed out changeset 515c3f15cf30 (bug 1364594) for failing test_window_maximize.py TestWindowMaximize.test_maximize on Linux. r=backout
Sebastian Hengst <archaeopteryx@coole-files.de> - Thu, 18 May 2017 23:38:30 +0200 - rev 359095
Push 31849 by ryanvm@gmail.com at Fri, 19 May 2017 15:38:10 +0000
Backed out changeset 515c3f15cf30 (bug 1364594) for failing test_window_maximize.py TestWindowMaximize.test_maximize on Linux. r=backout
74de5283b24d6626067de8d8f043f47b74a4cf6a: Bug 1363481 - Add the old attribute value as a parameter to Element::AfterSetAttr r=bz
Kirk Steuber <ksteuber@mozilla.com> - Thu, 18 May 2017 14:09:01 -0700 - rev 359094
Push 31849 by ryanvm@gmail.com at Fri, 19 May 2017 15:38:10 +0000
Bug 1363481 - Add the old attribute value as a parameter to Element::AfterSetAttr r=bz In order to facilitate the movement of code with side-effects called by Element::SetAttr to Element::BeforeSetAttr and Element::AfterSetAttr, Element::AfterSetAttr should have access to the old value of the attribute. This includes information about whether there was previously a value set or not. Accomplishing this involved passing an additional argument through functions that find and change the old attribute value in order to ensure that we can differentiate between an empty old value and an absent old value (attribute was not set). Note that while I tried to ensure that accurate values (and their absence) are reported to Element::AfterSetAttr, I largely ignored SVG. While the old value reported for SVG values should be however accurate the value already being reported to SetAttrAndNotify was, SVG elements do not currently report unset values properly because they will never pass a null pointer to SetAttrAndNotify. MozReview-Commit-ID: K1mha8CNFZP
a71df2a514f469810aefc94bc0b68218d20b41e0: servo: Merge #16937 - Fix calc() clamping issues (from nox:clamp-calc); r=emilio
Anthony Ramine <n.oxyde@gmail.com> - Thu, 18 May 2017 15:15:04 -0500 - rev 359093
Push 31849 by ryanvm@gmail.com at Fri, 19 May 2017 15:38:10 +0000
servo: Merge #16937 - Fix calc() clamping issues (from nox:clamp-calc); r=emilio Fix calc() clamping issues Source-Repo: https://github.com/servo/servo Source-Revision: b4204b5e2d8b2981408e2c092130af40ad3170bb
42ab019426c473de3e9c10865ecdc0e8f4112b32: servo: Merge #16938 - stylo: Don't mess up traversal drivers (from emilio:traversal-assert); r=bzbarsky
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 18 May 2017 13:38:41 -0500 - rev 359092
Push 31849 by ryanvm@gmail.com at Fri, 19 May 2017 15:38:10 +0000
servo: Merge #16938 - stylo: Don't mess up traversal drivers (from emilio:traversal-assert); r=bzbarsky Right now it's harmless, but it trips assertions that are nice to have, just in case it stops being the case. Source-Repo: https://github.com/servo/servo Source-Revision: e1ecd3b89cd9c78b6ce2e936064681df52d265ab
569c0bf0ddcb0adc0113ee45a215bde1272776ff: Bug 1351440 - Part 5. typedef DrawResult in nsDisplayItem to avoid the image:: qualification. r=jwatt
cku <cku@mozilla.com> - Thu, 18 May 2017 22:03:54 +0200 - rev 359091
Push 31849 by ryanvm@gmail.com at Fri, 19 May 2017 15:38:10 +0000
Bug 1351440 - Part 5. typedef DrawResult in nsDisplayItem to avoid the image:: qualification. r=jwatt MozReview-Commit-ID: 1l8DUL1z6U2
7995a23f4acef1a41cad4256aa021f04f41e7723: Bug 1351440 - Part 4. Pass imgDrawingParams to nsSVGPaintServerFrame::GetPaintServerPattern. r=jwatt
cku <cku@mozilla.com> - Thu, 18 May 2017 22:03:50 +0200 - rev 359090
Push 31849 by ryanvm@gmail.com at Fri, 19 May 2017 15:38:10 +0000
Bug 1351440 - Part 4. Pass imgDrawingParams to nsSVGPaintServerFrame::GetPaintServerPattern. r=jwatt The reason of this change is the same as for Part 2, except that this commit fixes nsSVGPaintServerFrame::GetPaintServerPattern rather than PaintSVG. Commit-ID: 691YrKZ0Lm9 MozReview-Commit-ID: KSnFhCndFUk
4c05d41afa8669b98aed1757dbcd0398834454a4: Bug 1351440 - Part 3. Pass imgDrawingParams to nsSVGMaskFrame::GetMaskForMaskedFrame. r=jwatt
cku <cku@mozilla.com> - Thu, 18 May 2017 22:03:45 +0200 - rev 359089
Push 31849 by ryanvm@gmail.com at Fri, 19 May 2017 15:38:10 +0000
Bug 1351440 - Part 3. Pass imgDrawingParams to nsSVGMaskFrame::GetMaskForMaskedFrame. r=jwatt The reason of this change is the same as for Part 2, except this commit fixes nsSVGMaskFrame::GetMaskForMaskedFrame rather than PaintSVG. MozReview-Commit-ID: DS0eG6eKDgs
bbf16eb4f821b2d535826141f9f33f3eb49a54c6: Bug 1351440 - Part 2. Encapsulate DrawResult and imgIContainer::FLAG_* into imgDrawingParams, and pass it to PaintSVG. r=jwatt
cku <cku@mozilla.com> - Thu, 18 May 2017 22:03:41 +0200 - rev 359088
Push 31849 by ryanvm@gmail.com at Fri, 19 May 2017 15:38:10 +0000
Bug 1351440 - Part 2. Encapsulate DrawResult and imgIContainer::FLAG_* into imgDrawingParams, and pass it to PaintSVG. r=jwatt The DrawResult return was not in fact anything to do with the success or failure of that method, but was actually passing out a very specific piece of information about the success or failure of any imagelib drawing that may not have occurred under the various PaintSVG calls. The signature of PaintSVG is changed from DrawResult PaintSVG(...., uint32 flags); to void PaintSVG(...., imgDrawingParams& aPackage); imgDrawingParams wraps DrawResult and imgIContainer::FLAG_* as a pack, pass through PaintSVG to imagelib draw calls under beneath. MozReview-Commit-ID: IOq2evUAOQF
edc2283b163789c76c8d44a4ab87b72955cd40ed: Bug 1351440 - Part 1. Implement imgDrawingParams. r=jwatt
cku <cku@mozilla.com> - Thu, 18 May 2017 22:03:36 +0200 - rev 359087
Push 31849 by ryanvm@gmail.com at Fri, 19 May 2017 15:38:10 +0000
Bug 1351440 - Part 1. Implement imgDrawingParams. r=jwatt A struct used during painting to provide input flags to determine how imagelib draw calls should behave, plus an output DrawResult to return information about the result of any imagelib draw calls that may have occurred. MozReview-Commit-ID: 3jGEh5vEPF
94f5cca9efbafee99ca4fb832b4aef6144ac469d: Bug 1365417 - mozJSComponentLoader::FindTargetObject() is infallible. r=kmag
Andrew McCreight <continuation@gmail.com> - Tue, 16 May 2017 14:27:50 -0700 - rev 359086
Push 31849 by ryanvm@gmail.com at Fri, 19 May 2017 15:38:10 +0000
Bug 1365417 - mozJSComponentLoader::FindTargetObject() is infallible. r=kmag MozReview-Commit-ID: Iw4ke16CTKg
bcb8a85c4663d16097d2dee9bfb4aa5d0259b38f: Bug 1365760 - Fix two tiny issues in mozJSComponentLoader::ImportInto(). r=kmag
Andrew McCreight <continuation@gmail.com> - Tue, 16 May 2017 15:51:14 -0700 - rev 359085
Push 31849 by ryanvm@gmail.com at Fri, 19 May 2017 15:38:10 +0000
Bug 1365760 - Fix two tiny issues in mozJSComponentLoader::ImportInto(). r=kmag MozReview-Commit-ID: 84I37uRi11H
515c3f15cf3069c7e14bc98773d6f0185281123d: Bug 1364594: Exit fullscreen when Set Window Rect is invoked. r=ato
David Burns <dburns@mozilla.com> - Thu, 18 May 2017 09:23:08 +0100 - rev 359084
Push 31849 by ryanvm@gmail.com at Fri, 19 May 2017 15:38:10 +0000
Bug 1364594: Exit fullscreen when Set Window Rect is invoked. r=ato If the browser is in fullscreen mode and Set Window Rect is called we need to exit fullscreen mode and then continue to manipulate the browser. As described in https://w3c.github.io/webdriver/webdriver-spec.html#set-window-rect Step 10 MozReview-Commit-ID: 5ixhGOXVBE4
ddb3969de046e2b08cc93ec963fbf61e6622c639: Bug 1355252 - Reject navigator.requestMediaKeySystemAccess if no MediaCapabilities supplied. r=jwwang
Chris Pearce <cpearce@mozilla.com> - Wed, 17 May 2017 17:10:14 +1200 - rev 359083
Push 31849 by ryanvm@gmail.com at Fri, 19 May 2017 15:38:10 +0000
Bug 1355252 - Reject navigator.requestMediaKeySystemAccess if no MediaCapabilities supplied. r=jwwang As required by step 15 of EME's Get Supported Configuration and Consent algorithm, https://www.w3.org/TR/encrypted-media/#get-supported-configuration-and-consent MozReview-Commit-ID: 8c24VrmygBx
c19d32e2a7103729ae40c874c93c5cb991369812: Bug 1355252 - Don't assume default codecs for MP4 and WebM in EME MediaCapabilities r=jwwang
Chris Pearce <cpearce@mozilla.com> - Thu, 11 May 2017 14:23:33 +1200 - rev 359082
Push 31849 by ryanvm@gmail.com at Fri, 19 May 2017 15:38:10 +0000
Bug 1355252 - Don't assume default codecs for MP4 and WebM in EME MediaCapabilities r=jwwang Step 10 of EME's "Get Supported Capabilities for Audio/Video Type" algorithm says we can assume default codecs only if a container normatively implies a specific set of codec and codec constraints. Our code assumes that WebM implies Vorbis/VP8 and MP4 implies AAC/H.264, but those aren't actually normatively required by either of these containers' specifications. So we shouldn't assume these containers imply those codecs. MozReview-Commit-ID: G9TDOmrjhpp
442e51c751719d855fedadde081ee6658d61a7f8: Bug 1365538 - Postpone FileBlockCache writes while reading - r=cpearce
Gerald Squelart <gsquelart@mozilla.com> - Wed, 17 May 2017 17:10:57 +1200 - rev 359081
Push 31849 by ryanvm@gmail.com at Fri, 19 May 2017 15:38:10 +0000
Bug 1365538 - Postpone FileBlockCache writes while reading - r=cpearce While inside FileBlock::Read, the mIsReading flag is set, which in turn prevents starting the writer task, or makes it exit as soon as possible. This should help multi-block reads (while data is being received) by giving them priority, so that that blocking call can return faster. At the end of Read, the writer task is re-dispatched if needed. MozReview-Commit-ID: J32tHGFRMNU
461b3e0c577f2b4fbef90ebc1dd94fd876aaef50: Bug 1365538 - Use Mutex instead of Monitor in FileBlockCache - r=cpearce
Gerald Squelart <gsquelart@mozilla.com> - Wed, 17 May 2017 16:36:26 +1200 - rev 359080
Push 31849 by ryanvm@gmail.com at Fri, 19 May 2017 15:38:10 +0000
Bug 1365538 - Use Mutex instead of Monitor in FileBlockCache - r=cpearce Since both FileBlockCache Monitor's are only used as locks, we should just use simple Mutex'es (smaller, faster, and tighter API). MozReview-Commit-ID: DdNNPN1kFrs
0724e572eb406dbe14bb414e287cc29e3f603cce: Backed out changeset 0065478cc7eb (bug 1351440) under suspicion for causing reftest failures on Stylo debug. r=backout
Sebastian Hengst <archaeopteryx@coole-files.de> - Thu, 18 May 2017 21:16:31 +0200 - rev 359079
Push 31849 by ryanvm@gmail.com at Fri, 19 May 2017 15:38:10 +0000
Backed out changeset 0065478cc7eb (bug 1351440) under suspicion for causing reftest failures on Stylo debug. r=backout
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip