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

Side by Side Diff: chrome/browser/printing/print_preview_message_handler.cc

Issue 12250008: Cleanup: Remove more uses of deprecated PrintPreviewDialogController methods. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: fix test Created 7 years, 10 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/printing/print_preview_message_handler.h" 5 #include "chrome/browser/printing/print_preview_message_handler.h"
6 6
7 #include <vector> 7 #include <vector>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/memory/ref_counted.h" 10 #include "base/memory/ref_counted.h"
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
66 66
67 PrintPreviewMessageHandler::PrintPreviewMessageHandler( 67 PrintPreviewMessageHandler::PrintPreviewMessageHandler(
68 WebContents* web_contents) 68 WebContents* web_contents)
69 : content::WebContentsObserver(web_contents) { 69 : content::WebContentsObserver(web_contents) {
70 DCHECK(web_contents); 70 DCHECK(web_contents);
71 } 71 }
72 72
73 PrintPreviewMessageHandler::~PrintPreviewMessageHandler() { 73 PrintPreviewMessageHandler::~PrintPreviewMessageHandler() {
74 } 74 }
75 75
76 WebContents* PrintPreviewMessageHandler::GetPrintPreviewTab() { 76 WebContents* PrintPreviewMessageHandler::GetPrintPreviewDialog() {
77 PrintPreviewDialogController* tab_controller = 77 PrintPreviewDialogController* dialog_controller =
78 PrintPreviewDialogController::GetInstance(); 78 PrintPreviewDialogController::GetInstance();
79 if (!tab_controller) 79 if (!dialog_controller)
80 return NULL; 80 return NULL;
81 81 return dialog_controller->GetPrintPreviewForContents(web_contents());
82 return tab_controller->GetPrintPreviewForTab(web_contents());
83 } 82 }
84 83
85 PrintPreviewUI* PrintPreviewMessageHandler::GetPrintPreviewUI() { 84 PrintPreviewUI* PrintPreviewMessageHandler::GetPrintPreviewUI() {
86 WebContents* tab = GetPrintPreviewTab(); 85 WebContents* dialog = GetPrintPreviewDialog();
87 if (!tab || !tab->GetWebUI()) 86 if (!dialog || !dialog->GetWebUI())
88 return NULL; 87 return NULL;
89 return static_cast<PrintPreviewUI*>(tab->GetWebUI()->GetController()); 88 return static_cast<PrintPreviewUI*>(dialog->GetWebUI()->GetController());
90 } 89 }
91 90
92 void PrintPreviewMessageHandler::OnRequestPrintPreview( 91 void PrintPreviewMessageHandler::OnRequestPrintPreview(
93 const PrintHostMsg_RequestPrintPreview_Params& params) { 92 const PrintHostMsg_RequestPrintPreview_Params& params) {
94 if (params.webnode_only) { 93 if (params.webnode_only) {
95 printing::PrintViewManager::FromWebContents(web_contents())-> 94 printing::PrintViewManager::FromWebContents(web_contents())->
96 PrintPreviewForWebNode(); 95 PrintPreviewForWebNode();
97 } 96 }
98 PrintPreviewDialogController::PrintPreview(web_contents()); 97 PrintPreviewDialogController::PrintPreview(web_contents());
99 PrintPreviewUI::SetInitialParams(GetPrintPreviewTab(), params); 98 PrintPreviewUI::SetInitialParams(GetPrintPreviewDialog(), params);
100 } 99 }
101 100
102 void PrintPreviewMessageHandler::OnDidGetPreviewPageCount( 101 void PrintPreviewMessageHandler::OnDidGetPreviewPageCount(
103 const PrintHostMsg_DidGetPreviewPageCount_Params& params) { 102 const PrintHostMsg_DidGetPreviewPageCount_Params& params) {
104 if (params.page_count <= 0) { 103 if (params.page_count <= 0) {
105 NOTREACHED(); 104 NOTREACHED();
106 return; 105 return;
107 } 106 }
108 107
109 PrintPreviewUI* print_preview_ui = GetPrintPreviewUI(); 108 PrintPreviewUI* print_preview_ui = GetPrintPreviewUI();
(...skipping 126 matching lines...) Expand 10 before | Expand all | Expand 10 after
236 IPC_MESSAGE_HANDLER(PrintHostMsg_PrintPreviewInvalidPrinterSettings, 235 IPC_MESSAGE_HANDLER(PrintHostMsg_PrintPreviewInvalidPrinterSettings,
237 OnInvalidPrinterSettings) 236 OnInvalidPrinterSettings)
238 IPC_MESSAGE_HANDLER(PrintHostMsg_PrintPreviewScalingDisabled, 237 IPC_MESSAGE_HANDLER(PrintHostMsg_PrintPreviewScalingDisabled,
239 OnPrintPreviewScalingDisabled) 238 OnPrintPreviewScalingDisabled)
240 IPC_MESSAGE_UNHANDLED(handled = false) 239 IPC_MESSAGE_UNHANDLED(handled = false)
241 IPC_END_MESSAGE_MAP() 240 IPC_END_MESSAGE_MAP()
242 return handled; 241 return handled;
243 } 242 }
244 243
245 } // namespace printing 244 } // namespace printing
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698