[Media Router] Add Media Router Mojo impl code.

chrome/browser/media/router/media_routes_observer.h

Index: chrome/browser/media/router/media_routes_observer.h
diff --git a/chrome/browser/media/router/media_routes_observer.h b/chrome/browser/media/router/media_routes_observer.h
index b4faa60ab0b6ce2519f0336e4edfc960054cfc92..3123778582572154b031142d09f860c3fde15be3 100644
--- a/chrome/browser/media/router/media_routes_observer.h
+++ b/chrome/browser/media/router/media_routes_observer.h
@@ -24,6 +24,9 @@ class 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: