Bug 1597587 - Remove unnecessary dirty rect intersection. r=sotaro
authorMarkus Stange <mstange@themasta.com>
Tue, 19 Nov 2019 21:35:15 +0000
changeset 502725 cf994af1ea817a9fdaa11431331f22cf6452ff1e
parent 502724 6294fecb7978f666a32ade2dc006d84ba5c2b37e
child 502726 0c63dcd7a1c6ff0fab249cd8395f3841a6bf3215
push id36823
push usermalexandru@mozilla.com
push dateWed, 20 Nov 2019 09:47:58 +0000
treeherdermozilla-central@79821df17239 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerssotaro
bugs1597587
milestone72.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1597587 - Remove unnecessary dirty rect intersection. r=sotaro Depends on D53289 Differential Revision: https://phabricator.services.mozilla.com/D53769
gfx/webrender_bindings/DCLayerTree.cpp
--- a/gfx/webrender_bindings/DCLayerTree.cpp
+++ b/gfx/webrender_bindings/DCLayerTree.cpp
@@ -324,19 +324,16 @@ bool DCLayer::CreateEGLSurfaceForComposi
   HRESULT hr;
   const auto gl = mDCLayerTree->GetGLContext();
   const auto config = mDCLayerTree->GetEGLConfig();
   RefPtr<ID3D11Texture2D> backBuf;
   POINT offset;
 
   LayoutDeviceIntRect dirtyRect(aDirtyRect.origin.x, aDirtyRect.origin.y,
                                 aDirtyRect.size.width, aDirtyRect.size.height);
-  // Bind is sometimes called with a dirty rect that extends beyond the layer
-  // (bug 1593845).
-  dirtyRect = dirtyRect.Intersect(LayoutDeviceIntRect({}, mBufferSize));
 
   RECT update_rect;
   update_rect.left = dirtyRect.X();
   update_rect.top = dirtyRect.Y();
   update_rect.right = dirtyRect.XMost();
   update_rect.bottom = dirtyRect.YMost();
 
   RECT* rect = &update_rect;