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

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

Issue 2639223002: [DO NOT COMMIT] Don't keep pointers to table sections when told to recalculate sections. (Closed)
Patch Set: Created 3 years, 11 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
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/layout/LayoutObject.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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, 2010 Apple Inc. 6 * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc.
7 * All rights reserved. 7 * All rights reserved.
8 * Copyright (C) 2013 Adobe Systems Incorporated. All rights reserved. 8 * Copyright (C) 2013 Adobe Systems Incorporated. All rights reserved.
9 * 9 *
10 * This library is free software; you can redistribute it and/or 10 * This library is free software; you can redistribute it and/or
(...skipping 2305 matching lines...) Expand 10 before | Expand all | Expand 10 after
2316 if (parent == ancestorToStopAt) 2316 if (parent == ancestorToStopAt)
2317 break; 2317 break;
2318 } 2318 }
2319 rect.move(-offsetFromContainer); 2319 rect.move(-offsetFromContainer);
2320 } 2320 }
2321 2321
2322 bool LayoutBox::mapToVisualRectInAncestorSpace( 2322 bool LayoutBox::mapToVisualRectInAncestorSpace(
2323 const LayoutBoxModelObject* ancestor, 2323 const LayoutBoxModelObject* ancestor,
2324 LayoutRect& rect, 2324 LayoutRect& rect,
2325 VisualRectFlags visualRectFlags) const { 2325 VisualRectFlags visualRectFlags) const {
2326 DCHECK(!needsLayout());
2326 inflateVisualRectForFilter(rect); 2327 inflateVisualRectForFilter(rect);
2327 2328
2328 if (ancestor == this) 2329 if (ancestor == this)
2329 return true; 2330 return true;
2330 2331
2331 bool ancestorSkipped; 2332 bool ancestorSkipped;
2332 bool filterSkipped; 2333 bool filterSkipped;
2333 LayoutObject* container = 2334 LayoutObject* container =
2334 this->container(ancestor, &ancestorSkipped, &filterSkipped); 2335 this->container(ancestor, &ancestorSkipped, &filterSkipped);
2335 LayoutBox* tableRowContainer = nullptr; 2336 LayoutBox* tableRowContainer = nullptr;
(...skipping 3353 matching lines...) Expand 10 before | Expand all | Expand 10 after
5689 block->adjustChildDebugRect(rect); 5690 block->adjustChildDebugRect(rect);
5690 5691
5691 return rect; 5692 return rect;
5692 } 5693 }
5693 5694
5694 bool LayoutBox::shouldClipOverflow() const { 5695 bool LayoutBox::shouldClipOverflow() const {
5695 return hasOverflowClip() || styleRef().containsPaint() || hasControlClip(); 5696 return hasOverflowClip() || styleRef().containsPaint() || hasControlClip();
5696 } 5697 }
5697 5698
5698 } // namespace blink 5699 } // namespace blink
OLDNEW
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/layout/LayoutObject.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698