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

Unified Diff: content/browser/tab_contents/render_view_host_manager.cc

Issue 7327007: Moving notification types which are chrome specific to a new header file chrome_notification_type... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 9 years, 5 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: content/browser/tab_contents/render_view_host_manager.cc
===================================================================
--- content/browser/tab_contents/render_view_host_manager.cc (revision 91968)
+++ content/browser/tab_contents/render_view_host_manager.cc (working copy)
@@ -20,7 +20,7 @@
#include "content/common/content_client.h"
#include "content/common/content_switches.h"
#include "content/common/notification_service.h"
-#include "content/common/notification_type.h"
+#include "content/common/content_notification_types.h"
#include "content/common/url_constants.h"
#include "content/common/view_messages.h"
@@ -69,7 +69,7 @@
GetControllerForRenderManager().session_storage_namespace());
// Keep track of renderer processes as they start to shut down.
- registrar_.Add(this, NotificationType::RENDERER_PROCESS_CLOSING,
+ registrar_.Add(this, content::NOTIFICATION_RENDERER_PROCESS_CLOSING,
NotificationService::AllSources());
}
@@ -114,7 +114,7 @@
details.new_host = render_view_host_;
details.old_host = NULL;
NotificationService::current()->Notify(
- NotificationType::RENDER_VIEW_HOST_CHANGED,
+ content::NOTIFICATION_RENDER_VIEW_HOST_CHANGED,
Source<NavigationController>(
&delegate_->GetControllerForRenderManager()),
Details<RenderViewHostSwitchedDetails>(&details));
@@ -314,11 +314,11 @@
CancelPending();
}
-void RenderViewHostManager::Observe(NotificationType type,
+void RenderViewHostManager::Observe(int type,
const NotificationSource& source,
const NotificationDetails& details) {
- switch (type.value) {
- case NotificationType::RENDERER_PROCESS_CLOSING:
+ switch (type) {
+ case content::NOTIFICATION_RENDERER_PROCESS_CLOSING:
RendererProcessClosing(Source<RenderProcessHost>(source).ptr());
break;
@@ -621,7 +621,7 @@
details.new_host = render_view_host_;
details.old_host = old_render_view_host;
NotificationService::current()->Notify(
- NotificationType::RENDER_VIEW_HOST_CHANGED,
+ content::NOTIFICATION_RENDER_VIEW_HOST_CHANGED,
Source<NavigationController>(&delegate_->GetControllerForRenderManager()),
Details<RenderViewHostSwitchedDetails>(&details));
@@ -842,7 +842,7 @@
details.new_host = render_view_host_;
details.old_host = old_render_view_host;
NotificationService::current()->Notify(
- NotificationType::RENDER_VIEW_HOST_CHANGED,
+ content::NOTIFICATION_RENDER_VIEW_HOST_CHANGED,
Source<NavigationController>(&delegate_->GetControllerForRenderManager()),
Details<RenderViewHostSwitchedDetails>(&details));

Powered by Google App Engine
This is Rietveld 408576698