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

Unified Diff: sky/engine/platform/scroll/ScrollableArea.cpp

Issue 842113005: Delete a bunch of noop paint invalidation code. (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: fix comment Created 5 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « sky/engine/platform/scroll/ScrollableArea.h ('k') | sky/engine/platform/scroll/Scrollbar.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sky/engine/platform/scroll/ScrollableArea.cpp
diff --git a/sky/engine/platform/scroll/ScrollableArea.cpp b/sky/engine/platform/scroll/ScrollableArea.cpp
index dff4cd9342f2975d5253025004aa0e1cbdb14ef0..a4db72862d238dad26de5bb189e1d14eac0e9fd6 100644
--- a/sky/engine/platform/scroll/ScrollableArea.cpp
+++ b/sky/engine/platform/scroll/ScrollableArea.cpp
@@ -47,7 +47,6 @@ namespace blink {
struct SameSizeAsScrollableArea {
virtual ~SameSizeAsScrollableArea();
- IntRect scrollbarDamage[2];
void* pointer;
unsigned bitfields : 16;
IntPoint origin;
@@ -173,13 +172,6 @@ void ScrollableArea::scrollPositionChanged(const IntPoint& position)
if (horizontalScrollbar->isOverlayScrollbar()) {
if (!verticalScrollbar)
horizontalScrollbar->invalidate();
- else {
- // If there is both a horizontalScrollbar and a verticalScrollbar,
- // then we must also invalidate the corner between them.
- IntRect boundsAndCorner = horizontalScrollbar->boundsRect();
- boundsAndCorner.setWidth(boundsAndCorner.width() + verticalScrollbar->width());
- horizontalScrollbar->invalidateRect(boundsAndCorner);
- }
}
}
if (verticalScrollbar) {
@@ -319,11 +311,6 @@ void ScrollableArea::setScrollbarOverlayStyle(ScrollbarOverlayStyle overlayStyle
}
}
-void ScrollableArea::invalidateScrollbar(Scrollbar* scrollbar, const IntRect& rect)
-{
- invalidateScrollbarRect(scrollbar, rect);
-}
-
bool ScrollableArea::scheduleAnimation()
{
WTF_LOG(ScriptedAnimationController, "ScrollableArea::scheduleAnimation: window = %d",
« no previous file with comments | « sky/engine/platform/scroll/ScrollableArea.h ('k') | sky/engine/platform/scroll/Scrollbar.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698