Add std::string constructors for Origin/SchemeHostPort to reduce copies

content/child/blink_platform_impl_unittest.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/child/blink_platform_impl.h"
 
 #include <stdint.h>
 
 #include "base/run_loop.h"
 #include "base/time/time.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "third_party/WebKit/public/platform/WebSecurityOrigin.h"
 #include "third_party/WebKit/public/platform/WebString.h"
 #include "url/origin.h"
 
 namespace content {
 
 void CheckCastedOriginsAlreadyNormalized(
     const blink::WebSecurityOrigin& origin) {
   url::Origin checked_origin =
       url::Origin::UnsafelyCreateOriginWithoutNormalization(
           origin.protocol().utf8(), origin.host().utf8(),
           origin.effectivePort());
-  url::Origin non_checked_origin = url::Origin::CreateFromNormalizedTuple(
-      origin.protocol().utf8(), origin.host().utf8(), origin.effectivePort());
+  url::Origin non_checked_origin =
+      url::Origin::CreateFromNormalizedTupleWithSuborigin(
+          origin.protocol().utf8(), origin.host().utf8(),
+          origin.effectivePort(), origin.suborigin().utf8());
   EXPECT_EQ(checked_origin.scheme(), non_checked_origin.scheme());
   EXPECT_EQ(checked_origin.host(), non_checked_origin.host());
   EXPECT_EQ(checked_origin.port(), non_checked_origin.port());
 }
 
 TEST(BlinkPlatformTest, CastWebSecurityOrigin) {
   struct TestCase {
     const char* url;
     const char* scheme;
     const char* host;
@@ skipping 101 matching lines @@
     url::Origin url_origin = web_origin;
     EXPECT_TRUE(url_origin.unique());
     EXPECT_EQ("", url_origin.suborigin());
 
     web_origin = url::Origin(GURL(""));
     EXPECT_TRUE(web_origin.isUnique());
   }
 }
 
 // This test ensures that WebSecurityOrigins can safely use
-// url::Origin::CreateFromNormalizedTuple when doing conversions.
+// url::Origin::CreateFromNormalizedTupleWithSuborigin when doing conversions.
 TEST(BlinkPlatformTest, WebSecurityOriginNormalization) {
   struct TestCases {
     const char* url;
   } cases[] = {{""},
                {"javascript:alert(1)"},
                {"file://example.com:443/etc/passwd"},
                {"blob:https://example.com/uuid-goes-here"},
                {"filesystem:https://example.com/temporary/yay.png"},
                {"data"},
                {"blob:"},
@@ skipping 20 matching lines @@
   for (const auto& test : cases) {
     SCOPED_TRACE(testing::Message() << test.url);
     blink::WebSecurityOrigin web_origin =
         blink::WebSecurityOrigin::createFromString(
             blink::WebString::fromUTF8(test.url));
     CheckCastedOriginsAlreadyNormalized(web_origin);
   }
 }
 
 }  // namespace content