Convert MediaRouter mojom apis to intake url::Origin objects instead of strings

chrome/browser/media/router/media_sinks_observer_unittest.cc

Index: chrome/browser/media/router/media_sinks_observer_unittest.cc
diff --git a/chrome/browser/media/router/media_sinks_observer_unittest.cc b/chrome/browser/media/router/media_sinks_observer_unittest.cc
index c5a340cdb0637fb4e4fd7de2504663f6f18630ac..839a485b31d6c4c2fc77df8b82107c348586dffc 100644
--- a/chrome/browser/media/router/media_sinks_observer_unittest.cc
+++ b/chrome/browser/media/router/media_sinks_observer_unittest.cc
@@ -14,8 +14,8 @@ TEST(MediaSinksObserverTest, OriginMatching) {
   MockMediaRouter router;
   MediaSource source(
       MediaSourceForPresentationUrl(GURL("https://presentation.com")));
-  GURL origin("https://origin.com");
-  std::vector<GURL> origin_list;
+  url::Origin origin((GURL("https://origin.com")));

[dcheng, 2017/01/26 04:16:32]

Nit: Here and elsewhere write:
url::Origin origin{GURL("https://origin.com")};

to avoid errors from the most vexing parse (see
https://sites.google.com/a/chromium.org/dev/developers/coding-style/cpp-dos-a...,
search for "most vexing")

[steimel, 2017/01/27 00:28:59]

Done.

+  std::vector<url::Origin> origin_list;

[mark a. foltz, 2017/01/24 04:04:17]

nit: origin_list({origin}) instead of calling push_back.

[steimel, 2017/01/25 19:01:11]

Done.

   origin_list.push_back(origin);
   std::vector<MediaSink> sink_list;
   sink_list.push_back(MediaSink("sinkId", "Sink", MediaSink::IconType::CAST));
@@ -25,9 +25,9 @@ TEST(MediaSinksObserverTest, OriginMatching) {
   observer.OnSinksUpdated(sink_list, origin_list);
 
   EXPECT_CALL(observer, OnSinksReceived(SequenceEquals(sink_list)));
-  observer.OnSinksUpdated(sink_list, std::vector<GURL>());
+  observer.OnSinksUpdated(sink_list, std::vector<url::Origin>());
 
-  GURL origin2("https://differentOrigin.com");
+  url::Origin origin2((GURL("https://differentOrigin.com")));
   origin_list.clear();
   origin_list.push_back(origin2);
   EXPECT_CALL(observer, OnSinksReceived(testing::IsEmpty()));