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

Unified Diff: Source/platform/scroll/ScrollbarThemeMacNonOverlayAPI.mm

Issue 1220583004: Refactor DrawingRecorders to check for cached drawings earlier (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Rebase Created 5 years, 5 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
Index: Source/platform/scroll/ScrollbarThemeMacNonOverlayAPI.mm
diff --git a/Source/platform/scroll/ScrollbarThemeMacNonOverlayAPI.mm b/Source/platform/scroll/ScrollbarThemeMacNonOverlayAPI.mm
index 4758d15c4e92d8831a425f6e4c124ba0299346e6..b2283ae770f87674f9c4af8735b12355dc262c11 100644
--- a/Source/platform/scroll/ScrollbarThemeMacNonOverlayAPI.mm
+++ b/Source/platform/scroll/ScrollbarThemeMacNonOverlayAPI.mm
@@ -79,10 +79,11 @@ void ScrollbarThemeMacNonOverlayAPI::updateButtonPlacement()
bool ScrollbarThemeMacNonOverlayAPI::paint(ScrollbarThemeClient* scrollbar, GraphicsContext* context, const IntRect& damageRect)
{
DisplayItem::Type displayItemType = scrollbar->orientation() == HorizontalScrollbar ? DisplayItem::ScrollbarHorizontal : DisplayItem::ScrollbarVertical;
- DrawingRecorder recorder(*context, *scrollbar, displayItemType, scrollbar->frameRect());
- if (recorder.canUseCachedDrawing())
+ if (DrawingRecorder::useCachedDrawingIfPossible(*context, *scrollbar, displayItemType))
return true;
+ DrawingRecorder recorder(*context, *scrollbar, displayItemType, scrollbar->frameRect());
+
// Get the tickmarks for the frameview.
Vector<IntRect> tickmarks;
scrollbar->getTickmarks(tickmarks);
« no previous file with comments | « Source/platform/scroll/ScrollbarThemeMacCommon.mm ('k') | Source/platform/scroll/ScrollbarThemeMacOverlayAPI.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698