mash: Have chrome set itself as a controller interface for changing volume

chrome/browser/ui/ash/volume_controller.h

 // Copyright (c) 2012 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_UI_ASH_VOLUME_CONTROLLER_CHROMEOS_H_
-#define CHROME_BROWSER_UI_ASH_VOLUME_CONTROLLER_CHROMEOS_H_
+#ifndef CHROME_BROWSER_UI_ASH_VOLUME_CONTROLLER_H_
+#define CHROME_BROWSER_UI_ASH_VOLUME_CONTROLLER_H_
 
 #include "ash/public/interfaces/volume.mojom.h"
 #include "base/macros.h"
-#include "mojo/public/cpp/bindings/binding_set.h"
+#include "mojo/public/cpp/bindings/binding.h"
 
 // Controls the volume when F8-10 or a multimedia key for volume is pressed.

[msw, 2016/12/05 20:15:09]

optional nit: add a TODO about broken accelerators in mash? crbug.com/647781 

[James Cook, 2016/12/06 19:23:57]

Done.

-class VolumeController : public ash::mojom::VolumeController {
+class VolumeController : public ash::mojom::VolumeClient {

[msw, 2016/12/05 20:15:09]

Yeah, the controller implementing a client, just doesn't make sense...

[James Cook, 2016/12/06 19:23:57]

Done.

  public:
   VolumeController();
   ~VolumeController() override;
 
-  // Binds the mojom::VolumeController interface request to this object.
-  void BindRequest(ash::mojom::VolumeControllerRequest request);
-
-  // Overridden from ash::mojom::VolumeController:
+  // Overridden from ash::mojom::VolumeClient:
   void VolumeMute() override;
   void VolumeDown() override;
   void VolumeUp() override;
 
  private:
-  mojo::BindingSet<ash::mojom::VolumeController> bindings_;
+  mojo::Binding<ash::mojom::VolumeClient> binding_;
 
   DISALLOW_COPY_AND_ASSIGN(VolumeController);
 };
 
-#endif  // CHROME_BROWSER_UI_ASH_VOLUME_CONTROLLER_CHROMEOS_H_
+#endif  // CHROME_BROWSER_UI_ASH_VOLUME_CONTROLLER_H_