Cut VrShell dependency on WebUi.

chrome/browser/android/vr_shell/ui_interface.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 CHROME_BROWSER_ANDROID_VR_SHELL_UI_INTERFACE_H_
 #define CHROME_BROWSER_ANDROID_VR_SHELL_UI_INTERFACE_H_
 
 #include "base/macros.h"
 #include "base/values.h"
 
-class VrShellUIMessageHandler;
+namespace vr_shell {
 
-namespace vr_shell {
+class UiCommandHandler {

[bshe, 2016/10/20 22:26:03]

nit: perhaps add delegate to name since this is essentially a delegate?

[cjgrant, 2016/10/21 13:44:39]

Hmmm, I see this as an interface rather than delegate.  go/style says interfaces
may optionally end with an Interface suffix, but don't have to.  I'll put on the
Interface suffix for now, and we can discuss.

+ public:
+  virtual void SendCommandToUi(const base::Value& value) = 0;
+};
 
 // This class manages the communication of browser state from VR shell to the
 // HTML UI. State information is asynchronous and unidirectional.
 class UiInterface {
  public:
   enum Mode {
     STANDARD,
     WEB_VR,
   };
 
   UiInterface();
   virtual ~UiInterface();
 
   void SetMode(Mode mode);
   void SetSecureOrigin(bool secure);
 
   // Called by WebUI when starting VR.
   void OnDomContentsLoaded();
-  void SetUiMessageHandler(VrShellUIMessageHandler* handler);
+  void SetUiCommandHandler(UiCommandHandler* handler);
 
  private:
   void FlushUpdates();
 
-  VrShellUIMessageHandler* handler_;
+  UiCommandHandler* handler_;
   bool loaded_ = false;
   base::DictionaryValue updates_;
 
   DISALLOW_COPY_AND_ASSIGN(UiInterface);
 };
 
 }  // namespace vr_shell
 
 #endif  // CHROME_BROWSER_ANDROID_VR_SHELL_UI_INTERFACE_H_