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

Side by Side Diff: third_party/WebKit/Source/core/layout/LayoutBoxModelObject.cpp

Issue 1997033002: Move continuation getter and setter down to LayoutBlockFlow. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase 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) 1999 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
3 * (C) 1999 Antti Koivisto (koivisto@kde.org) 3 * (C) 1999 Antti Koivisto (koivisto@kde.org)
4 * (C) 2005 Allan Sandfeld Jensen (kde@carewolf.com) 4 * (C) 2005 Allan Sandfeld Jensen (kde@carewolf.com)
5 * (C) 2005, 2006 Samuel Weinig (sam.weinig@gmail.com) 5 * (C) 2005, 2006 Samuel Weinig (sam.weinig@gmail.com)
6 * Copyright (C) 2005, 2006, 2007, 2008, 2009 Apple Inc. All rights reserved. 6 * Copyright (C) 2005, 2006, 2007, 2008, 2009 Apple Inc. All rights reserved.
7 * Copyright (C) 2010 Google Inc. All rights reserved. 7 * Copyright (C) 2010 Google Inc. All rights reserved.
8 * 8 *
9 * This library is free software; you can redistribute it and/or 9 * This library is free software; you can redistribute it and/or
10 * modify it under the terms of the GNU Library General Public 10 * modify it under the terms of the GNU Library General Public
(...skipping 444 matching lines...) Expand 10 before | Expand all | Expand 10 after
455 void LayoutBoxModelObject::addOutlineRectsForNormalChildren(Vector<LayoutRect>& rects, const LayoutPoint& additionalOffset, IncludeBlockVisualOverflowOrNot incl udeBlockOverflows) const 455 void LayoutBoxModelObject::addOutlineRectsForNormalChildren(Vector<LayoutRect>& rects, const LayoutPoint& additionalOffset, IncludeBlockVisualOverflowOrNot incl udeBlockOverflows) const
456 { 456 {
457 for (LayoutObject* child = slowFirstChild(); child; child = child->nextSibli ng()) { 457 for (LayoutObject* child = slowFirstChild(); child; child = child->nextSibli ng()) {
458 // Outlines of out-of-flow positioned descendants are handled in LayoutB lock::addOutlineRects(). 458 // Outlines of out-of-flow positioned descendants are handled in LayoutB lock::addOutlineRects().
459 if (child->isOutOfFlowPositioned()) 459 if (child->isOutOfFlowPositioned())
460 continue; 460 continue;
461 461
462 // Outline of an element continuation or anonymous block continuation is added when we iterate the continuation chain. 462 // Outline of an element continuation or anonymous block continuation is added when we iterate the continuation chain.
463 // See LayoutBlock::addOutlineRects() and LayoutInline::addOutlineRects( ). 463 // See LayoutBlock::addOutlineRects() and LayoutInline::addOutlineRects( ).
464 if (child->isElementContinuation() 464 if (child->isElementContinuation()
465 || (child->isLayoutBlock() && toLayoutBlock(child)->isAnonymousBlock Continuation())) 465 || (child->isLayoutBlockFlow() && toLayoutBlockFlow(child)->isAnonym ousBlockContinuation()))
466 continue; 466 continue;
467 467
468 addOutlineRectsForDescendant(*child, rects, additionalOffset, includeBlo ckOverflows); 468 addOutlineRectsForDescendant(*child, rects, additionalOffset, includeBlo ckOverflows);
469 } 469 }
470 } 470 }
471 471
472 void LayoutBoxModelObject::addOutlineRectsForDescendant(const LayoutObject& desc endant, Vector<LayoutRect>& rects, const LayoutPoint& additionalOffset, IncludeB lockVisualOverflowOrNot includeBlockOverflows) const 472 void LayoutBoxModelObject::addOutlineRectsForDescendant(const LayoutObject& desc endant, Vector<LayoutRect>& rects, const LayoutPoint& additionalOffset, IncludeB lockVisualOverflowOrNot includeBlockOverflows) const
473 { 473 {
474 if (descendant.isText() || descendant.isListMarker()) 474 if (descendant.isText() || descendant.isListMarker())
475 return; 475 return;
(...skipping 642 matching lines...) Expand 10 before | Expand all | Expand 10 after
1118 if (rootElementStyle->hasBackground()) 1118 if (rootElementStyle->hasBackground())
1119 return false; 1119 return false;
1120 1120
1121 if (node() != document().firstBodyElement()) 1121 if (node() != document().firstBodyElement())
1122 return false; 1122 return false;
1123 1123
1124 return true; 1124 return true;
1125 } 1125 }
1126 1126
1127 } // namespace blink 1127 } // namespace blink
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/layout/LayoutBlockFlow.h ('k') | third_party/WebKit/Source/core/layout/LayoutInline.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698