cea8af673cece90d538a0fc4782aa1b15aa6c020: Bug 1383131: Fall back to ole32.dll if combase.dll is not present; r=davidb
Aaron Klotz <aklotz@mozilla.com> - Fri, 20 Oct 2017 14:31:20 -0600 - rev 684869
Push 85740 by bmo:rail@mozilla.com at Mon, 23 Oct 2017 17:05:30 +0000
Bug 1383131: Fall back to ole32.dll if combase.dll is not present; r=davidb MozReview-Commit-ID: 3plQGwqaeP9
b721be9535ab7fa072f51f88eddac4510d14c463: Bug 1410279 - Fix bad merge of bug 1407351, restoring the process id indicator on tabs (nightly only). r=me
Felipe Gomes <felipc@gmail.com> - Mon, 23 Oct 2017 14:33:59 -0200 - rev 684868
Push 85740 by bmo:rail@mozilla.com at Mon, 23 Oct 2017 17:05:30 +0000
Bug 1410279 - Fix bad merge of bug 1407351, restoring the process id indicator on tabs (nightly only). r=me Just relanding the parts of the patch that were accidentally left out during a merge. These changes were all r=mikedeboer MozReview-Commit-ID: 7yFZ4WIR8y7
91f976f5e4c5b59db6a8b1eb6bc39c3ec992774d: Backed out changeset dc9d9c01080a (bug 1399453) for static bustage at layers/CompositorThread.h:68: bad implicit conversion constructor for 'CompositorThreadHolderDebug'. r=backout on a CLOSED TREE
Sebastian Hengst <archaeopteryx@coole-files.de> - Mon, 23 Oct 2017 18:26:14 +0200 - rev 684867
Push 85740 by bmo:rail@mozilla.com at Mon, 23 Oct 2017 17:05:30 +0000
Backed out changeset dc9d9c01080a (bug 1399453) for static bustage at layers/CompositorThread.h:68: bad implicit conversion constructor for 'CompositorThreadHolderDebug'. r=backout on a CLOSED TREE
aaf86ab8bea034ee41a172b494752facbc88a396: Bug 1402044 - don't inline InputEventStatistics::Get(); r=erahm
Nathan Froyd <froydnj@mozilla.com> - Mon, 23 Oct 2017 12:15:16 -0400 - rev 684866
Push 85740 by bmo:rail@mozilla.com at Mon, 23 Oct 2017 17:05:30 +0000
Bug 1402044 - don't inline InputEventStatistics::Get(); r=erahm Defining Get() in the declaration of InputEventStatistics implicitly sticks an "inline" on the function, which is not what we want: inlining it spreads around a lot of static initialization code. Providing an out-of-line definition is much better in terms of code size.
dc9d9c01080a3a9d36af97706a0ec612663c48b0: Bug 1399453 - Temporarily annotate crash reports with compositor thread owners. r=dvander
Andrew Osmond <aosmond@mozilla.com> - Mon, 23 Oct 2017 11:31:02 -0400 - rev 684865
Push 85740 by bmo:rail@mozilla.com at Mon, 23 Oct 2017 17:05:30 +0000
Bug 1399453 - Temporarily annotate crash reports with compositor thread owners. r=dvander From the crashes associated with bug 1389021, we know that some compositor thread IPDL owners are not being cleaned up properly. We do not know which protocols are causing the problem, so we temporarily will annotate the logs with the ownership status. This should be limited to under a dozen instances of the protocols. This change will be backed out after a few builds are produced with it and we see the first crash reports with the relevant data.
cd56917208167befaec59970e26d4dbe865fbedc: Mark SourceRotatedBuffer methods as override to silence warnings (bug 1409871 part 22, r=nical)
Ryan Hunt <rhunt@eqrion.net> - Fri, 20 Oct 2017 16:32:39 -0400 - rev 684864
Push 85740 by bmo:rail@mozilla.com at Mon, 23 Oct 2017 17:05:30 +0000
Mark SourceRotatedBuffer methods as override to silence warnings (bug 1409871 part 22, r=nical) MozReview-Commit-ID: 1xNF8sfXukg
f68539cff202944efd59ba1b1264abd17c88912f: Add documentation for the new rotated buffer subclasses. (bug 1409871 part 21, r=nical)
Ryan Hunt <rhunt@eqrion.net> - Fri, 20 Oct 2017 12:48:20 -0400 - rev 684863
Push 85740 by bmo:rail@mozilla.com at Mon, 23 Oct 2017 17:05:30 +0000
Add documentation for the new rotated buffer subclasses. (bug 1409871 part 21, r=nical) MozReview-Commit-ID: E2XejdUju9Q
a4e35be8cf4c3cb4f2723817568b3d0b64bf806a: Be sure to unlock the old back buffer before we release it. (bug 1409871 part 20, r=nical)
Ryan Hunt <rhunt@eqrion.net> - Fri, 20 Oct 2017 12:28:41 -0400 - rev 684862
Push 85740 by bmo:rail@mozilla.com at Mon, 23 Oct 2017 17:05:30 +0000
Be sure to unlock the old back buffer before we release it. (bug 1409871 part 20, r=nical) MozReview-Commit-ID: HR9EiYrDeqT
56fe4c3c3b04054b46a27df97d62a6c7d5916a2c: Remove mInAsyncPaint and move locking new back buffer into BeginPaint (bug 1409871 part 19, r=nical)
Ryan Hunt <rhunt@eqrion.net> - Wed, 18 Oct 2017 15:31:15 -0400 - rev 684861
Push 85740 by bmo:rail@mozilla.com at Mon, 23 Oct 2017 17:05:30 +0000
Remove mInAsyncPaint and move locking new back buffer into BeginPaint (bug 1409871 part 19, r=nical) mInAsyncPaint is only needed because the code using CreateBuffer expects the buffer to be locked when it is returned. This isn't necessary because we can now lock in BeginPaint. MozReview-Commit-ID: 8WazvBKMbvb
352a6f520d27c968a5556e54d10f48e97f938880: Remove BeginPaintBuffer (bug 1409871 part 18, r=nical)
Ryan Hunt <rhunt@eqrion.net> - Wed, 18 Oct 2017 15:20:14 -0400 - rev 684860
Push 85740 by bmo:rail@mozilla.com at Mon, 23 Oct 2017 17:05:30 +0000
Remove BeginPaintBuffer (bug 1409871 part 18, r=nical) Now that BeginPaintBuffer and BeginPaint are both implemented by content client, I think it makes sense to just merge them into one method. This simplifies the interface clients have to use. MozReview-Commit-ID: 8bjH6WcpZS9
466f20f34a1313590be38b32af758287263e16f1: Clean up some methods on ContentClient (bug 1409871 part 17, r=nical)
Ryan Hunt <rhunt@eqrion.net> - Wed, 18 Oct 2017 14:54:26 -0400 - rev 684859
Push 85740 by bmo:rail@mozilla.com at Mon, 23 Oct 2017 17:05:30 +0000
Clean up some methods on ContentClient (bug 1409871 part 17, r=nical) This commit does some more cleanup on the content client class hierarchy. Some methods were virtual or instance methods when they didn't need to be. MozReview-Commit-ID: 2y2D3zYtYvM
7e2fc94f5be9cec14adbd4624e72147afbe40f65: Fix PrepareDrawTargetForPainting (bug 1409871 part 16, r=dvander)
Ryan Hunt <rhunt@eqrion.net> - Wed, 18 Oct 2017 14:46:54 -0400 - rev 684858
Push 85740 by bmo:rail@mozilla.com at Mon, 23 Oct 2017 17:05:30 +0000
Fix PrepareDrawTargetForPainting (bug 1409871 part 16, r=dvander) CapturedPaintState::mTarget is actually a dual draw target to the black and white draw targets, so this code will have the white draw target cleared black and then white. This isn't incorrect, it's just wasteful. MozReview-Commit-ID: ItgiSmegPK6
482309a25d13285d50d4e4deab98dcda78a3f2c1: Use a rect instead of a region for UpdateDestinationFrom (bug 1409871 part 15, r=nical)
Ryan Hunt <rhunt@eqrion.net> - Wed, 18 Oct 2017 14:35:18 -0400 - rev 684857
Push 85740 by bmo:rail@mozilla.com at Mon, 23 Oct 2017 17:05:30 +0000
Use a rect instead of a region for UpdateDestinationFrom (bug 1409871 part 15, r=nical) UpdateDestinationFrom actually only uses a rect, so it's best to not be misleading and force the clients of it to do the conversion. MozReview-Commit-ID: EsUv3apqnku
c57db572271df74153bd75270f3dc2c2833c55b5: Cleanup rotated buffer declarations and add some documentation. (bug 1409871 part 14, r=nical)
Ryan Hunt <rhunt@eqrion.net> - Wed, 18 Oct 2017 14:10:56 -0400 - rev 684856
Push 85740 by bmo:rail@mozilla.com at Mon, 23 Oct 2017 17:05:30 +0000
Cleanup rotated buffer declarations and add some documentation. (bug 1409871 part 14, r=nical) This organizes the rotated buffer declarations and adds some documentation. MozReview-Commit-ID: 5r24RiUMM6o
2d9d7435eb63b9739714738fa53c621ba991ff5e: Update comments for ContentClient (bug 1409871 part 13, r=nical)
Ryan Hunt <rhunt@eqrion.net> - Wed, 18 Oct 2017 13:42:19 -0400 - rev 684855
Push 85740 by bmo:rail@mozilla.com at Mon, 23 Oct 2017 17:05:30 +0000
Update comments for ContentClient (bug 1409871 part 13, r=nical) This could be a part of the previous commit, but I omitted it to keep the size down. Just some updates to comments and documentation. MozReview-Commit-ID: HkCOSIQhBNv
7b7bcdf875394dee9cb7d9fe90235580ae7464c1: Kill RotatedContentBuffer (bug 1409871 part 12, r=nical)
Ryan Hunt <rhunt@eqrion.net> - Mon, 16 Oct 2017 19:45:11 -0400 - rev 684854
Push 85740 by bmo:rail@mozilla.com at Mon, 23 Oct 2017 17:05:30 +0000
Kill RotatedContentBuffer (bug 1409871 part 12, r=nical) This commit is the guts of the refactoring. This commit removes rotated content buffer, and moves the necessary functionality to content client. All content clients now do not inherit from rotated content buffer, but contain a refptr to a rotated buffer. For the basic content client, this is a DrawTargetRotatedBuffer. For the remote content client, this is a RemoteRotatedBuffer. The double buffered content client contains an additional RemoteRotatedBuffer for the front buffer. Each derived class of content client only needs to implement a method to create its own rotated buffer. The rest is handled by the base content client. The remote content clients still override some additional methods to sync buffers and do IPC. MozReview-Commit-ID: B9vcSi8UYhm
ad4b3448dc2eeb548175ca478d0e321c1b054cf9: Move buffer related decisions into their own function. (bug 1409871 part 11, r=nical)
Ryan Hunt <rhunt@eqrion.net> - Thu, 12 Oct 2017 19:52:29 -0400 - rev 684853
Push 85740 by bmo:rail@mozilla.com at Mon, 23 Oct 2017 17:05:30 +0000
Move buffer related decisions into their own function. (bug 1409871 part 11, r=nical) This commit splits off the part of BeginPaint that makes the decision about whether to keep the buffer, its surface type, its content type, and the regions to invalidate or paint. MozReview-Commit-ID: JcPlv8GiRpA
0c1485b37ae9557daf075d859f3b9706de7699f0: Add RotatedBuffer::AdjustTo. (bug 1409871 part 10, r=nical)
Ryan Hunt <rhunt@eqrion.net> - Thu, 12 Oct 2017 18:37:29 -0400 - rev 684852
Push 85740 by bmo:rail@mozilla.com at Mon, 23 Oct 2017 17:05:30 +0000
Add RotatedBuffer::AdjustTo. (bug 1409871 part 10, r=nical) This commit adds a method to rotated buffer to perform the adjusting to the new destination rect, and potential unrotating that may need to happen. The goal of this is to simplify and prepare BeginPaint to be moved to content client. MozReview-Commit-ID: B4qeZqLjORR
cce97a15c945f881a2d57dd7c0bf4d5947b1f95a: Remove PaintState::mDidSelfCopy. (bug 1409871 part 9, r=nical)
Ryan Hunt <rhunt@eqrion.net> - Thu, 12 Oct 2017 17:24:50 -0400 - rev 684851
Push 85740 by bmo:rail@mozilla.com at Mon, 23 Oct 2017 17:05:30 +0000
Remove PaintState::mDidSelfCopy. (bug 1409871 part 9, r=nical) Miscellaneous cleanup, PaintState doesn't need to know this as no one uses this information outside of the content client classes. MozReview-Commit-ID: 6K8LxEPvp4V
d57058c8c12e1ab373cbbaa93d71e58c38f04ef1: Move mFrontAndBackBufferDiffer to ContentClientDoubleBuffered. (bug 1409871 part 8, r=nical)
Ryan Hunt <rhunt@eqrion.net> - Thu, 12 Oct 2017 12:58:17 -0400 - rev 684850
Push 85740 by bmo:rail@mozilla.com at Mon, 23 Oct 2017 17:05:30 +0000
Move mFrontAndBackBufferDiffer to ContentClientDoubleBuffered. (bug 1409871 part 8, r=nical) Miscellaneous cleanup, ContentClientSingleBuffered doesn't need to track if the front and back buffer differ. MozReview-Commit-ID: IvAw0b92jsh
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip