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

Unified Diff: cc/output/delegating_renderer.cc

Issue 514123002: Manual fixups in compositor code scoped_refptr operator T* removal. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 4 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: cc/output/delegating_renderer.cc
diff --git a/cc/output/delegating_renderer.cc b/cc/output/delegating_renderer.cc
index 8a9f46188be887696e92bc23670da55115d6ed2f..af637646530e482de346ed77bd073867928be362 100644
--- a/cc/output/delegating_renderer.cc
+++ b/cc/output/delegating_renderer.cc
@@ -118,7 +118,7 @@ void DelegatingRenderer::DidChangeVisibility() {
if (!visible()) {
TRACE_EVENT0("cc", "DelegatingRenderer::SetVisible dropping resources");
resource_provider_->ReleaseCachedData();
- if (context_provider) {
+ if (context_provider.get()) {
context_provider->DeleteCachedResources();
context_provider->ContextGL()->Flush();
}
@@ -126,7 +126,7 @@ void DelegatingRenderer::DidChangeVisibility() {
// We loop visibility to the GPU process, since that's what manages memory.
// That will allow it to feed us with memory allocations that we can act
// upon.
- if (context_provider)
+ if (context_provider.get())
context_provider->ContextSupport()->SetSurfaceVisible(visible());
}

Powered by Google App Engine
This is Rietveld 408576698