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

Unified Diff: blimp/client/feature/compositor/blimp_compositor.cc

Issue 1840843005: blimp: Add logging for compositor and render widget feature. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed Haibin's comments. Created 4 years, 9 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: blimp/client/feature/compositor/blimp_compositor.cc
diff --git a/blimp/client/feature/compositor/blimp_compositor.cc b/blimp/client/feature/compositor/blimp_compositor.cc
index 3bb910fd64272a2d32560081e3bbc736a4d912ce..dc889419afaf93c3c221b366b9893e4cb19675bb 100644
--- a/blimp/client/feature/compositor/blimp_compositor.cc
+++ b/blimp/client/feature/compositor/blimp_compositor.cc
@@ -165,6 +165,9 @@ void BlimpCompositor::SetVisibleInternal(bool visible) {
if (!host_)
return;
+ VLOG(1) << "Setting visibility to: " << visible
+ << " for render widget: " << render_widget_id_;
+
if (visible && window_ != gfx::kNullAcceleratedWidget) {
// If we're supposed to be visible and we have a valid
// gfx::AcceleratedWidget make our compositor visible. If the compositor
@@ -186,6 +189,7 @@ void BlimpCompositor::SetVisibleInternal(bool visible) {
void BlimpCompositor::CreateLayerTreeHost(
const cc::proto::InitializeImpl& initialize_message) {
DCHECK(!host_);
+ VLOG(1) << "Creating LayerTreeHost for render widget: " << render_widget_id_;
// Create the LayerTreeHost
cc::LayerTreeHost::InitParams params;
@@ -217,6 +221,8 @@ void BlimpCompositor::CreateLayerTreeHost(
void BlimpCompositor::DestroyLayerTreeHost() {
DCHECK(host_);
+ VLOG(1) << "Destroying LayerTreeHost for render widget: "
+ << render_widget_id_;
// Tear down the output surface connection with the old LayerTreeHost
// instance.
SetVisibleInternal(false);

Powered by Google App Engine
This is Rietveld 408576698