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..44c5fcd4be5cd6900ad793d6920a446864c6be51 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 char kOrigin[] = "https://google.com";

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

Nit: constexpr if possible

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

AFAIK, const is preferred for char[]. A quick check of codesearch seems to
support that

[dcheng, 2017/01/27 09:11:11]

Many of our string literals were in the codebase before constexpr support =)

+}  // 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")));
+        url::Origin(GURL(kOrigin))));
     EXPECT_TRUE(observer_->Init());
   }