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

Unified Diff: cc/trees/layer_tree_host.h

Issue 1336733002: Re-land: cc: Implement shared worker contexts. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: shutdown fix Created 5 years, 3 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/trees/layer_tree_host.h
diff --git a/cc/trees/layer_tree_host.h b/cc/trees/layer_tree_host.h
index e17431d89c00f4b35cec785252110860e273ec28..29990f930c3725aabb473c96b0d1309caa7bf61f 100644
--- a/cc/trees/layer_tree_host.h
+++ b/cc/trees/layer_tree_host.h
@@ -434,6 +434,13 @@ class CC_EXPORT LayerTreeHost : public MutatorHostClient {
int meta_information_sequence_number_;
scoped_ptr<RenderingStatsInstrumentation> rendering_stats_instrumentation_;
+ // |current_output_surface_| can't be updated until we've successfully
+ // initialized a new output surface. |new_output_surface_| contains the
+ // new output surface that is currently being initialized. If initialization
+ // is successful then |new_output_surface_| replaces
+ // |current_output_surface_|.
+ scoped_ptr<OutputSurface> new_output_surface_;
danakj 2015/09/12 00:09:14 +sievers: From what I can tell there's no good rea
reveman 2015/09/12 12:49:57 Sgtm but I think I prefer to save that for a follo
danakj 2015/09/14 18:05:46 Ya that's all I mean. No TODO is fine I think. I
no sievers 2015/09/14 21:18:05 Are you suggesting as a follow-up to move the owne
danakj 2015/09/14 21:22:47 I was thinking the former. Documenting ownership/l
+ scoped_ptr<OutputSurface> current_output_surface_;
bool output_surface_lost_;
scoped_refptr<Layer> root_layer_;

Powered by Google App Engine
This is Rietveld 408576698