From 00886da677f1e71f21941c0fc13469ca77a9552b Mon Sep 17 00:00:00 2001 From: Scott Violet Date: Thu, 20 Nov 2014 13:04:25 -0800 Subject: [PATCH] Makes views be initially hidden This better matches what other toolkits do and lets the client configure the view before we attempt to show something for it. BUG=434429 TEST=covered by test R=erg@chromium.org Review URL: https://codereview.chromium.org/745743002 --- engine/src/flutter/tools/debugger/debugger.cc | 1 + engine/src/flutter/tools/tester/tester.cc | 1 + 2 files changed, 2 insertions(+) diff --git a/engine/src/flutter/tools/debugger/debugger.cc b/engine/src/flutter/tools/debugger/debugger.cc index e2d4c908b0..77cc2615ec 100644 --- a/engine/src/flutter/tools/debugger/debugger.cc +++ b/engine/src/flutter/tools/debugger/debugger.cc @@ -59,6 +59,7 @@ void SkyDebugger::OnEmbed( content_ = mojo::View::Create(view_manager_); content_->SetBounds(root_->bounds()); root_->AddChild(content_); + content_->SetVisible(true); window_manager_app_->InitFocus( scoped_ptr(new mojo::BasicFocusRules(content_))); diff --git a/engine/src/flutter/tools/tester/tester.cc b/engine/src/flutter/tools/tester/tester.cc index c71a6f7516..baf69c2965 100644 --- a/engine/src/flutter/tools/tester/tester.cc +++ b/engine/src/flutter/tools/tester/tester.cc @@ -74,6 +74,7 @@ class SkyTester : public mojo::ApplicationDelegate, content_ = mojo::View::Create(view_manager_); content_->SetBounds(root_->bounds()); root_->AddChild(content_); + content_->SetVisible(true); std::cout << "#READY\n"; std::cout.flush();