Revert 272587 "Remove kEnablePrintPreview now that the pdf plugi..."

trunk/src/chrome/browser/apps/app_browsertest.cc

 // Copyright 2013 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 "apps/app_window.h"
 #include "apps/app_window_registry.h"
 #include "apps/common/api/app_runtime.h"
 #include "apps/launcher.h"
 #include "apps/ui/native_app_window.h"
 #include "base/bind.h"
@@ skipping 1083 matching lines @@
   LoadAndLaunchPlatformApp("minimal");
   ASSERT_TRUE(launched_listener.WaitUntilSatisfied());
 
   EXPECT_EQ(1LU, GetAppWindowCount());
   EXPECT_TRUE(GetFirstAppWindow()
                   ->web_contents()
                   ->GetRenderWidgetHostView()
                   ->HasFocus());
 }
 
-#if defined(OS_WIN) || defined(OS_LINUX)
+// The next three tests will only run automatically with Chrome branded builds
+// because they require the PDF preview plug-in. To run these tests manually for
+// Chromium (non-Chrome branded) builds in a development environment:
+//
+//   1) Remove "MAYBE_" in the first line of each test definition
+//   2) Build Chromium browser_tests
+//   3) Make a copy of the PDF plug-in from a recent version of Chrome (Canary
+//      or a recent development build) to your Chromium build:
+//      - On Linux and Chrome OS, copy /opt/google/chrome/libpdf.so to
+//        <path-to-your-src>/out/Debug
+//      - On OS X, copy PDF.plugin from
+//        <recent-chrome-app-folder>/*/*/*/*/"Internet Plug-Ins" to
+//        <path-to-your-src>/out/Debug/Chromium.app/*/*/*/*/"Internet Plug-Ins"
+//   4) Run browser_tests with the --enable-print-preview flag
+
+#if !defined(GOOGLE_CHROME_BUILD) || \
+    (defined(GOOGLE_CHROME_BUILD) && (defined(OS_WIN) || defined(OS_LINUX)))
 #define MAYBE_WindowDotPrintShouldBringUpPrintPreview \
     DISABLED_WindowDotPrintShouldBringUpPrintPreview
 #else
 #define MAYBE_WindowDotPrintShouldBringUpPrintPreview \
     WindowDotPrintShouldBringUpPrintPreview
 #endif
 
 IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest,
                        MAYBE_WindowDotPrintShouldBringUpPrintPreview) {
   ScopedPreviewTestingDelegate preview_delegate(true);
   ASSERT_TRUE(RunPlatformAppTest("platform_apps/print_api")) << message_;
   preview_delegate.WaitUntilPreviewIsReady();
 }
 
+#if !defined(GOOGLE_CHROME_BUILD)
+#define MAYBE_ClosingWindowWhilePrintingShouldNotCrash \
+    DISABLED_ClosingWindowWhilePrintingShouldNotCrash
+#else
+#define MAYBE_ClosingWindowWhilePrintingShouldNotCrash \
+    ClosingWindowWhilePrintingShouldNotCrash
+#endif
+
 // This test verifies that http://crbug.com/297179 is fixed.
 IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest,
-                       ClosingWindowWhilePrintingShouldNotCrash) {
+                       MAYBE_ClosingWindowWhilePrintingShouldNotCrash) {
   ScopedPreviewTestingDelegate preview_delegate(false);
   ASSERT_TRUE(RunPlatformAppTest("platform_apps/print_api")) << message_;
   preview_delegate.WaitUntilPreviewIsReady();
   GetFirstAppWindow()->GetBaseWindow()->Close();
 }
 
 // This test currently only passes on OS X (on other platforms the print preview
 // dialog's size is limited by the size of the window being printed).
-#if !defined(OS_MACOSX)
+#if !defined(GOOGLE_CHROME_BUILD) || !defined(OS_MACOSX)
 #define MAYBE_PrintPreviewShouldNotBeTooSmall \
     DISABLED_PrintPreviewShouldNotBeTooSmall
 #else
 #define MAYBE_PrintPreviewShouldNotBeTooSmall \
     PrintPreviewShouldNotBeTooSmall
 #endif
 
 IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest,
                        MAYBE_PrintPreviewShouldNotBeTooSmall) {
   // Print preview dialogs with widths less than 410 pixels will have preview
@@ skipping 175 matching lines @@
 
     ExtensionApiTest::ResultCatcher result_catcher;
 
     ASSERT_TRUE(launched_listener.WaitUntilSatisfied());
 
     EXPECT_TRUE(result_catcher.GetNextResult());
   }
 }
 
 }  // namespace extensions