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

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

Issue 17500003: Close web contents modal dialogs on content load start (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Bonus patchset: initiator tab -> initiator Created 7 years, 6 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/printing/background_printing_manager.cc
diff --git a/chrome/browser/printing/background_printing_manager.cc b/chrome/browser/printing/background_printing_manager.cc
index c97f89ad0f27572f30cbda3ede0e456925565693..803a316150171630842f85bc2c09735a4b885b3d 100644
--- a/chrome/browser/printing/background_printing_manager.cc
+++ b/chrome/browser/printing/background_printing_manager.cc
@@ -63,16 +63,16 @@ void BackgroundPrintingManager::OwnPrintPreviewDialog(
rph_source);
}
- // Activate the initiator tab.
+ // Activate the initiator.
PrintPreviewDialogController* dialog_controller =
PrintPreviewDialogController::GetInstance();
if (!dialog_controller)
return;
- WebContents* initiator_tab =
- dialog_controller->GetInitiatorTab(preview_dialog);
- if (!initiator_tab)
+ WebContents* initiator =
+ dialog_controller->GetInitiator(preview_dialog);
Lei Zhang 2013/06/29 00:24:18 nit: fits on the previous line.
Mike Wittman 2013/07/01 17:16:12 Done.
+ if (!initiator)
return;
- initiator_tab->GetDelegate()->ActivateContents(initiator_tab);
+ initiator->GetDelegate()->ActivateContents(initiator);
}
void BackgroundPrintingManager::Observe(

Powered by Google App Engine
This is Rietveld 408576698