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

Unified Diff: content/browser/media/media_web_contents_observer.h

Issue 318753010: Introduce the ENABLE_BROWSER_CDMS macro. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: MEDIA_EXPORT Created 6 years, 6 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/media/media_web_contents_observer.h
diff --git a/content/browser/media/android/media_web_contents_observer.h b/content/browser/media/media_web_contents_observer.h
similarity index 83%
rename from content/browser/media/android/media_web_contents_observer.h
rename to content/browser/media/media_web_contents_observer.h
index 38b82b27ea1c1450a7d24e9f6d98318c9aef5612..8ed89932057a0f8dc11ba5f8cdcc76582d5bb3e9 100644
--- a/content/browser/media/android/media_web_contents_observer.h
+++ b/content/browser/media/media_web_contents_observer.h
@@ -2,8 +2,8 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef CONTENT_BROWSER_MEDIA_ANDROID_MEDIA_WEB_CONTENTS_OBSERVER_H_
-#define CONTENT_BROWSER_MEDIA_ANDROID_MEDIA_WEB_CONTENTS_OBSERVER_H_
+#ifndef CONTENT_BROWSER_MEDIA_MEDIA_WEB_CONTENTS_OBSERVER_H_
+#define CONTENT_BROWSER_MEDIA_MEDIA_WEB_CONTENTS_OBSERVER_H_
#include "base/compiler_specific.h"
#include "base/containers/scoped_ptr_hash_map.h"
@@ -13,7 +13,9 @@
namespace content {
class BrowserCdmManager;
+#if defined(OS_ANDROID)
class BrowserMediaPlayerManager;
+#endif // defined(OS_ANDROID)
class RenderViewHost;
// This class manages all RenderFrame based media related managers at the
@@ -30,12 +32,20 @@ class CONTENT_EXPORT MediaWebContentsObserver : public WebContentsObserver {
virtual bool OnMessageReceived(const IPC::Message& message,
RenderFrameHost* render_frame_host) OVERRIDE;
- // Helper functions to handle various IPC messages. Returns whether the
+ // Helper function to handle CDM IPC messages. Returns whether the |message|
+ // is handled in the function.
+ bool OnCdmMessageReceived(const IPC::Message& message,
+ RenderFrameHost* render_frame_host);
+
+ // Gets the CDM manager associated with |render_frame_host|. Creates
+ // a new one if it doesn't exist. The caller doesn't own the returned pointer.
+ BrowserCdmManager* GetCdmManager(RenderFrameHost* render_frame_host);
+
+#if defined(OS_ANDROID)
+ // Helper functions to handle media player IPC messages. Returns whether the
// |message| is handled in the function.
bool OnMediaPlayerMessageReceived(const IPC::Message& message,
RenderFrameHost* render_frame_host);
- bool OnCdmMessageReceived(const IPC::Message& message,
- RenderFrameHost* render_frame_host);
bool OnMediaPlayerSetCdmMessageReceived(const IPC::Message& message,
RenderFrameHost* render_frame_host);
@@ -44,10 +54,6 @@ class CONTENT_EXPORT MediaWebContentsObserver : public WebContentsObserver {
BrowserMediaPlayerManager* GetMediaPlayerManager(
RenderFrameHost* render_frame_host);
- // Gets the CDM manager associated with |render_frame_host|. Creates
- // a new one if it doesn't exist. The caller doesn't own the returned pointer.
- BrowserCdmManager* GetCdmManager(RenderFrameHost* render_frame_host);
-
void OnSetCdm(RenderFrameHost* render_frame_host, int player_id, int cdm_id);
// Pauses all media player.
@@ -57,19 +63,23 @@ class CONTENT_EXPORT MediaWebContentsObserver : public WebContentsObserver {
void OnFrameInfoUpdated();
#endif // defined(VIDEO_HOLE)
+#endif // defined(OS_ANDROID)
+
private:
+ // Map from RenderFrameHost* to BrowserCdmManager.
+ typedef base::ScopedPtrHashMap<uintptr_t, BrowserCdmManager> CdmManagerMap;
+ CdmManagerMap cdm_managers_;
+
+#if defined(OS_ANDROID)
// Map from RenderFrameHost* to BrowserMediaPlayerManager.
typedef base::ScopedPtrHashMap<uintptr_t, BrowserMediaPlayerManager>
MediaPlayerManagerMap;
MediaPlayerManagerMap media_player_managers_;
-
- // Map from RenderFrameHost* to BrowserCdmManager.
- typedef base::ScopedPtrHashMap<uintptr_t, BrowserCdmManager> CdmManagerMap;
- CdmManagerMap cdm_managers_;
+#endif // defined(OS_ANDROID)
DISALLOW_COPY_AND_ASSIGN(MediaWebContentsObserver);
};
} // namespace content
-#endif // CONTENT_BROWSER_MEDIA_ANDROID_MEDIA_WEB_CONTENTS_OBSERVER_H_
+#endif // CONTENT_BROWSER_MEDIA_MEDIA_WEB_CONTENTS_OBSERVER_H_
« no previous file with comments | « content/browser/media/cdm/browser_cdm_manager.cc ('k') | content/browser/media/media_web_contents_observer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698