Implements AudioMessageFilter as member in RenderThread

content/renderer/render_view.h

 // 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.
 
 #ifndef CONTENT_RENDERER_RENDER_VIEW_H_
 #define CONTENT_RENDERER_RENDER_VIEW_H_
 #pragma once
 
 #include <deque>
 #include <map>
 #include <queue>
 #include <set>
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/gtest_prod_util.h"
 #include "base/id_map.h"
 #include "base/memory/linked_ptr.h"
 #include "base/memory/weak_ptr.h"
 #include "base/observer_list.h"
 #include "base/timer.h"
 #include "build/build_config.h"
 #include "content/renderer/renderer_webcookiejar_impl.h"
 #include "content/common/edit_command.h"
 #include "content/common/navigation_gesture.h"
 #include "content/common/page_zoom.h"
 #include "content/common/renderer_preferences.h"
+#include "content/renderer/media/audio_message_filter.h"
+#include "content/renderer/media/audio_message_filter_creator.h"
 #include "content/renderer/pepper_plugin_delegate_impl.h"
 #include "content/renderer/render_widget.h"
 #include "ipc/ipc_platform_file.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebAccessibilityNotification.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebConsoleMessage.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebFileSystem.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebFrameClient.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebIconURL.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebNode.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebPageVisibilityState.h"
@@ skipping 163 matching lines @@
   WebKit::WebView* webview() const;
 
   // Called by a GraphicsContext associated with this view when swapbuffers
   // completes or is aborted.
   void OnViewContextSwapBuffersComplete();
   void OnViewContextSwapBuffersAborted();
 
   int page_id() const { return page_id_; }
   PepperPluginDelegateImpl* pepper_delegate() { return &pepper_delegate_; }
 
   AudioMessageFilter* audio_message_filter() {

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

will you move this to RenderThread or get rid of this method entirely?

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

Removed. Now uses singleton directly.

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

-    return audio_message_filter_;
+    AudioMessageFilter* audio_message_filter =
+        AudioMessageFilterCreator::SharedFilter();
+    return audio_message_filter;
   }
 
   const WebPreferences& webkit_preferences() const {
     return webkit_preferences_;
   }
 
   bool content_state_immediately() { return send_content_state_immediately_; }
   int enabled_bindings() const { return enabled_bindings_; }
   void set_enabled_bindings(int b) { enabled_bindings_ = b; }
   void set_send_content_state_immediately(bool value) {
@@ skipping 935 matching lines @@
   // bunch of stuff, you should probably create a helper class and put your
   // data and methods on that to avoid bloating RenderView more.  You can use
   // the Observer interface to filter IPC messages and receive frame change
   // notifications.
   // ---------------------------------------------------------------------------
 
   DISALLOW_COPY_AND_ASSIGN(RenderView);
 };
 
 #endif  // CONTENT_RENDERER_RENDER_VIEW_H_