Print preview: Disable the right context menu items in print preview.

chrome/browser/printing/print_preview_context_menu_observer.cc

Index: chrome/browser/printing/print_preview_context_menu_observer.cc
===================================================================
--- chrome/browser/printing/print_preview_context_menu_observer.cc	(revision 0)
+++ chrome/browser/printing/print_preview_context_menu_observer.cc	(revision 0)
@@ -0,0 +1,53 @@
+// Copyright (c) 2011 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/printing/print_preview_context_menu_observer.h"
+
+#include "base/logging.h"
+#include "chrome/app/chrome_command_ids.h"
+#include "chrome/browser/printing/print_preview_tab_controller.h"
+
+PrintPreviewContextMenuObserver::PrintPreviewContextMenuObserver(
+    TabContentsWrapper* tab) : tab_(tab) {
+}
+
+PrintPreviewContextMenuObserver::~PrintPreviewContextMenuObserver() {
+}
+
+bool PrintPreviewContextMenuObserver::IsPrintPreviewTab() {
+  printing::PrintPreviewTabController* controller =
+      printing::PrintPreviewTabController::GetInstance();
+  if (!controller)
+    return false;
+  return controller->GetPrintPreviewForTab(tab_);
+}
+
+bool PrintPreviewContextMenuObserver::IsCommandIdSupported(int command_id) {
+  switch (command_id) {
+    case IDC_PRINT:
+    case IDC_VIEW_SOURCE:
+    case IDC_CONTENT_CONTEXT_VIEWFRAMESOURCE:
+    case IDC_CONTENT_CONTEXT_VIEWPAGEINFO:
+    case IDC_CONTENT_CONTEXT_SEARCHWEBFOR:
+      return true;

[vandebo (ex-Chrome), 2011/12/10 02:18:22]

I think IdSupported is the one that should return false for non print preview
tabs, otherwise other observers or the base class can't alter these behaviors.

[Lei Zhang, 2011/12/10 02:31:33]

Done.

+
+    default:
+      return false;
+  }
+}
+
+bool PrintPreviewContextMenuObserver::IsCommandIdEnabled(int command_id) {
+  switch (command_id) {
+    case IDC_PRINT:
+    case IDC_VIEW_SOURCE:
+    case IDC_CONTENT_CONTEXT_VIEWFRAMESOURCE:
+    case IDC_CONTENT_CONTEXT_VIEWPAGEINFO:
+    case IDC_CONTENT_CONTEXT_SEARCHWEBFOR:
+      return !IsPrintPreviewTab();
+
+    default:
+      NOTREACHED();
+      return true;
+  }
+}
Property changes on: chrome/browser/printing/print_preview_context_menu_observer.cc
___________________________________________________________________
Added: svn:eol-style
   + LF