mash: Move directly linked NewWindowDelegate to mojom::NewWindowClient.

ash/common/new_window_client_remote.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 ASH_COMMON_NEW_WINDOW_CLIENT_REMOTE_H_
+#define ASH_COMMON_NEW_WINDOW_CLIENT_REMOTE_H_
+
+#include "ash/public/interfaces/new_window.mojom.h"
+#include "base/macros.h"
+
+namespace service_manager {
+class Connector;
+}
+
+namespace ash {
+
+// A NewWindowClient which lazily connects to an exported mojom::NewWindowClient
+// in "service:content_browser" on first use.
+class NewWindowClientRemote : public mojom::NewWindowClient {

[James Cook, 2016/10/21 22:22:31]

I'm OK with NewWindowClient, NewWindowClientImpl, or even NewWindowClientProxy.

[Elliot Glaysher, 2016/10/21 22:45:56]

NewWindowClientProxy it is. (I want to have something in the name referencing
that it will be contacting a remote object.)

+ public:
+  NewWindowClientRemote(service_manager::Connector* connector);
+  ~NewWindowClientRemote() override;
+
+  // NewWindowClient:
+  void NewTab() override;
+  void NewWindow(bool incognito) override;
+  void OpenFileManager() override;
+  void OpenCrosh() override;
+  void OpenGetHelp() override;
+  void RestoreTab() override;
+  void ShowKeyboardOverlay() override;
+  void ShowTaskManager() override;
+  void OpenFeedbackPage() override;
+
+ private:
+  // Ensures that we have a valid |client_| before we try to use it.
+  void EnsureInterface();
+
+  // Clears |client_| when |client_| has a connection error.
+  void OnClientConnectionError();
+
+  service_manager::Connector* connector_;
+  mojom::NewWindowClientPtr client_;
+
+  DISALLOW_COPY_AND_ASSIGN(NewWindowClientRemote);
+};
+
+}  // namespace ash
+
+#endif  // ASH_COMMON_NEW_WINDOW_CLIENT_REMOTE_H_