Disabled "Print" option in wrench menu for a crashed page.

chrome/browser/ui/browser_commands.cc

 // Copyright 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/browser_commands.h"
 
 #include "base/command_line.h"
 #include "base/metrics/histogram.h"
 #include "base/strings/utf_string_conversions.h"
 #include "build/build_config.h"
@@ skipping 859 matching lines @@
 #if defined(ENABLE_PRINTING)
   printing::StartPrint(
       browser->tab_strip_model()->GetActiveWebContents(),
       browser->profile()->GetPrefs()->GetBoolean(prefs::kPrintPreviewDisabled),
       false);
 #endif  // defined(ENABLE_PRINTING)
 }
 
 bool CanPrint(Browser* browser) {
   // Do not print when printing is disabled via pref or policy.
+  // Do not print when a page has crashed.
   // Do not print when a constrained window is showing. It's confusing.
   // TODO(gbillock): Need to re-assess the call to
   // IsShowingWebContentsModalDialog after a popup management policy is
   // refined -- we will probably want to just queue the print request, not
   // block it.
+  WebContents* current_tab = browser->tab_strip_model()->GetActiveWebContents();
   return browser->profile()->GetPrefs()->GetBoolean(prefs::kPrintingEnabled) &&
+      (current_tab != nullptr && !current_tab->IsCrashed()) &&

[Peter Kasting, 2016/03/29 02:55:13]

Nit: "!= nullptr" usually gets omitted

[Tom (Use chromium acct), 2016/03/31 17:13:44]

Done.

       !(IsShowingWebContentsModalDialog(browser) ||
       GetContentRestrictions(browser) & CONTENT_RESTRICTION_PRINT);

[Peter Kasting, 2016/03/29 02:55:13]

Nit: While here: Seems like this line should be indented more

[Tom (Use chromium acct), 2016/03/31 17:13:44]

Done.

 }
 
 #if defined(ENABLE_BASIC_PRINTING)
 void BasicPrint(Browser* browser) {
   printing::StartBasicPrint(browser->tab_strip_model()->GetActiveWebContents());
 }
 
 bool CanBasicPrint(Browser* browser) {
   // If printing is not disabled via pref or policy, it is always possible to
   // advanced print when the print preview is visible.
@@ skipping 385 matching lines @@
       app_name, true /* trusted_source */, gfx::Rect(), browser->profile()));
   app_browser->tab_strip_model()->AppendWebContents(contents, true);
 
   contents->GetMutableRendererPrefs()->can_accept_load_drops = false;
   contents->GetRenderViewHost()->SyncRendererPrefs();
   app_browser->window()->Show();
 }
 #endif  // defined(ENABLE_EXTENSIONS)
 
 }  // namespace chrome