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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: Source/core/page/FrameView.cpp

Issue 14550002: Making GraphicsContext the owner of PlatformContext. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fixing tests and addressing comments. Created 7 years, 8 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 | « Source/WebKit/chromium/tests/WebFrameTest.cpp ('k') | Source/core/platform/chromium/support/GraphicsContext3DPrivate.cpp » ('j') | Source/core/platform/graphics/skia/ImageBufferSkia.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/page/FrameView.cpp
diff --git a/Source/core/page/FrameView.cpp b/Source/core/page/FrameView.cpp
index 4757b48ee2ac59f7965506ae8056467552c12dfd..eb9f270fd3c4598e9e34bb6cc5132454869a3599 100644
--- a/Source/core/page/FrameView.cpp
+++ b/Source/core/page/FrameView.cpp
@@ -2990,8 +2990,8 @@ void FrameView::paintControlTints()
{
if (needsLayout())
layout();
- PlatformGraphicsContext* const noContext = 0;
- GraphicsContext context(noContext);
+ // FIXME: THe use of paint seems like overkill: crbug.com/236892
jamesr 2013/04/30 19:57:08 typo "THe"
+ GraphicsContext context(0); // NULL canvas to get a non-painting context.
context.setUpdatingControlTints(true);
paint(&context, frameRect());
}
« no previous file with comments | « Source/WebKit/chromium/tests/WebFrameTest.cpp ('k') | Source/core/platform/chromium/support/GraphicsContext3DPrivate.cpp » ('j') | Source/core/platform/graphics/skia/ImageBufferSkia.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698