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

Unified Diff: components/exo/buffer.cc

Issue 2493223002: Change exo::SurfaceFactoryOwner to exo::ExoCompositorFrameSink (Closed)
Patch Set: Addressed comments Created 4 years, 1 month 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: components/exo/buffer.cc
diff --git a/components/exo/buffer.cc b/components/exo/buffer.cc
index 9b1778d989d6c0250efff832d6372f401e15e129..9d4cddca808b20c7ac0623bd164ed0fa6dcf1591 100644
--- a/components/exo/buffer.cc
+++ b/components/exo/buffer.cc
@@ -432,6 +432,7 @@ std::unique_ptr<cc::SingleReleaseCallback> Buffer::ProduceTextureMailbox(
context_factory->SharedMainThreadContextProvider();
if (!context_provider) {
DLOG(WARNING) << "Failed to acquire a context provider";
+
Fady Samuel 2016/11/23 15:39:02 No need for code churn. Just restore this file to
Alex Z. 2016/11/23 17:46:09 Done.
Release(); // Decrements the use count
return nullptr;
}
@@ -542,7 +543,6 @@ void Buffer::ReleaseTexture(std::unique_ptr<Texture> texture) {
void Buffer::ReleaseContentsTexture(std::unique_ptr<Texture> texture) {
TRACE_EVENT0("exo", "Buffer::ReleaseContentsTexture");
-
contents_texture_ = std::move(texture);
Release();
}

Powered by Google App Engine
This is Rietveld 408576698