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

Unified Diff: content/browser/renderer_host/render_message_filter.cc

Issue 9845033: Move the MediaObserver getter from ResourceContext to ContentBrowserClient, since we only need to s… (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: fix dcheck Created 8 years, 9 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/renderer_host/render_message_filter.cc
===================================================================
--- content/browser/renderer_host/render_message_filter.cc (revision 128803)
+++ content/browser/renderer_host/render_message_filter.cc (working copy)
@@ -269,7 +269,8 @@
PluginServiceImpl* plugin_service,
content::BrowserContext* browser_context,
net::URLRequestContextGetter* request_context,
- RenderWidgetHelper* render_widget_helper)
+ RenderWidgetHelper* render_widget_helper,
+ content::MediaObserver* media_observer)
: resource_dispatcher_host_(ResourceDispatcherHostImpl::Get()),
plugin_service_(plugin_service),
browser_context_(browser_context),
@@ -280,7 +281,8 @@
dom_storage_context_(static_cast<DOMStorageContextImpl*>(
BrowserContext::GetDOMStorageContext(browser_context))),
render_process_id_(render_process_id),
- cpu_usage_(0) {
+ cpu_usage_(0),
+ media_observer_(media_observer) {
DCHECK(request_context_);
render_widget_helper_->Init(render_process_id_, resource_dispatcher_host_);
@@ -881,10 +883,8 @@
}
void RenderMessageFilter::OnMediaLogEvent(const media::MediaLogEvent& event) {
- if (!resource_context_->GetMediaObserver())
- return;
- resource_context_->GetMediaObserver()->OnMediaEvent(
- render_process_id_, event);
+ if (media_observer_)
+ media_observer_->OnMediaEvent(render_process_id_, event);
}
void RenderMessageFilter::CheckPolicyForCookies(
« no previous file with comments | « content/browser/renderer_host/render_message_filter.h ('k') | content/browser/renderer_host/render_process_host_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698