Bug 1022612. Part 41: Use itemType instead of calling GetType() again. r=mattwoodrow
☠☠ backed out by 8121cd2778b5 ☠ ☠
authorRobert O'Callahan <robert@ocallahan.org>
Fri, 11 Jul 2014 13:18:10 +1200
changeset 214830 5c1f3340c45f018d6653edfe0ddd7195bad28da2
parent 214829 e6be65e455a2c99098c41b175c85c0ad4bb882fc
child 214831 f12d7d68b685bb894d4f23efc6af638c789eb9e6
push id3857
push userraliiev@mozilla.com
push dateTue, 02 Sep 2014 16:39:23 +0000
treeherdermozilla-beta@5638b907b505 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmattwoodrow
bugs1022612
milestone33.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 1022612. Part 41: Use itemType instead of calling GetType() again. r=mattwoodrow
layout/base/FrameLayerBuilder.cpp
--- a/layout/base/FrameLayerBuilder.cpp
+++ b/layout/base/FrameLayerBuilder.cpp
@@ -2815,17 +2815,17 @@ ContainerState::ProcessDisplayItems(nsDi
         ownLayer->SetClipRect(nullptr);
       }
       ThebesLayerData* data = GetTopThebesLayerData();
       if (data) {
         // Prerendered transform items can be updated without layer building
         // (async animations or an empty transaction), so we need to put items
         // that the transform item can potentially move under into a layer
         // above this item.
-        if (item->GetType() == nsDisplayItem::TYPE_TRANSFORM &&
+        if (itemType == nsDisplayItem::TYPE_TRANSFORM &&
             nsDisplayTransform::ShouldPrerenderTransformedContent(mBuilder,
                                                                   item->Frame(),
                                                                   false)) {
           if (!itemClip.HasClip()) {
             // The transform item can move anywhere, treat all other content
             // as being above this item.
             data->SetAllDrawingAbove();
           } else {