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

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

Issue 2523533002: Call scrollbarsChanged in PLSA::updateAfterStyleChange. (Closed)
Patch Set: fix test Created 4 years 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) 1997 Martin Jones (mjones@kde.org) 2 * Copyright (C) 1997 Martin Jones (mjones@kde.org)
3 * (C) 1997 Torben Weis (weis@kde.org) 3 * (C) 1997 Torben Weis (weis@kde.org)
4 * (C) 1998 Waldo Bastian (bastian@kde.org) 4 * (C) 1998 Waldo Bastian (bastian@kde.org)
5 * (C) 1999 Lars Knoll (knoll@kde.org) 5 * (C) 1999 Lars Knoll (knoll@kde.org)
6 * (C) 1999 Antti Koivisto (koivisto@kde.org) 6 * (C) 1999 Antti Koivisto (koivisto@kde.org)
7 * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2009 Apple Inc. 7 * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2009 Apple Inc.
8 * All rights reserved. 8 * 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 1362 matching lines...) Expand 10 before | Expand all | Expand 10 after
1373 const LayoutPoint& paintOffset) const { 1373 const LayoutPoint& paintOffset) const {
1374 TableCellPainter(*this).paintBoxDecorationBackground(paintInfo, paintOffset); 1374 TableCellPainter(*this).paintBoxDecorationBackground(paintInfo, paintOffset);
1375 } 1375 }
1376 1376
1377 void LayoutTableCell::paintMask(const PaintInfo& paintInfo, 1377 void LayoutTableCell::paintMask(const PaintInfo& paintInfo,
1378 const LayoutPoint& paintOffset) const { 1378 const LayoutPoint& paintOffset) const {
1379 TableCellPainter(*this).paintMask(paintInfo, paintOffset); 1379 TableCellPainter(*this).paintMask(paintInfo, paintOffset);
1380 } 1380 }
1381 1381
1382 void LayoutTableCell::scrollbarsChanged(bool horizontalScrollbarChanged, 1382 void LayoutTableCell::scrollbarsChanged(bool horizontalScrollbarChanged,
1383 bool verticalScrollbarChanged) { 1383 bool verticalScrollbarChanged,
1384 ScrollbarChangeContext context) {
1384 LayoutBlock::scrollbarsChanged(horizontalScrollbarChanged, 1385 LayoutBlock::scrollbarsChanged(horizontalScrollbarChanged,
1385 verticalScrollbarChanged); 1386 verticalScrollbarChanged);
1387 if (context != Layout)
1388 return;
1389
1386 int scrollbarHeight = scrollbarLogicalHeight(); 1390 int scrollbarHeight = scrollbarLogicalHeight();
1387 // Not sure if we should be doing something when a scrollbar goes away or not. 1391 // Not sure if we should be doing something when a scrollbar goes away or not.
1388 if (!scrollbarHeight) 1392 if (!scrollbarHeight)
1389 return; 1393 return;
1390 1394
1391 // We only care if the scrollbar that affects our intrinsic padding has been 1395 // We only care if the scrollbar that affects our intrinsic padding has been
1392 // added. 1396 // added.
1393 if ((isHorizontalWritingMode() && !horizontalScrollbarChanged) || 1397 if ((isHorizontalWritingMode() && !horizontalScrollbarChanged) ||
1394 (!isHorizontalWritingMode() && !verticalScrollbarChanged)) 1398 (!isHorizontalWritingMode() && !verticalScrollbarChanged))
1395 return; 1399 return;
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
1477 cb->adjustChildDebugRect(rect); 1481 cb->adjustChildDebugRect(rect);
1478 1482
1479 return rect; 1483 return rect;
1480 } 1484 }
1481 1485
1482 void LayoutTableCell::adjustChildDebugRect(LayoutRect& r) const { 1486 void LayoutTableCell::adjustChildDebugRect(LayoutRect& r) const {
1483 r.move(0, -intrinsicPaddingBefore()); 1487 r.move(0, -intrinsicPaddingBefore());
1484 } 1488 }
1485 1489
1486 } // namespace blink 1490 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698