Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(68)

Side by Side Diff: third_party/WebKit/Source/platform/graphics/paint/ClipDisplayItem.h

Issue 2513893002: Make ifdefs consistent in WebKit/Source/core/ (Closed)
Patch Set: Fix build. Update after review. Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef ClipDisplayItem_h 5 #ifndef ClipDisplayItem_h
6 #define ClipDisplayItem_h 6 #define ClipDisplayItem_h
7 7
8 #include "SkRegion.h" 8 #include "SkRegion.h"
9 #include "platform/PlatformExport.h" 9 #include "platform/PlatformExport.h"
10 #include "platform/geometry/FloatRoundedRect.h" 10 #include "platform/geometry/FloatRoundedRect.h"
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
57 EndClipDisplayItem(const DisplayItemClient& client, Type type) 57 EndClipDisplayItem(const DisplayItemClient& client, Type type)
58 : PairedEndDisplayItem(client, type, sizeof(*this)) { 58 : PairedEndDisplayItem(client, type, sizeof(*this)) {
59 ASSERT(isEndClipType(type)); 59 ASSERT(isEndClipType(type));
60 } 60 }
61 61
62 void replay(GraphicsContext&) const override; 62 void replay(GraphicsContext&) const override;
63 void appendToWebDisplayItemList(const IntRect&, 63 void appendToWebDisplayItemList(const IntRect&,
64 WebDisplayItemList*) const override; 64 WebDisplayItemList*) const override;
65 65
66 private: 66 private:
67 #if ENABLE(ASSERT) 67 #ifdef NDEBUG
68 bool isEndAndPairedWith(DisplayItem::Type otherType) const final { 68 bool isEndAndPairedWith(DisplayItem::Type otherType) const final {
69 return DisplayItem::isClipType(otherType); 69 return DisplayItem::isClipType(otherType);
Alexander Alekseev 2016/11/18 22:32:15 This is declared in the base class only if DCHECK_
70 } 70 }
71 #endif 71 #endif
72 }; 72 };
73 73
74 } // namespace blink 74 } // namespace blink
75 75
76 #endif // ClipDisplayItem_h 76 #endif // ClipDisplayItem_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698