Implements AudioMessageFilter as member in RenderThread

content/renderer/render_view.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/render_view.h"
 
 #include <algorithm>
 #include <cmath>
 #include <string>
 #include <vector>
@@ skipping 386 matching lines @@
           webkit_glue::kForegroundTabTimerInterval);
 
   OnSetRendererPrefs(renderer_prefs);
 
   host_window_ = parent_hwnd;
 
   const CommandLine& command_line = *CommandLine::ForCurrentProcess();
   if (command_line.HasSwitch(switches::kEnableAccessibility))
     WebAccessibilityCache::enableAccessibility();
 
-  audio_message_filter_ = new AudioMessageFilter(routing_id_);
-  render_thread_->AddFilter(audio_message_filter_);
-
 #if defined(ENABLE_P2P_APIS)
   if (CommandLine::ForCurrentProcess()->HasSwitch(switches::kEnableP2PApi))
     p2p_socket_dispatcher_ = new P2PSocketDispatcher(this);
 #endif
 
   content::GetContentClient()->renderer()->RenderViewCreated(this);
 }
 
 RenderView::~RenderView() {
   if (decrement_shared_popup_at_destruction_)
@@ skipping 11 matching lines @@
 #if defined(OS_MACOSX)
   // Destroy all fake plugin window handles on the browser side.
   while (!fake_plugin_window_handles_.empty()) {
     // Make sure no NULL plugin window handles were inserted into this list.
     DCHECK(*fake_plugin_window_handles_.begin());
     // DestroyFakePluginWindowHandle modifies fake_plugin_window_handles_.
     DestroyFakePluginWindowHandle(*fake_plugin_window_handles_.begin());
   }
 #endif
 
-  render_thread_->RemoveFilter(audio_message_filter_);
-
 #ifndef NDEBUG
   // Make sure we are no longer referenced by the ViewMap.
   ViewMap* views = g_view_map.Pointer();
   for (ViewMap::iterator it = views->begin(); it != views->end(); ++it)
     DCHECK_NE(this, it->second) << "Failed to call Close?";
 #endif
 
   FOR_EACH_OBSERVER(RenderViewObserver, observers_, set_render_view(NULL));
   FOR_EACH_OBSERVER(RenderViewObserver, observers_, OnDestruct());
 }
@@ skipping 894 matching lines @@
     return WebStorageNamespace::createSessionStorageNamespace(quota);
   CHECK(session_storage_namespace_id_ != kInvalidSessionStorageNamespaceId);
   return new RendererWebStorageNamespaceImpl(DOM_STORAGE_SESSION,
                                              session_storage_namespace_id_);
 }
 
 void RenderView::didAddMessageToConsole(
     const WebConsoleMessage& message, const WebString& source_name,
     unsigned source_line) {
   logging::LogSeverity log_severity = logging::LOG_VERBOSE;
-  switch(message.level) {
+  switch (message.level) {

[scherkus (not reviewing), 2011/06/14 22:22:12]

?

[henrika_dont_use, 2011/06/15 15:16:39]

Resolves lint warning. Should it not be a space here?

On 2011/06/14 22:22:12, scherkus wrote:

[scherkus (not reviewing), 2011/06/16 21:56:22]

nope that's right -- I just had a brainfart :)

     case WebConsoleMessage::LevelTip:
       log_severity = logging::LOG_VERBOSE;
       break;
     case WebConsoleMessage::LevelLog:
       log_severity = logging::LOG_INFO;
       break;
     case WebConsoleMessage::LevelWarning:
       log_severity = logging::LOG_WARNING;
       break;
     case WebConsoleMessage::LevelError:
@@ skipping 2877 matching lines @@
     const webkit_glue::CustomContextMenuContext& custom_context) {
   if (custom_context.is_pepper_menu)
     pepper_delegate_.OnContextMenuClosed(custom_context);
   else
     context_menu_node_.reset();
 }
 
 void RenderView::OnNetworkStateChanged(bool online) {
   WebNetworkStateNotifier::setOnLine(online);
 }