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

Side by Side Diff: third_party/WebKit/Source/core/layout/line/LineLayoutState.h

Issue 2015523004: Don't explicitly initialize LayoutUnit to 0. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Go through the subdirs of core/layout/ too. Created 4 years, 7 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
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2000 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 2000 Lars Knoll (knoll@kde.org)
3 * Copyright (C) 2003, 2004, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All r ight reserved. 3 * Copyright (C) 2003, 2004, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All r ight reserved.
4 * Copyright (C) 2010 Google Inc. All rights reserved. 4 * Copyright (C) 2010 Google Inc. All rights reserved.
5 * Copyright (C) 2014 Adobe Systems Incorporated. All rights reserved. 5 * Copyright (C) 2014 Adobe Systems Incorporated. All rights reserved.
6 * 6 *
7 * This library is free software; you can redistribute it and/or 7 * This library is free software; you can redistribute it and/or
8 * modify it under the terms of the GNU Library General Public 8 * modify it under the terms of the GNU Library General Public
9 * License as published by the Free Software Foundation; either 9 * License as published by the Free Software Foundation; either
10 * version 2 of the License, or (at your option) any later version. 10 * version 2 of the License, or (at your option) any later version.
(...skipping 21 matching lines...) Expand all
32 32
33 // Like LayoutState for layout(), LineLayoutState keeps track of global informat ion 33 // Like LayoutState for layout(), LineLayoutState keeps track of global informat ion
34 // during an entire linebox tree layout pass (aka layoutInlineChildren). 34 // during an entire linebox tree layout pass (aka layoutInlineChildren).
35 class LineLayoutState { 35 class LineLayoutState {
36 STACK_ALLOCATED(); 36 STACK_ALLOCATED();
37 public: 37 public:
38 LineLayoutState(bool fullLayout, LayoutUnit& paintInvalidationLogicalTop, La youtUnit& paintInvalidationLogicalBottom) 38 LineLayoutState(bool fullLayout, LayoutUnit& paintInvalidationLogicalTop, La youtUnit& paintInvalidationLogicalBottom)
39 : m_lastFloat(nullptr) 39 : m_lastFloat(nullptr)
40 , m_endLine(nullptr) 40 , m_endLine(nullptr)
41 , m_floatIndex(0) 41 , m_floatIndex(0)
42 , m_endLineLogicalTop(0)
43 , m_endLineMatched(false) 42 , m_endLineMatched(false)
44 , m_hasInlineChild(false) 43 , m_hasInlineChild(false)
45 , m_isFullLayout(fullLayout) 44 , m_isFullLayout(fullLayout)
46 , m_paintInvalidationLogicalTop(paintInvalidationLogicalTop) 45 , m_paintInvalidationLogicalTop(paintInvalidationLogicalTop)
47 , m_paintInvalidationLogicalBottom(paintInvalidationLogicalBottom) 46 , m_paintInvalidationLogicalBottom(paintInvalidationLogicalBottom)
48 , m_usesPaintInvalidationBounds(false) 47 , m_usesPaintInvalidationBounds(false)
49 { } 48 { }
50 49
51 void markForFullLayout() { m_isFullLayout = true; } 50 void markForFullLayout() { m_isFullLayout = true; }
52 bool isFullLayout() const { return m_isFullLayout; } 51 bool isFullLayout() const { return m_isFullLayout; }
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
112 LayoutUnit& m_paintInvalidationLogicalBottom; 111 LayoutUnit& m_paintInvalidationLogicalBottom;
113 112
114 LayoutUnit m_adjustedLogicalLineTop; 113 LayoutUnit m_adjustedLogicalLineTop;
115 114
116 bool m_usesPaintInvalidationBounds; 115 bool m_usesPaintInvalidationBounds;
117 }; 116 };
118 117
119 } // namespace blink 118 } // namespace blink
120 119
121 #endif // LineLayoutState_h 120 #endif // LineLayoutState_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698