Place the spinner in the right corner of the controls window.

chrome/browser/chromeos/login/helper.h

 // Copyright (c) 2010 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.
 //
 // This file contains helper functions used by Chromium OS login.
 
 #ifndef CHROME_BROWSER_CHROMEOS_LOGIN_HELPER_H_
 #define CHROME_BROWSER_CHROMEOS_LOGIN_HELPER_H_
 #pragma once
 
+#include "third_party/skia/include/core/SkColor.h"
 #include "views/controls/button/native_button.h"
-#include "third_party/skia/include/core/SkColor.h"
+#include "views/widget/widget_gtk.h"
 
 class GURL;
 
 namespace gfx {
 class Rect;
 class Size;
 }  // namespace gfx
 
 namespace views {
 class Label;
 class MenuButton;
 class Painter;
 class Textfield;
 class Throbber;
+class Widget;
 }  // namespace views
 
 namespace chromeos {
 
+// This class manages showing of the throbber in the separete popup widget.

[whywhat, 2010/12/09 16:20:18]

nit: separete -> separate, showing of -> showing, in the -> in a

[altimofeev, 2010/12/10 16:37:40]

Done.

+class ThrobberManager {
+ public:
+  ThrobberManager();
+  virtual ~ThrobberManager();
+
+  // Creates throbber above the given host in the right side using the shift
+  // given. Also places throbber in the middle of the vertical host size.
+  void StartShow(views::Widget* host,

[whywhat, 2010/12/09 16:20:18]

Why not just Start?
Also, maybe pass host and throbber in ctor?

[altimofeev, 2010/12/10 16:37:40]

Actually it creates popup and starts the throbber. So switched to
'CreateAndStart' naming. This also explains why throbber and host have to be
passed directly to this method and not in ctor (also added comment about it).

+                 views::Throbber* throbber,
+                 int right_shift);

[whywhat, 2010/12/09 16:20:18]

Mb right_shift -> right_margin?

[altimofeev, 2010/12/10 16:37:40]

Done.

+
+  // Creates throbber above the given host using the relative bounds given.
+  void StartShow(views::Widget* host,
+                 views::Throbber* throbber,
+                 const gfx::Rect relative_bounds);

[whywhat, 2010/12/09 16:20:18]

const gfx::Rect&?

[altimofeev, 2010/12/10 16:37:40]

Done.

+
+  // Stops and hides the throbber. Throbber view is no longer valid.
+  void Stop();
+
+ private:
+  // Holds the widget that shows the throbber.
+  views::Widget* throbber_widget_;
+};

[whywhat, 2010/12/09 16:20:18]

DISABLE_COPY_AND_ASSIGN?

[altimofeev, 2010/12/10 16:37:40]

Done.

+
 // Creates default smoothed throbber for time consuming operations on login.
 views::Throbber* CreateDefaultSmoothedThrobber();
 
 // Creates default throbber.
 views::Throbber* CreateDefaultThrobber();
 
 // Creates painter for login background.
 views::Painter* CreateBackgroundPainter();
 
 // Returns bounds of the screen to use for login wizard.
@@ skipping 33 matching lines @@
 
 // Background color of the login controls.
 const SkColor kBackgroundColor = SK_ColorWHITE;
 
 // Text color on the login controls.
 const SkColor kTextColor = SK_ColorWHITE;
 
 // Default link color on login/OOBE controls.
 const SkColor kLinkColor = 0xFF0066CC;
 
+const int kThrobberRightShift = 10;
+
 // Default size of the OOBE screen. Includes 10px shadow from each side.
 // See rounded_rect_painter.cc for border definitions.
 const int kWizardScreenWidth = 800;
 const int kWizardScreenHeight = 450;
 
 const int kScreenCornerRadius = 10;
 const int kUserCornerRadius = 6;
 
 // Username label height in different states.
 const int kSelectedLabelHeight = 25;
@@ skipping 29 matching lines @@
 
 // New pod sizes.
 const int kNewUserPodFullWidth = 372;
 const int kNewUserPodFullHeight = 372;
 const int kNewUserPodSmallWidth = 360;
 const int kNewUserPodSmallHeight = 322;
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_LOGIN_HELPER_H_