Pass origin information for remote frame creation.

url/origin.cc

Index: url/origin.cc
diff --git a/url/origin.cc b/url/origin.cc
index eb2cf141126bd0c83d70f461907483128354310b..a09c212a8ab3d02651be6fe10c9f57cc5d5cb3ce 100644
--- a/url/origin.cc
+++ b/url/origin.cc
@@ -13,7 +13,8 @@ Origin::Origin() : string_("null") {}
 Origin::Origin(const std::string& origin) : string_(origin) {
   DCHECK(origin == "null" || MatchPattern(origin, "?*://?*"));
   DCHECK_GT(origin.size(), 0u);
-  DCHECK_NE(origin[origin.size() - 1], '/');
+// this conflicts with how GURL::GetOrigin works:
+// DCHECK_NE(origin[origin.size() - 1], '/');

[alexmos, 2014/11/12 00:05:47]

I commented out this check since GURL::GetOrigin() always returns origins that
end with '/'.  I could just remove the trailing '/' instead of this, though it
feels the two classes should be  consistent for this.  This won't be necessary
if we switch FrameReplicationState to use GURL or another class to track origins
instead of url::Origin.

[Charlie Reis, 2014/11/13 18:00:59]

Hmm, yes, let's continue the discussion about what class to use in
FrameReplicationState.

[alexmos, 2014/11/18 18:25:33]

Undone, this is no longer necessary given that Blink is now sending us the
serialized origin.  And actually, not having the '/' at the end was correct per
RFC 6454.

 }
 
 }  // namespace url