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

Unified Diff: chrome/test/base/test_html_dialog_observer.cc

Issue 10262027: Rename HTML_DIALOG to WEB_DIALOG. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 8 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
« no previous file with comments | « chrome/common/chrome_notification_types.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/base/test_html_dialog_observer.cc
diff --git a/chrome/test/base/test_html_dialog_observer.cc b/chrome/test/base/test_html_dialog_observer.cc
index 47bfdeda4d7482ff8e38a6f7867bd68177ff0621..6ef8ecb95779f82e8c5540079e0bc55434b16631 100644
--- a/chrome/test/base/test_html_dialog_observer.cc
+++ b/chrome/test/base/test_html_dialog_observer.cc
@@ -20,8 +20,10 @@ using content::NavigationController;
TestHtmlDialogObserver::TestHtmlDialogObserver(
JsInjectionReadyObserver* js_injection_ready_observer)
: js_injection_ready_observer_(js_injection_ready_observer),
- web_ui_(NULL), done_(false), running_(false) {
- registrar_.Add(this, chrome::NOTIFICATION_HTML_DIALOG_SHOWN,
+ web_ui_(NULL),
+ done_(false),
+ running_(false) {
+ registrar_.Add(this, chrome::NOTIFICATION_WEB_DIALOG_SHOWN,
content::NotificationService::AllSources());
}
@@ -33,13 +35,13 @@ void TestHtmlDialogObserver::Observe(
const content::NotificationSource& source,
const content::NotificationDetails& details) {
switch (type) {
- case chrome::NOTIFICATION_HTML_DIALOG_SHOWN:
+ case chrome::NOTIFICATION_WEB_DIALOG_SHOWN:
if (js_injection_ready_observer_) {
js_injection_ready_observer_->OnJsInjectionReady(
content::Details<content::RenderViewHost>(details).ptr());
}
web_ui_ = content::Source<content::WebUI>(source).ptr();
- registrar_.Remove(this, chrome::NOTIFICATION_HTML_DIALOG_SHOWN,
+ registrar_.Remove(this, chrome::NOTIFICATION_WEB_DIALOG_SHOWN,
content::NotificationService::AllSources());
// Wait for navigation on the new WebUI instance to complete. This depends
// on receiving the notification of the HtmlDialog being shown before the
« no previous file with comments | « chrome/common/chrome_notification_types.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698