Creating default implementation for aura::client::ScreenPositionClient

ui/wm/core/default_screen_position_client.h

-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Copyright (c) 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.
 
-#ifndef UI_VIEWS_WIDGET_DESKTOP_AURA_DESKTOP_SCREEN_POSITION_CLIENT_H_
-#define UI_VIEWS_WIDGET_DESKTOP_AURA_DESKTOP_SCREEN_POSITION_CLIENT_H_
+#ifndef UI_WM_CORE_DEFAULT_SCREEN_POSITION_CLIENT_H_
+#define UI_WM_CORE_DEFAULT_SCREEN_POSITION_CLIENT_H_
 
 #include "ui/aura/client/screen_position_client.h"
-#include "ui/views/views_export.h"
+#include "ui/wm/wm_export.h"
 
-namespace views {
+namespace wm {
 
 // Client that always offsets by the toplevel RootWindow of the passed
 // in child NativeWidgetAura.
-class VIEWS_EXPORT DesktopScreenPositionClient
+class WM_EXPORT DefaultScreenPositionClient
     : public aura::client::ScreenPositionClient {
  public:
-  explicit DesktopScreenPositionClient(aura::Window* root_window);
-  ~DesktopScreenPositionClient() override;
+  DefaultScreenPositionClient();
+  ~DefaultScreenPositionClient() override;
 
   // aura::client::ScreenPositionClient overrides:
   void ConvertPointToScreen(const aura::Window* window,
                             gfx::Point* point) override;
   void ConvertPointFromScreen(const aura::Window* window,
                               gfx::Point* point) override;
   void ConvertHostPointToScreen(aura::Window* window,
                                 gfx::Point* point) override;
   void SetBounds(aura::Window* window,
                  const gfx::Rect& bounds,
                  const gfx::Display& display) override;
 
- private:
-  aura::Window* root_window_;
+ protected:
+  // Returns true if bounds passed to window in SetBounds should be treated as
+  //  though they are in screen coordinates.
+  virtual bool PositionWindowInScreenCoordinates(aura::Window* window);
 
-  DISALLOW_COPY_AND_ASSIGN(DesktopScreenPositionClient);
+  DISALLOW_COPY_AND_ASSIGN(DefaultScreenPositionClient);

[sadrul, 2014/10/30 16:09:11]

This should be in private:

[mfomitchev, 2014/10/30 20:12:54]

Done.

 };
 
-}  // namespace views
+}  // namespace wm
 
-#endif  // UI_VIEWS_WIDGET_DESKTOP_AURA_DESKTOP_SCREEN_POSITION_CLIENT_H_
+#endif  // UI_WM_CORE_DEFAULT_SCREEN_POSITION_CLIENT_H_