[Blimp Engine] Adds a WindowTreeClient so that popup will have non-null paren…

blimp/engine/app/ui/blimp_window_tree_client.h

+// Copyright 2016 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 BLIMP_ENGINE_APP_UI_BLIMP_WINDOW_TREE_CLIENT_H_
+#define BLIMP_ENGINE_APP_UI_BLIMP_WINDOW_TREE_CLIENT_H_
+
+#include "base/macros.h"
+#include "ui/aura/client/window_tree_client.h"
+
+namespace blimp {
+namespace engine {
+
+class BlimpWindowTreeClient : public aura::client::WindowTreeClient {
+ public:
+  explicit BlimpWindowTreeClient(aura::Window* root_window);

[Wez, 2016/03/29 21:20:06]

Add a brief comment explaining the lifetime expectation for |root_window|, since
it is a bare pointer.

[haibinlu, 2016/03/29 21:57:15]

Done.

+  ~BlimpWindowTreeClient() override;
+
+  // Overridden from aura::client::WindowTreeClient.

[Wez, 2016/03/29 21:20:06]

Conventionally we use the form

// aura::client::WindowTreeClient interface.

[haibinlu, 2016/03/29 21:57:15]

Done.

+  aura::Window* GetDefaultParent(aura::Window* context,
+                                 aura::Window* window,
+                                 const gfx::Rect& bounds) override;
+
+ private:
+  aura::Window* root_window_;
+
+  DISALLOW_COPY_AND_ASSIGN(BlimpWindowTreeClient);
+};
+
+}  // namespace engine
+}  // namespace blimp
+
+#endif  // BLIMP_ENGINE_APP_UI_BLIMP_WINDOW_TREE_CLIENT_H_