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

Unified Diff: content/common/gpu/client/grcontext_for_gles2_interface.cc

Issue 1862693002: Replace skia::RefPtr with sk_sp<> in content/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 4 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
Index: content/common/gpu/client/grcontext_for_gles2_interface.cc
diff --git a/content/common/gpu/client/grcontext_for_gles2_interface.cc b/content/common/gpu/client/grcontext_for_gles2_interface.cc
index 8805a4abf09377159b34b8152ea9c8926c4b6d6d..eda63a3c73e723f92936dd969c0c414a67a20c8c 100644
--- a/content/common/gpu/client/grcontext_for_gles2_interface.cc
+++ b/content/common/gpu/client/grcontext_for_gles2_interface.cc
@@ -20,10 +20,10 @@ namespace content {
GrContextForGLES2Interface::GrContextForGLES2Interface(
gpu::gles2::GLES2Interface* gl) {
- sk_sp<GrGLInterface> interface(
- skia_bindings::CreateGLES2InterfaceBindings(gl));
- gr_context_ = skia::AdoptRef(
- GrContext::Create(kOpenGL_GrBackend,
+ sk_sp<GrGLInterface> interface = sk_ref_sp(
+ skia_bindings::CreateGLES2InterfaceBindings(gl).get());
f(malita) 2016/04/06 18:37:21 This looks odd - is there anything wrong with the
tomhudson 2016/04/06 19:00:23 Done. Ugh, I was counting on the rebase or post-r
+ gr_context_ =
+ sk_sp<GrContext>(GrContext::Create(kOpenGL_GrBackend,
// GrContext takes ownership of |interface|.
reinterpret_cast<GrBackendContext>(interface.get())));
if (gr_context_) {

Powered by Google App Engine
This is Rietveld 408576698