Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1749)

Unified Diff: chrome/browser/ui/webui/print_preview/print_preview_handler.h

Issue 2376193006: Disable Ctrl+Shift+P on Chrome OS (Closed)
Patch Set: Fix test. Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/webui/print_preview/print_preview_handler.h
diff --git a/chrome/browser/ui/webui/print_preview/print_preview_handler.h b/chrome/browser/ui/webui/print_preview/print_preview_handler.h
index 09d3ab82fc25feb9b6d164af8860b2a2f769a1a8..b8ebd9b81ddb0bf1873b568031bf7b5dfd4c4526 100644
--- a/chrome/browser/ui/webui/print_preview/print_preview_handler.h
+++ b/chrome/browser/ui/webui/print_preview/print_preview_handler.h
@@ -149,11 +149,11 @@ class PrintPreviewHandler
// Gets the printer capabilities. First element of |args| is the printer name.
void HandleGetPrinterCapabilities(const base::ListValue* args);
-#if defined(ENABLE_BASIC_PRINTING)
+#if defined(ENABLE_BASIC_PRINTING) && !defined(OS_CHROMEOS)
// Asks the initiator renderer to show the native print system dialog. |args|
// is unused.
void HandleShowSystemDialog(const base::ListValue* args);
-#endif // defined(ENABLE_BASIC_PRINTING)
+#endif // defined(ENABLE_BASIC_PRINTING) && !defined(OS_CHROMEOS)
Lei Zhang 2016/10/03 23:03:58 I would just omit the comment since the #if and #e
skau 2016/10/03 23:32:13 Done.
// Callback for the signin dialog to call once signin is complete.
void OnSigninComplete();

Powered by Google App Engine
This is Rietveld 408576698