[Payment Request] Change the lifetime management of PaymentRequestImpl

chrome/browser/payments/site_per_process_payments_browsertest.cc

 // Copyright 2016 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 "base/command_line.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_tabstrip.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "chrome/test/base/in_process_browser_test.h"
 #include "chrome/test/base/ui_test_utils.h"
@@ skipping 14 matching lines @@
 class SitePerProcessPaymentsBrowserTest : public InProcessBrowserTest {
  public:
   SitePerProcessPaymentsBrowserTest() {}
   ~SitePerProcessPaymentsBrowserTest() override {}
 
   void SetUpCommandLine(base::CommandLine* command_line) override {
     InProcessBrowserTest::SetUpCommandLine(command_line);
     // HTTPS server only serves a valid cert for localhost, so this is needed
     // to load pages from other hosts without an error.
     command_line->AppendSwitch(switches::kIgnoreCertificateErrors);
-    command_line->AppendSwitch("enable-experimental-web-platform-features");
+    command_line->AppendSwitch(
+        switches::kEnableExperimentalWebPlatformFeatures);

[please use gerrit instead, 2017/01/09 14:50:13]

thx! :-)

[Mathieu, 2017/01/09 16:54:19]

Acknowledged.

     // Append --site-per-process flag.
     content::IsolateAllSitesForTesting(command_line);
   }
 
   void SetUpOnMainThread() override {
     https_server_.reset(
         new net::EmbeddedTestServer(net::EmbeddedTestServer::TYPE_HTTPS));
     host_resolver()->AddRule("*", "127.0.0.1");
     ASSERT_TRUE(https_server_->InitializeAndListen());
     content::SetupCrossSiteRedirector(https_server_.get());
@@ skipping 18 matching lines @@
       https_server_->GetURL("b.com", "/payment_request_iframe.html");
   EXPECT_TRUE(content::NavigateIframeToURL(tab, "test", iframe_url));
 
   EXPECT_TRUE(tab->GetRenderWidgetHostView()->IsShowing());
   content::RenderFrameHost* frame = ChildFrameAt(tab->GetMainFrame(), 0);
   EXPECT_TRUE(frame);
   EXPECT_NE(frame->GetSiteInstance(), tab->GetMainFrame()->GetSiteInstance());
 }
 
 }  // namespace payments