ae01415f56b22fc573dac4cef502d808afb3c62f: Bug 1257303 - Cast to intptr_t to avoid C4312 on VS2015; r=tbsaunde
Gregory Szorc <gps@mozilla.com> - Wed, 23 Mar 2016 15:46:40 -0700 - rev 290188
Push 18353 by cbook@mozilla.com at Thu, 24 Mar 2016 15:20:25 +0000
Bug 1257303 - Cast to intptr_t to avoid C4312 on VS2015; r=tbsaunde Without this, we get: C4312: 'reinterpret_cast': conversion from 'const LONG' to 'void *' of greater size MozReview-Commit-ID: 4WXmvVUd5c8
531508a2e75575e007d3eb578c77fd3ecf5fee1e: Bug 1011826 - Split Prefable into two pieces. r=bz.
Nicholas Nethercote <nnethercote@mozilla.com> - Thu, 24 Mar 2016 06:09:24 +1100 - rev 290187
Push 18353 by cbook@mozilla.com at Thu, 24 Mar 2016 15:20:25 +0000
Bug 1011826 - Split Prefable into two pieces. r=bz. The fields in Prefable relating to disabling are usually all zero. This patch moves them into a new struct PrefableDisablers. This reduces static data size by 92 KB, which applies to every process. It might also make isEnabled() faster because the common case only involves one test instead of two.
c75b2b195f28a5ed556a7dfc12e9b45bc56c971a: Bug 1257101. imgFrame::IsImageComplete says whether we've had pixels decoded to the whole image rect, but it's used to check if the frame is finished decoding. These are different things when the image has more than one progress pass. r=seth
Timothy Nikkel <tnikkel@gmail.com> - Wed, 23 Mar 2016 19:31:42 -0500 - rev 290186
Push 18353 by cbook@mozilla.com at Thu, 24 Mar 2016 15:20:25 +0000
Bug 1257101. imgFrame::IsImageComplete says whether we've had pixels decoded to the whole image rect, but it's used to check if the frame is finished decoding. These are different things when the image has more than one progress pass. r=seth This means that in RasterImage::LookupFrame when we are asked to do a sync decode (if needed) we use WaitUntilComplete to wait until the frame is finished decoding. But we would actually return after the next progressive pass notified the monitor to wake up. Thus, we would draw a not-fully-decoded image even though the sync decode flag was passed. The change in FrameAnimator means that we won't draw the next frame in an animated image until all progressive passes of that image are complete. This seems like what we want anyways. There is one real use of IsImageComplete left, in imgFrame::Draw, where we need to know if the decoded image data covers the whole image frame. (There are a couple of uses of IsImageComplete in asserts.)
3e142ba20ecad6ee3aab402740a7f8f903089a5b: Bug 1245260 - Add crashtest; r=hiro
Brian Birtles <birtles@gmail.com> - Thu, 24 Mar 2016 09:04:53 +0900 - rev 290185
Push 18353 by cbook@mozilla.com at Thu, 24 Mar 2016 15:20:25 +0000
Bug 1245260 - Add crashtest; r=hiro MozReview-Commit-ID: IBe0zbsCJJy
76d58b17343e47d057bbbc22634e0ff3e537b8b5: Bug 1257823 - Move add_old_configure_assignment() to the global scope. r=nalexander
Mike Hommey <mh+mozilla@glandium.org> - Wed, 23 Mar 2016 16:34:59 +0900 - rev 290184
Push 18353 by cbook@mozilla.com at Thu, 24 Mar 2016 15:20:25 +0000
Bug 1257823 - Move add_old_configure_assignment() to the global scope. r=nalexander Similar to set_config, set_define and imply_option, but this is a sandboxed function that stays sandboxed.
06dc23858ed715e62229c2f80147af28b0416fff: Bug 1257823 - Move imply_option() to the global scope. r=nalexander
Mike Hommey <mh+mozilla@glandium.org> - Wed, 23 Mar 2016 14:18:57 +0900 - rev 290183
Push 18353 by cbook@mozilla.com at Thu, 24 Mar 2016 15:20:25 +0000
Bug 1257823 - Move imply_option() to the global scope. r=nalexander Like set_config and set_define, we move imply_option to the global scope.
407e18a1a0241fb62f0391d698ff9954625ad06b: Bug 1257823 - Keep track of the dependencies of @depends functions. r=nalexander
Mike Hommey <mh+mozilla@glandium.org> - Wed, 23 Mar 2016 11:58:57 +0900 - rev 290182
Push 18353 by cbook@mozilla.com at Thu, 24 Mar 2016 15:20:25 +0000
Bug 1257823 - Keep track of the dependencies of @depends functions. r=nalexander @depends functions are declared like the following: @depends('--option', other_function, '--other-option', ...) def ... To simplify some of the processing related to those arguments it's passed, keep a tuple of Option and DummyFunction objects corresponding to those arguments.
62ae3968b2d82f17ec2853714ab565c02a7cb9bc: Bug 1257823 - Move set_define() to the global scope. r=nalexander
Mike Hommey <mh+mozilla@glandium.org> - Wed, 23 Mar 2016 10:22:08 +0900 - rev 290181
Push 18353 by cbook@mozilla.com at Thu, 24 Mar 2016 15:20:25 +0000
Bug 1257823 - Move set_define() to the global scope. r=nalexander For the same reasons as set_config is being moved to the global scope, we're moving set_define to the global scope here. An additional change is that set_define is now part of the sandbox itself instead of being defined within the sandbox, which makes it share the implementation details with set_config.
9d2b6f4c3ee8366365c16cee6cbda90c4688c743: Bug 1257823 - Move set_config() to the global scope. r=nalexander
Mike Hommey <mh+mozilla@glandium.org> - Tue, 22 Mar 2016 14:21:32 +0900 - rev 290180
Push 18353 by cbook@mozilla.com at Thu, 24 Mar 2016 15:20:25 +0000
Bug 1257823 - Move set_config() to the global scope. r=nalexander The way set_config is set currently makes it difficult to introspect moz.configure files to know what configuration items are being set, because they're hidden in the control flow of functions. This makes some of the moz.configure more convoluted, but this is why there are templates, and we can improve the recurring cases afterwards.
8debac7402e85ae922bf42c752bc835794b5e40f: Bug 1257823 - Split ConfigureSandbox._db. r=nalexander
Mike Hommey <mh+mozilla@glandium.org> - Wed, 23 Mar 2016 14:49:29 +0900 - rev 290179
Push 18353 by cbook@mozilla.com at Thu, 24 Mar 2016 15:20:25 +0000
Bug 1257823 - Split ConfigureSandbox._db. r=nalexander Currently, ConfigureSandbox._db stores two different kind of information. This split those in two different instance variables instead, making things clearer.
1cca90a25cbbcdb0d4c8616f4104d4d1a993a025: Bug 1257823 - Move set_define() to the sandbox. r=nalexander
Mike Hommey <mh+mozilla@glandium.org> - Wed, 23 Mar 2016 11:05:54 +0900 - rev 290178
Push 18353 by cbook@mozilla.com at Thu, 24 Mar 2016 15:20:25 +0000
Bug 1257823 - Move set_define() to the sandbox. r=nalexander In order to make the transition to global set_define easier, move its current definition from inside the sandbox to the sandbox itself.
abc466f526d949eaef90e5cbe8de6527ced2c69d: Bug 1256552 - Disable C4577 to unblock compilation on VS2015; r=jrmuizel
Gregory Szorc <gps@mozilla.com> - Wed, 23 Mar 2016 16:39:56 -0700 - rev 290177
Push 18353 by cbook@mozilla.com at Thu, 24 Mar 2016 15:20:25 +0000
Bug 1256552 - Disable C4577 to unblock compilation on VS2015; r=jrmuizel As part of unblocking building with VS2015u1 in automation, I'm mass disabling compiler warnings that are turned into errors. This is not the preferred mechanism to fix compilation warnings. So hopefully this patch never lands because someone insists of fixing the underlying problem instead. But if it does land, hopefully the workaround is only temporary. MozReview-Commit-ID: BNhXxMhJaZh
c26f9e1dbad7eef1d74ec2e1c7c6aa7fe023a501: Bug 1249849: Make sure the correct implementations of malloc and free are used in DLL blocklist; r=glandium
Aaron Klotz <aklotz@mozilla.com> - Mon, 07 Mar 2016 14:12:31 -0800 - rev 290176
Push 18353 by cbook@mozilla.com at Thu, 24 Mar 2016 15:20:25 +0000
Bug 1249849: Make sure the correct implementations of malloc and free are used in DLL blocklist; r=glandium MozReview-Commit-ID: 7piMI6MwOZh
1726498035c1ddfab4a0492f433f6b236c87098f: Bug 1258299 - Move RTCIdentityProvider to a dictionary, r=jib,khuey
Martin Thomson <martin.thomson@gmail.com> - Thu, 24 Mar 2016 03:03:06 +1100 - rev 290175
Push 18353 by cbook@mozilla.com at Thu, 24 Mar 2016 15:20:25 +0000
Bug 1258299 - Move RTCIdentityProvider to a dictionary, r=jib,khuey MozReview-Commit-ID: EP1fCvQlYqH
9b6ff1fc5e7a4a46e1d81a4aa277ff02b234136f: Bug 1256502 - Use a uint32_t cast to avoid C4319 on VS2015; r=botond
Gregory Szorc <gps@mozilla.com> - Wed, 23 Mar 2016 15:12:30 -0700 - rev 290174
Push 18353 by cbook@mozilla.com at Thu, 24 Mar 2016 15:20:25 +0000
Bug 1256502 - Use a uint32_t cast to avoid C4319 on VS2015; r=botond The sizeof() was creating a size_t which made VS complain about a mismatch between uint32_t and size_t. The sizeof() should never be larger than a uint32_t, so just cast it. MozReview-Commit-ID: 5HrFOfQhAMp
7e3275233f42c2e78f4f2db796e884998eda4599: Bug 1259025 - Enable test_reftests_with_caret.html on e10s; r=mrbkap
Ehsan Akhgari <ehsan@mozilla.com> - Tue, 22 Mar 2016 18:05:36 -0400 - rev 290173
Push 18353 by cbook@mozilla.com at Thu, 24 Mar 2016 15:20:25 +0000
Bug 1259025 - Enable test_reftests_with_caret.html on e10s; r=mrbkap
b0f1eb9c1f386de175b3c2e06d85d9896c33353e: Bug 1257779 - Lower a loop-limit (number selected by some local experimentation) to avoid cgc timeouts. r=orange in a CLOSED TREE
Jeff Walden <jwalden@mit.edu> - Wed, 23 Mar 2016 16:29:46 -0700 - rev 290172
Push 18353 by cbook@mozilla.com at Thu, 24 Mar 2016 15:20:25 +0000
Bug 1257779 - Lower a loop-limit (number selected by some local experimentation) to avoid cgc timeouts. r=orange in a CLOSED TREE
fee477cd153bac782159094148c72b9076bc5766: Backed out 3 changesets (bug 1254134) for build failures in LayerTreeInvalidation.cpp, entirely unrelated to the current CLOSED TREE
Wes Kocher <wkocher@mozilla.com> - Wed, 23 Mar 2016 15:55:20 -0700 - rev 290171
Push 18353 by cbook@mozilla.com at Thu, 24 Mar 2016 15:20:25 +0000
Backed out 3 changesets (bug 1254134) for build failures in LayerTreeInvalidation.cpp, entirely unrelated to the current CLOSED TREE Backed out changeset 5a0d2c853e13 (bug 1254134) Backed out changeset f650dd293a4b (bug 1254134) Backed out changeset 3b6b646ea0e2 (bug 1254134) MozReview-Commit-ID: 2iQvX8NsARG
050be20c329ffee6a43faeb721c46c15016db341: Bug 1259182 - Shrink keyCodeData. r=bz.
Nicholas Nethercote <nnethercote@mozilla.com> - Wed, 16 Mar 2016 14:36:37 +1100 - rev 290170
Push 18353 by cbook@mozilla.com at Thu, 24 Mar 2016 15:20:25 +0000
Bug 1259182 - Shrink keyCodeData. r=bz. This reduces static data size by 1480 B per process.
5a0d2c853e13240e9546a9ba7fafab9ac0784eef: Bug 1254134 - Remove unused mask flag. r=mattwoodrow
Kartikaya Gupta <kgupta@mozilla.com> - Wed, 23 Mar 2016 11:05:42 -0400 - rev 290169
Push 18353 by cbook@mozilla.com at Thu, 24 Mar 2016 15:20:25 +0000
Bug 1254134 - Remove unused mask flag. r=mattwoodrow MozReview-Commit-ID: BmUyzuxDg0M
(0) -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 tip