Makes views be initially hidden

examples/nesting_app/nesting_app.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/bind.h"
 #include "base/macros.h"
 #include "base/message_loop/message_loop.h"
 #include "base/strings/stringprintf.h"
 #include "examples/bitmap_uploader/bitmap_uploader.h"
 #include "examples/window_manager/window_manager.mojom.h"
@@ skipping 51 matching lines @@
   virtual void OnEmbed(ViewManager* view_manager,
                        View* root,
                        ServiceProviderImpl* exported_services,
                        scoped_ptr<ServiceProvider> imported_services) override {
     root->AddObserver(this);
     bitmap_uploader_.reset(new BitmapUploader(root));
     bitmap_uploader_->Init(shell_);
     bitmap_uploader_->SetColor(SK_ColorCYAN);
 
     nested_ = View::Create(view_manager);
+    nested_->SetVisible(true);

[Elliot Glaysher, 2014/11/20 20:34:37]

If you're going to add SetVisible() calls to let people configure things so
there isn't flicker, should you do these after you've SetBounds()?

[sky, 2014/11/20 20:51:30]

I'm actually doing this more for animations. But point taken. Updated.

     root->AddChild(nested_);
     Rect rect;
     rect.x = rect.y = 20;
     rect.width = rect.height = 50;
     nested_->SetBounds(rect);
     nested_->Embed(kEmbeddedAppURL);
   }
   virtual void OnViewManagerDisconnected(ViewManager* view_manager) override {
     base::MessageLoop::current()->Quit();
   }
@@ skipping 18 matching lines @@
   DISALLOW_COPY_AND_ASSIGN(NestingApp);
 };
 
 }  // namespace examples
 }  // namespace mojo
 
 MojoResult MojoMain(MojoHandle shell_handle) {
   mojo::ApplicationRunnerChromium runner(new mojo::examples::NestingApp);
   return runner.Run(shell_handle);
 }