Add a "Simplify Page" option to the print preview dialog

chrome/browser/ui/webui/print_preview/print_preview_handler.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 "chrome/browser/ui/webui/print_preview/print_preview_handler.h"
 
 #include <ctype.h>
 
 #include <map>
 #include <string>
@@ skipping 12 matching lines @@
 #include "base/path_service.h"
 #include "base/prefs/pref_service.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/threading/thread.h"
 #include "base/threading/thread_restrictions.h"
 #include "base/values.h"
 #include "chrome/browser/app_mode/app_mode_utils.h"
 #include "chrome/browser/browser_process.h"
+#include "chrome/browser/dom_distiller/tab_utils.h"
 #include "chrome/browser/platform_util.h"
 #include "chrome/browser/printing/print_dialog_cloud.h"
 #include "chrome/browser/printing/print_error_dialog.h"
 #include "chrome/browser/printing/print_job_manager.h"
 #include "chrome/browser/printing/print_preview_dialog_controller.h"
 #include "chrome/browser/printing/print_view_manager.h"
 #include "chrome/browser/printing/printer_manager_dialog.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/signin/gaia_cookie_manager_service_factory.h"
 #include "chrome/browser/signin/profile_oauth2_token_service_factory.h"
 #include "chrome/browser/signin/signin_manager_factory.h"
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/browser_tabstrip.h"
 #include "chrome/browser/ui/chrome_select_file_policy.h"
 #include "chrome/browser/ui/scoped_tabbed_browser_displayer.h"
 #include "chrome/browser/ui/webui/print_preview/print_preview_ui.h"
 #include "chrome/browser/ui/webui/print_preview/printer_handler.h"
 #include "chrome/browser/ui/webui/print_preview/sticky_settings.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/cloud_print/cloud_print_cdd_conversion.h"
 #include "chrome/common/cloud_print/cloud_print_constants.h"
 #include "chrome/common/crash_keys.h"
 #include "chrome/common/pref_names.h"
 #include "components/cloud_devices/common/cloud_device_description.h"
 #include "components/cloud_devices/common/cloud_devices_urls.h"
 #include "components/cloud_devices/common/printer_description.h"
+#include "components/dom_distiller/core/url_utils.h"
 #include "components/printing/common/print_messages.h"
 #include "components/signin/core/browser/gaia_cookie_manager_service.h"
 #include "components/signin/core/browser/profile_oauth2_token_service.h"
 #include "components/signin/core/browser/signin_manager.h"
 #include "components/signin/core/common/profile_management_switches.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/navigation_entry.h"
 #include "content/public/browser/render_process_host.h"
@@ skipping 88 matching lines @@
 // Dictionary field to indicate whether Chrome is running in forced app (app
 // kiosk) mode. It's not the same as desktop Chrome kiosk (the one above).
 const char kAppKioskMode[] = "appKioskMode";
 // Dictionary field to store Cloud Print base URL.
 const char kCloudPrintUrl[] = "cloudPrintUrl";
 #if defined(OS_WIN)
 const char kHidePrintWithSystemDialogLink[] = "hidePrintWithSystemDialogLink";
 #endif
 // Name of a dictionary field holding the state of selection for document.
 const char kDocumentHasSelection[] = "documentHasSelection";
+// Dictionary field indicating whether to simplify the page.
+const char kPrintFriendlyEnabled[] = "printFriendlyEnabled";
 
 // Id of the predefined PDF printer.
 const char kLocalPdfPrinterId[] = "Save as PDF";
 
 // Additional printer capability setting keys.
 const char kPrinterId[] = "printerId";
 const char kPrinterCapabilities[] = "capabilities";
 
 // Get the print job settings dictionary from |args|. The caller takes
 // ownership of the returned DictionaryValue. Returns NULL on failure.
@@ skipping 638 matching lines @@
     success = args->GetBoolean(2, &preview_modifiable);
     DCHECK(success);
 
     if (draft_page_count != -1 && preview_modifiable &&
         print_preview_ui()->GetAvailableDraftPageCount() != draft_page_count) {
       settings->SetBoolean(printing::kSettingGenerateDraftData, true);
     }
   }
 
   VLOG(1) << "Print preview request start";
