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

Unified Diff: chrome/browser/ui/views/extensions/extension_dialog.cc

Issue 8497008: Implement Bookmark All Tabs Dialog with WebUI. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: review fix 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/ui/views/extensions/extension_dialog.cc
diff --git a/chrome/browser/ui/views/extensions/extension_dialog.cc b/chrome/browser/ui/views/extensions/extension_dialog.cc
index c2bc0b60e090f673eb0bba3f50be5093f3826a99..fd184048d7d41ec590b4dfd7f456eb42721ca450 100644
--- a/chrome/browser/ui/views/extensions/extension_dialog.cc
+++ b/chrome/browser/ui/views/extensions/extension_dialog.cc
@@ -63,6 +63,7 @@ ExtensionDialog* ExtensionDialog::Show(
TabContents* tab_contents,
int width,
int height,
+ const string16& title,
ExtensionDialogObserver* observer) {
CHECK(browser);
ExtensionHost* host = CreateExtensionHost(url, browser);
@@ -71,6 +72,7 @@ ExtensionDialog* ExtensionDialog::Show(
host->set_associated_tab_contents(tab_contents);
ExtensionDialog* dialog = new ExtensionDialog(host, observer);
+ dialog->set_title(title);
dialog->InitWindow(browser, width, height);
// Ensure the DOM JavaScript can respond immediately to keyboard shortcuts.
host->render_view_host()->view()->Focus();
@@ -129,7 +131,11 @@ bool ExtensionDialog::IsModal() const {
}
bool ExtensionDialog::ShouldShowWindowTitle() const {
- return false;
+ return !window_title_.empty();
+}
+
+string16 ExtensionDialog::GetWindowTitle() const {
+ return window_title_;
}
void ExtensionDialog::DeleteDelegate() {
« no previous file with comments | « chrome/browser/ui/views/extensions/extension_dialog.h ('k') | chrome/browser/ui/views/select_file_dialog_extension.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698