[Media Router] Add Media Router Mojo impl code.

chrome/browser/media/router/media_routes_observer.h

 // Copyright 2015 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_MEDIA_ROUTER_MEDIA_ROUTES_OBSERVER_H_
 #define CHROME_BROWSER_MEDIA_ROUTER_MEDIA_ROUTES_OBSERVER_H_
 
 #include <vector>
 
 #include "base/macros.h"
 #include "chrome/browser/media/router/media_route.h"
 
 namespace media_router {
 
 class MediaRouter;
 
 // Base class for observing when the set of MediaRoutes and their associated
 // MediaSinks have been updated.
 class MediaRoutesObserver {
  public:
   explicit MediaRoutesObserver(MediaRouter* router);
   virtual ~MediaRoutesObserver();
 
   // This function is invoked when the list of routes and their associated

[Wez, 2015/05/26 23:41:30]

nit: No need for "This function is"; can start with "Invoked when.." here.

[imcheng (use chromium acct), 2015/05/27 01:06:04]

Done.

   // sinks have been updated. Routes included in the list are created either
   // locally or remotely.

[Wez, 2015/05/26 23:41:30]

nit: "Routes included in the list..." - not clear what this comment actually
means? Is it relevant to the caller's behaviour, or can we remove it?

[imcheng (use chromium acct), 2015/05/27 01:06:04]

Done.

+  // Implementations may not perform operations that modifies the Media Router's

[Wez, 2015/05/26 23:41:30]

s/modifies/modify

[imcheng (use chromium acct), 2015/05/27 01:06:04]

Done.

+  // observer list. In particular, invoking this instance's destructor within

[Wez, 2015/05/26 23:41:30]

nit: s/instance/observer

[imcheng (use chromium acct), 2015/05/27 01:06:04]

Done.

+  // this method will result in undefined behavior.

[Wez, 2015/05/26 23:41:30]

nit: s/this method/OnRoutesUpdated

[imcheng (use chromium acct), 2015/05/27 01:06:04]

Done.

   virtual void OnRoutesUpdated(const std::vector<MediaRoute>& routes) {}
 
  private:
   MediaRouter* router_;
 
   DISALLOW_COPY_AND_ASSIGN(MediaRoutesObserver);
 };
 
 }  // namespace media_router
 
 #endif  // CHROME_BROWSER_MEDIA_ROUTER_MEDIA_ROUTES_OBSERVER_H_