-  RenderViewHost* rvh = initiator->GetRenderViewHost();
-  rvh->Send(new PrintMsg_PrintPreview(rvh->GetRoutingID(), *settings));
+
+  bool print_friendly = false;
+  if (!settings->GetBoolean(printing::kSettingPrintFriendlyEnabled,
+                            &print_friendly)) {
+    NOTREACHED();
+  }
+
+  if (print_friendly) {
+    if (!hidden_print_preview_)
+      MaybeStartDistillation(initiator);

[nyquist, 2015/05/13 07:56:38]

Could this be simplified by using the new DistillAndView from
https://codereview.chromium.org/1058193002/ ?

Depending on how we deal with the @media print things, it might have to change a
little bit though to use the correct URL.

[arjunpatel, 2015/05/27 00:16:45]

Yes, depending on whether or not we keep the synchronous DOM Distiller UI, this
can use DistillAndView.

+
+      hidden_print_preview_.reset(
+        new HiddenPrintPreview(initiator, print_preview_ui()));
+      hidden_print_preview_->SetSettings(settings.Pass());
+  } else {
+    RenderViewHost* rvh = initiator->GetRenderViewHost();
+    rvh->Send(new PrintMsg_PrintPreview(rvh->GetRoutingID(), *settings));
+  }
 }
 
 void PrintPreviewHandler::HandlePrint(const base::ListValue* args) {
   ReportStats();
 
   // Record the number of times the user requests to regenerate preview data
   // before printing.
   UMA_HISTOGRAM_COUNTS("PrintPreview.RegeneratePreviewRequest.BeforePrint",
                        regenerate_preview_request_count_);
 
@@ skipping 392 matching lines @@
   initial_settings.SetBoolean(kAppKioskMode,
                               chrome::IsRunningInForcedAppMode());
 #if defined(OS_WIN)
   // In Win8 metro, the system print dialog can only open on the desktop.  Doing
   // so will cause the browser to appear hung, so we don't show the link in
   // metro.
   bool is_ash = (chrome::GetActiveDesktop() == chrome::HOST_DESKTOP_TYPE_ASH);
   initial_settings.SetBoolean(kHidePrintWithSystemDialogLink, is_ash);
 #endif
 
+  WebContents* initiator = GetInitiator();
+  bool can_simplify = false;
+  if (initiator) {
+      can_simplify = cmdline->HasSwitch(switches::kEnableDomDistiller) &&

[nyquist, 2015/05/13 07:56:38]

Should this also check to see if we assume that the distillation will work?
See dom_distiller::IsDistillablePage (
https://code.google.com/p/chromium/codesearch#chromium/src/components/dom_dis...
)

[arjunpatel, 2015/05/27 17:10:11]

The latest version of the CL does this, showing the checkbox asynchronously as
the dialog appears. It seems like many pages are not marked as distillable even
using the AdaBoost algorithm, but we've been testing with the alwaystrue
heuristic.

+         dom_distiller::url_utils::IsUrlDistillable(
+            initiator->GetLastCommittedURL());
+    }
+  initial_settings.SetBoolean(kPrintFriendlyEnabled, can_simplify);
+
   if (print_preview_ui()->source_is_modifiable())
     GetNumberFormatAndMeasurementSystem(&initial_settings);
   web_ui()->CallJavascriptFunction("setInitialSettings", initial_settings);
 }
 
 void PrintPreviewHandler::ClosePreviewDialog() {
   print_preview_ui()->OnClosePrintPreviewDialog();
 }
 
 void PrintPreviewHandler::SendAccessToken(const std::string& type,
@@ skipping 486 matching lines @@
 
 void PrintPreviewHandler::UnregisterForGaiaCookieChanges() {
   if (gaia_cookie_manager_service_)
     gaia_cookie_manager_service_->RemoveObserver(this);
 }
 
 void PrintPreviewHandler::SetPdfSavedClosureForTesting(
     const base::Closure& closure) {
   pdf_file_saved_closure_ = closure;
 }