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

Unified Diff: third_party/WebKit/Source/core/paint/PaintLayer.h

Issue 2291723002: Manually rename \<m_[0-9]... fields to help rewrite_to_chrome_style tool. (Closed)
Patch Set: Created 4 years, 4 months 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/paint/PaintLayer.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/paint/PaintLayer.h
diff --git a/third_party/WebKit/Source/core/paint/PaintLayer.h b/third_party/WebKit/Source/core/paint/PaintLayer.h
index bf0330163de5eaf2444b49d99f3ec0e0336e1455..d003945cf4f18d84301a29ab3fa3d2d1ac88a7ab 100644
--- a/third_party/WebKit/Source/core/paint/PaintLayer.h
+++ b/third_party/WebKit/Source/core/paint/PaintLayer.h
@@ -293,11 +293,11 @@ public:
bool subtreeIsInvisible() const { return !hasVisibleContent() && !hasVisibleDescendant(); }
- // FIXME: hasVisibleContent() should call updateDescendantDependentFlags() if m_visibleContentStatusDirty.
- bool hasVisibleContent() const { ASSERT(!m_visibleContentStatusDirty); return m_hasVisibleContent; }
+ // FIXME: hasVisibleContent() should call updateDescendantDependentFlags() if m_isVisibleContentDirty.
+ bool hasVisibleContent() const { DCHECK(!m_isVisibleContentDirty); return m_hasVisibleContent; }
- // FIXME: hasVisibleDescendant() should call updateDescendantDependentFlags() if m_visibleDescendantStatusDirty.
- bool hasVisibleDescendant() const { ASSERT(!m_visibleDescendantStatusDirty); return m_hasVisibleDescendant; }
+ // FIXME: hasVisibleDescendant() should call updateDescendantDependentFlags() if m_isVisibleDescendantDirty.
+ bool hasVisibleDescendant() const { DCHECK(!m_isVisibleDescendantDirty); return m_hasVisibleDescendant; }
void dirtyVisibleContentStatus();
void potentiallyDirtyVisibleContentStatus(EVisibility);
@@ -707,7 +707,7 @@ public:
void dirty3DTransformedDescendantStatus();
// Both updates the status, and returns true if descendants of this have 3d.
bool update3DTransformedDescendantStatus();
- bool has3DTransformedDescendant() const { DCHECK(!m_3DTransformedDescendantStatusDirty); return m_has3DTransformedDescendant; }
+ bool has3DTransformedDescendant() const { DCHECK(!m_is3DTransformedDescendantDirty); return m_has3DTransformedDescendant; }
#if CHECK_DISPLAY_ITEM_CLIENT_ALIVENESS
void endShouldKeepAliveAllClientsRecursive();
@@ -813,16 +813,16 @@ private:
const unsigned m_isRootLayer : 1;
- unsigned m_visibleContentStatusDirty : 1;
+ unsigned m_isVisibleContentDirty : 1;
unsigned m_hasVisibleContent : 1;
- unsigned m_visibleDescendantStatusDirty : 1;
+ unsigned m_isVisibleDescendantDirty : 1;
unsigned m_hasVisibleDescendant : 1;
#if ENABLE(ASSERT)
unsigned m_needsPositionUpdate : 1;
#endif
- unsigned m_3DTransformedDescendantStatusDirty : 1;
+ unsigned m_is3DTransformedDescendantDirty : 1;
// Set on a stacking context layer that has 3D descendants anywhere
// in a preserves3D hierarchy. Hint to do 3D-aware hit testing.
unsigned m_has3DTransformedDescendant : 1;
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/paint/PaintLayer.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698