ABANDONED: OOPIFs: Moving stitching of local paths from renderer to browser process.

chrome/browser/download/save_page_browsertest.cc

 // Copyright (c) 2012 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/bind.h"
 #include "base/bind_helpers.h"
 #include "base/command_line.h"
 #include "base/files/file_path.h"
 #include "base/files/file_util.h"
 #include "base/files/scoped_temp_dir.h"
@@ skipping 783 matching lines @@
   EXPECT_TRUE(base::PathExists(full_file_name));
 }
 
 class SavePageSitePerProcessBrowserTest : public SavePageBrowserTest {
  public:
   SavePageSitePerProcessBrowserTest() {}
 
  protected:
   void SetUpCommandLine(base::CommandLine* command_line) override {
     SavePageBrowserTest::SetUpCommandLine(command_line);
-
-    // TODO(lukasza): Enable --site-per-process once crbug.com/526786 is fixed.
-    // (currently, when the line below is uncommented out, the test crashes
-    // under blink::WebLocalFrameImpl::fromFrameOwnerElement called from
-    // blink::WebPageSerializerImpl::openTagToString).
-    //
-    // content::IsolateAllSitesForTesting(command_line);
+    content::IsolateAllSitesForTesting(command_line);
   }
 
   void SetUpOnMainThread() override {
     SavePageBrowserTest::SetUpOnMainThread();
 
     host_resolver()->AddRule("*", "127.0.0.1");
     ASSERT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady());
     content::SetupCrossSiteRedirector(embedded_test_server());
   }
 
  private:
   DISALLOW_COPY_AND_ASSIGN(SavePageSitePerProcessBrowserTest);
 };
 
 // Test for crbug.com/526786.  Without OOPIFs fixes, the test will trigger
 // a crash in the renderer process.
 IN_PROC_BROWSER_TEST_F(SavePageSitePerProcessBrowserTest, SaveCrossSitePage) {
-  // TODO(lukasza): Remove this check once crbug.com/526786 is fixed.
-  if (content::AreAllSitesIsolatedForTesting()) {
-    LOG(WARNING) << "Skipping the test.";
-    return;  // Avoid failing on Site Isolation FYI bot.
-  }
-
   GURL url(embedded_test_server()->GetURL(
       "a.com", "/frame_tree/page_with_two_frames_remote_and_local.html"));
   ui_test_utils::NavigateToURL(browser(), url);
 
   base::FilePath full_file_name, dir;
   SaveCurrentTab(url, content::SAVE_PAGE_TYPE_AS_COMPLETE_HTML, "xsite1", 3,
                  &dir, &full_file_name);
   ASSERT_FALSE(HasFailure());
 
   EXPECT_TRUE(base::DirectoryExists(dir));
   base::FilePath expected_files[] = {
       full_file_name, dir.AppendASCII("title1.html"),
       dir.AppendASCII("title1(1).html"),
   };
   for (auto file_path : expected_files) {
     EXPECT_TRUE(base::PathExists(file_path)) << "Does " << file_path.value()
                                              << " exist?";
     int64 actual_file_size = 0;
     EXPECT_TRUE(base::GetFileSize(file_path, &actual_file_size));
     EXPECT_NE(0, actual_file_size) << "Is " << file_path.value()
                                    << " non-empty?";
   }
 }
 
 }  // namespace