Call ShowRootWindow on NativeWindow's RootWindow to display the window.

content/shell/shell_aura.cc

 // Copyright (c) 2012 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 "content/shell/shell.h"
 
 #include "base/command_line.h"
 #include "base/utf_string_conversions.h"
 #include "content/public/browser/web_contents.h"
 #include "ui/aura/env.h"
@@ skipping 265 matching lines @@
 
 aura::client::StackingClient* Shell::stacking_client_ = NULL;
 views::ViewsDelegate* Shell::views_delegate_ = NULL;
 
 // static
 void Shell::PlatformInitialize() {
 #if defined(OS_CHROMEOS)
   chromeos::DBusThreadManager::Initialize();
 #endif
 #if defined(OS_CHROMEOS)
-  stacking_client_ = new content::ShellStackingClientAsh();
   gfx::Screen::SetScreenInstance(
       gfx::SCREEN_TYPE_NATIVE, new aura::TestScreen);
+  aura::RootWindow* root_window = CreateRootWindow(kTestWindowWidth, kTestWindowHeight);

[Nayan, 2013/01/15 23:39:03]

Done.

+  stacking_client_ = new content::ShellStackingClientAsh(root_window);
 #else
   stacking_client_ = new views::DesktopStackingClient();
   gfx::Screen::SetScreenInstance(
       gfx::SCREEN_TYPE_NATIVE, views::CreateDesktopScreen());
 #endif
   aura::client::SetStackingClient(stacking_client_);
   views_delegate_ = new ShellViewsDelegateAura();
 }
 
+aura::RootWindow* Shell::CreateRootWindow(int width, int height) {

[sky, 2013/01/15 21:53:49]

Convention for static members/methods is to put:
// static
on the previous line.

[Nayan, 2013/01/15 23:39:03]

Done.

+  aura::RootWindow* root_window = new aura::RootWindow(
+      aura::RootWindow::CreateParams(gfx::Rect(0, 0, width, height)));
+  root_window->Init();
+  return root_window;
+}
+
 void Shell::PlatformExit() {
   if (stacking_client_)
     delete stacking_client_;
   if (views_delegate_)
     delete views_delegate_;
 #if defined(OS_CHROMEOS)
   chromeos::DBusThreadManager::Shutdown();
 #endif
   aura::Env::DeleteInstance();
 }
@@ skipping 20 matching lines @@
   ShellWindowDelegateView* delegate_view =
     static_cast<ShellWindowDelegateView*>(window_widget_->widget_delegate());
   delegate_view->SetAddressBarURL(url);
 }
 
 void Shell::PlatformSetIsLoading(bool loading) {
 }
 
 void Shell::PlatformCreateWindow(int width, int height) {
   window_widget_ =
-      views::Widget::CreateWindowWithBounds(new ShellWindowDelegateView(this),
-               gfx::Rect(0, 0, width, height));
+      views::Widget::CreateWindowWithContextAndBounds(new ShellWindowDelegateView(this),

[Nayan, 2013/01/15 23:39:03]

Done.

+                                                      Shell::stacking_client_->GetDefaultParent(NULL, NULL, gfx::Rect()),
+                                                      gfx::Rect(0, 0, width, height));
   window_ = window_widget_->GetNativeWindow();
+  // Explicitly call show on the root window.

[sky, 2013/01/15 21:53:49]

This just says what the code does. Comment should describe why the code is the
way it is.

[Nayan, 2013/01/15 23:39:03]

Done.

+  window_->GetRootWindow()->ShowRootWindow();
   window_widget_->Show();
 }
 
 void Shell::PlatformSetContents() {
   ShellWindowDelegateView* delegate_view =
     static_cast<ShellWindowDelegateView*>(window_widget_->widget_delegate());
   delegate_view->SetWebContents(web_contents_.get());
 }
 
 void Shell::PlatformResizeSubViews() {
 }
 
 void Shell::Close() {
   window_widget_->Close();
 }
 
 void Shell::PlatformSetTitle(const string16& title) {
   ShellWindowDelegateView* delegate_view =
     static_cast<ShellWindowDelegateView*>(window_widget_->widget_delegate());
   delegate_view->SetWindowTitle(title);
   window_widget_->UpdateWindowTitle();
 }
 
 }  // namespace content