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

Side by Side Diff: Source/core/rendering/RenderView.h

Issue 327323002: Start removing the double-negative !ASSERT_DISABLED (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 6 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 /* 1 /*
2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
3 * Copyright (C) 2006 Apple Computer, Inc. 3 * Copyright (C) 2006 Apple Computer, Inc.
4 * 4 *
5 * This library is free software; you can redistribute it and/or 5 * This library is free software; you can redistribute it and/or
6 * modify it under the terms of the GNU Library General Public 6 * modify it under the terms of the GNU Library General Public
7 * License as published by the Free Software Foundation; either 7 * License as published by the Free Software Foundation; either
8 * version 2 of the License, or (at your option) any later version. 8 * version 2 of the License, or (at your option) any later version.
9 * 9 *
10 * This library is distributed in the hope that it will be useful, 10 * This library is distributed in the hope that it will be useful,
(...skipping 95 matching lines...) Expand 10 before | Expand all | Expand 10 after
106 ASSERT(!RuntimeEnabledFeatures::repaintAfterLayoutEnabled()); 106 ASSERT(!RuntimeEnabledFeatures::repaintAfterLayoutEnabled());
107 return m_layoutState ? m_layoutState->layoutDelta() : LayoutSize(); 107 return m_layoutState ? m_layoutState->layoutDelta() : LayoutSize();
108 } 108 }
109 void addLayoutDelta(const LayoutSize& delta) 109 void addLayoutDelta(const LayoutSize& delta)
110 { 110 {
111 ASSERT(!RuntimeEnabledFeatures::repaintAfterLayoutEnabled()); 111 ASSERT(!RuntimeEnabledFeatures::repaintAfterLayoutEnabled());
112 if (m_layoutState) 112 if (m_layoutState)
113 m_layoutState->addLayoutDelta(delta); 113 m_layoutState->addLayoutDelta(delta);
114 } 114 }
115 115
116 #if !ASSERT_DISABLED 116 #if ASSERT_ENABLED
117 bool layoutDeltaMatches(const LayoutSize& delta) 117 bool layoutDeltaMatches(const LayoutSize& delta)
118 { 118 {
119 ASSERT(!RuntimeEnabledFeatures::repaintAfterLayoutEnabled()); 119 ASSERT(!RuntimeEnabledFeatures::repaintAfterLayoutEnabled());
120 if (!m_layoutState) 120 if (!m_layoutState)
121 return false; 121 return false;
122 return (delta.width() == m_layoutState->layoutDelta().width() || m_layou tState->layoutDeltaXSaturated()) && (delta.height() == m_layoutState->layoutDelt a().height() || m_layoutState->layoutDeltaYSaturated()); 122 return (delta.width() == m_layoutState->layoutDelta().width() || m_layou tState->layoutDeltaXSaturated()) && (delta.height() == m_layoutState->layoutDelt a().height() || m_layoutState->layoutDeltaYSaturated());
123 } 123 }
124 #endif 124 #endif
125 125
126 bool shouldDoFullRepaintForNextLayout() const; 126 bool shouldDoFullRepaintForNextLayout() const;
(...skipping 252 matching lines...) Expand 10 before | Expand all | Expand 10 after
379 { 379 {
380 m_view.enableLayoutState(); 380 m_view.enableLayoutState();
381 } 381 }
382 private: 382 private:
383 RenderView& m_view; 383 RenderView& m_view;
384 }; 384 };
385 385
386 } // namespace WebCore 386 } // namespace WebCore
387 387
388 #endif // RenderView_h 388 #endif // RenderView_h
OLDNEW
« no previous file with comments | « Source/core/rendering/RenderLayerStackingNode.cpp ('k') | Source/core/rendering/compositing/CompositedLayerMapping.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698