Use PdfCompositor for printing by default

Turn on using pdf compositor service for printing by default. Still keep
the feature flag as a kill switch for now.

BUG=824867

Change-Id: I5b7751c5ce453a7ffcda1bf38b5f141135f850f6
Reviewed-on: https://chromium-review.googlesource.com/1087872Reviewed-by: Lei Zhang <thestig@chromium.org>
Reviewed-by: Charlie Reis <creis@chromium.org>
Commit-Queue: Wei Li <weili@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565018}

chrome/browser/printing/print_browsertest.cc

@@ -525,21 +525,13 @@ IN_PROC_BROWSER_TEST_F(IsolateOriginsPrintBrowserTest, PrintIsolatedSubframe) {
 }
 
 // Printing preview a webpage.
-// Test that we won't use oopif printing by default, unless the
-// test is run with site-per-process flag enabled.
+// Test that we use oopif printing by default.
 IN_PROC_BROWSER_TEST_F(PrintBrowserTest, RegularPrinting) {
   ASSERT_TRUE(embedded_test_server()->Started());
   GURL url(embedded_test_server()->GetURL("/printing/test1.html"));
   ui_test_utils::NavigateToURL(browser(), url);
 
-  if (base::CommandLine::ForCurrentProcess()->HasSwitch(
-          switches::kSitePerProcess) ||
-      base::FeatureList::IsEnabled(
-          printing::features::kUsePdfCompositorServiceForPrint)) {
-    EXPECT_TRUE(IsOopifEnabled());
-  } else {
-    EXPECT_FALSE(IsOopifEnabled());
-  }
+  EXPECT_TRUE(IsOopifEnabled());
 }
 
 // Printing preview a webpage with isolate-origins enabled.

components/printing/browser/features.cc

@@ -8,7 +8,7 @@ namespace printing {
 namespace features {
 
 const base::Feature kUsePdfCompositorServiceForPrint{
-    "UsePdfCompositorServiceForPrint", base::FEATURE_DISABLED_BY_DEFAULT};
+    "UsePdfCompositorServiceForPrint", base::FEATURE_ENABLED_BY_DEFAULT};
 
 }  // namespace features
 }  // namespace printing