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

chrome/browser/media/router/presentation_media_sinks_observer_unittest.cc

Index: chrome/browser/media/router/presentation_media_sinks_observer_unittest.cc
diff --git a/chrome/browser/media/router/presentation_media_sinks_observer_unittest.cc b/chrome/browser/media/router/presentation_media_sinks_observer_unittest.cc
index 96166a8b2b1eccc5ac18fc138fbc44b170321803..aae70985c0f9872c9dc37106f45de4fb344bf837 100644
--- a/chrome/browser/media/router/presentation_media_sinks_observer_unittest.cc
+++ b/chrome/browser/media/router/presentation_media_sinks_observer_unittest.cc
@@ -20,6 +20,10 @@ using testing::Return;
 
 namespace media_router {
 
+namespace {
+const url::Origin kOrigin = url::Origin((GURL("https://google.com")));

[mark a. foltz, 2017/01/17 21:30:45]

Don't use complex ctors in static declarations.

https://google.github.io/styleguide/cppguide.html#Static_and_Global_Variables

Can you revert to doing this in the Test ctor?

Also, there are extra ().

[steimel, 2017/01/18 01:52:35]

Done.

+}  // namespace
+
 class PresentationMediaSinksObserverTest : public ::testing::Test {
  public:
   PresentationMediaSinksObserverTest()
@@ -31,7 +35,7 @@ class PresentationMediaSinksObserverTest : public ::testing::Test {
     observer_.reset(new PresentationMediaSinksObserver(
         &router_, &listener_, MediaSourceForPresentationUrl(
                                   GURL("http://example.com/presentation.html")),
-        GURL("https://google.com")));
+        kOrigin));
     EXPECT_TRUE(observer_->Init());
   }