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

Unified Diff: chrome/browser/printing/print_view_manager.cc

Issue 8136027: Print Preview: Make print preview tab modal. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: rebase, no renderer changes Created 9 years, 1 month 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/printing/print_view_manager.cc
===================================================================
--- chrome/browser/printing/print_view_manager.cc (revision 108195)
+++ chrome/browser/printing/print_view_manager.cc (working copy)
@@ -32,13 +32,6 @@
namespace {
-string16 GenerateRenderSourceName(TabContents* tab_contents) {
- string16 name(tab_contents->GetTitle());
- if (name.empty())
- name = l10n_util::GetStringUTF16(IDS_DEFAULT_PRINT_DOCUMENT_TITLE);
- return name;
-}
-
// Release the PrinterQuery identified by |cookie|.
void ReleasePrinterQuery(int cookie) {
printing::PrintJobManager* print_job_manager =
@@ -66,7 +59,6 @@
number_pages_(0),
printing_succeeded_(false),
inside_inner_message_loop_(false),
- is_title_overridden_(false),
observer_(NULL),
cookie_(0) {
#if defined(OS_POSIX) && !defined(OS_MACOSX)
@@ -125,10 +117,6 @@
observer_ = observer;
}
-void PrintViewManager::ResetTitleOverride() {
- is_title_overridden_ = false;
-}
-
void PrintViewManager::StopNavigation() {
// Cancel the current job, wait for the worker to finish.
TerminatePrintJob(true);
@@ -147,15 +135,11 @@
}
}
-void PrintViewManager::OverrideTitle(TabContents* tab_contents) {
- is_title_overridden_ = true;
- overridden_title_ = GenerateRenderSourceName(tab_contents);
-}
-
string16 PrintViewManager::RenderSourceName() {
- if (is_title_overridden_)
- return overridden_title_;
- return GenerateRenderSourceName(tab_contents());
+ string16 name(tab_contents()->GetTitle());
+ if (name.empty())
+ name = l10n_util::GetStringUTF16(IDS_DEFAULT_PRINT_DOCUMENT_TITLE);
+ return name;
}
void PrintViewManager::OnDidGetPrintedPagesCount(int cookie, int number_pages) {

Powered by Google App Engine
This is Rietveld 408576698