diff options
-rw-r--r-- | build/common.gypi | 5 | ||||
-rw-r--r-- | cc/layers/video_layer_impl.cc | 8 | ||||
-rw-r--r-- | cc/resources/video_resource_updater.cc | 12 | ||||
-rw-r--r-- | cc/resources/video_resource_updater.h | 4 | ||||
-rw-r--r-- | content/browser/android/content_view_core_impl.cc | 8 | ||||
-rw-r--r-- | content/browser/media/android/browser_media_player_manager.cc | 12 | ||||
-rw-r--r-- | content/browser/media/android/browser_media_player_manager.h | 8 | ||||
-rw-r--r-- | content/browser/web_contents/web_contents_view_android.cc | 4 | ||||
-rw-r--r-- | content/browser/web_contents/web_contents_view_android.h | 4 | ||||
-rw-r--r-- | content/common/media/media_player_messages_android.h | 5 | ||||
-rw-r--r-- | content/renderer/media/android/renderer_media_player_manager.cc | 4 | ||||
-rw-r--r-- | content/renderer/media/android/renderer_media_player_manager.h | 8 | ||||
-rw-r--r-- | content/renderer/media/android/webmediaplayer_android.cc | 43 | ||||
-rw-r--r-- | content/renderer/media/android/webmediaplayer_android.h | 19 | ||||
-rw-r--r-- | media/base/video_frame.cc | 18 | ||||
-rw-r--r-- | media/base/video_frame.h | 8 |
16 files changed, 88 insertions, 82 deletions
diff --git a/build/common.gypi b/build/common.gypi index 99b336a..be71642 100644 --- a/build/common.gypi +++ b/build/common.gypi @@ -2145,7 +2145,10 @@ 'defines': ['OS_CHROMEOS=1'], }], ['google_tv==1', { - 'defines': ['GOOGLE_TV=1'], + 'defines': [ + 'GOOGLE_TV=1', + 'VIDEO_HOLE=1', + ], }], ['use_xi2_mt!=0 and use_x11==1', { 'defines': ['USE_XI2_MT=<(use_xi2_mt)'], diff --git a/cc/layers/video_layer_impl.cc b/cc/layers/video_layer_impl.cc index b48b20f..314fed6 100644 --- a/cc/layers/video_layer_impl.cc +++ b/cc/layers/video_layer_impl.cc @@ -18,9 +18,9 @@ #include "cc/trees/proxy.h" #include "media/base/video_frame.h" -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) #include "cc/quads/solid_color_draw_quad.h" -#endif +#endif // defined(VIDEO_HOLE) namespace cc { @@ -245,7 +245,7 @@ void VideoLayerImpl::AppendQuads(QuadSink* quad_sink, append_quads_data); break; } -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) // This block and other blocks wrapped around #if defined(GOOGLE_TV) is not // maintained by the general compositor team. Please contact the following // people instead: @@ -266,7 +266,7 @@ void VideoLayerImpl::AppendQuads(QuadSink* quad_sink, append_quads_data); break; } -#endif +#endif // defined(VIDEO_HOLE) case VideoFrameExternalResources::NONE: NOTIMPLEMENTED(); break; diff --git a/cc/resources/video_resource_updater.cc b/cc/resources/video_resource_updater.cc index c67e0d2..4abeab1 100644 --- a/cc/resources/video_resource_updater.cc +++ b/cc/resources/video_resource_updater.cc @@ -71,9 +71,9 @@ bool VideoResourceUpdater::VerifyFrame( case media::VideoFrame::YV16: case media::VideoFrame::YV12J: case media::VideoFrame::NATIVE_TEXTURE: -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) case media::VideoFrame::HOLE: -#endif +#endif // defined(VIDEO_HOLE) return true; // Unacceptable inputs. ¯\(°_o)/¯ @@ -109,9 +109,9 @@ static gfx::Size SoftwarePlaneDimension( case media::VideoFrame::I420: case media::VideoFrame::NATIVE_TEXTURE: case media::VideoFrame::HISTOGRAM_MAX: -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) case media::VideoFrame::HOLE: -#endif +#endif // defined(VIDEO_HOLE) NOTREACHED(); } } @@ -124,13 +124,13 @@ VideoFrameExternalResources VideoResourceUpdater::CreateForSoftwarePlanes( const scoped_refptr<media::VideoFrame>& video_frame) { media::VideoFrame::Format input_frame_format = video_frame->format(); -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) if (input_frame_format == media::VideoFrame::HOLE) { VideoFrameExternalResources external_resources; external_resources.type = VideoFrameExternalResources::HOLE; return external_resources; } -#endif +#endif // defined(VIDEO_HOLE) // Only YUV software video frames are supported. DCHECK(input_frame_format == media::VideoFrame::YV12 || diff --git a/cc/resources/video_resource_updater.h b/cc/resources/video_resource_updater.h index e3476d4..2a27739 100644 --- a/cc/resources/video_resource_updater.h +++ b/cc/resources/video_resource_updater.h @@ -37,11 +37,11 @@ class CC_EXPORT VideoFrameExternalResources { STREAM_TEXTURE_RESOURCE, IO_SURFACE, -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) // TODO(danakj): Implement this with a solid color layer instead of a video // frame and video layer. HOLE, -#endif +#endif // defined(VIDEO_HOLE) // TODO(danakj): Remove this and abstract TextureMailbox into // "ExternalResource" that can hold a hardware or software backing. diff --git a/content/browser/android/content_view_core_impl.cc b/content/browser/android/content_view_core_impl.cc index 7fb557f..5b7fb8b 100644 --- a/content/browser/android/content_view_core_impl.cc +++ b/content/browser/android/content_view_core_impl.cc @@ -1284,27 +1284,27 @@ void ContentViewCoreImpl::AttachExternalVideoSurface(JNIEnv* env, jobject obj, jint player_id, jobject jsurface) { -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) RenderViewHostImpl* rvhi = static_cast<RenderViewHostImpl*>( web_contents_->GetRenderViewHost()); if (rvhi && rvhi->media_player_manager()) { rvhi->media_player_manager()->AttachExternalVideoSurface( static_cast<int>(player_id), jsurface); } -#endif +#endif // defined(VIDEO_HOLE) } void ContentViewCoreImpl::DetachExternalVideoSurface(JNIEnv* env, jobject obj, jint player_id) { -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) RenderViewHostImpl* rvhi = static_cast<RenderViewHostImpl*>( web_contents_->GetRenderViewHost()); if (rvhi && rvhi->media_player_manager()) { rvhi->media_player_manager()->DetachExternalVideoSurface( static_cast<int>(player_id)); } -#endif +#endif // defined(VIDEO_HOLE) } jboolean ContentViewCoreImpl::IsRenderWidgetHostViewReady(JNIEnv* env, diff --git a/content/browser/media/android/browser_media_player_manager.cc b/content/browser/media/android/browser_media_player_manager.cc index 99b785f..43b93da 100644 --- a/content/browser/media/android/browser_media_player_manager.cc +++ b/content/browser/media/android/browser_media_player_manager.cc @@ -134,10 +134,10 @@ bool BrowserMediaPlayerManager::OnMessageReceived(const IPC::Message& msg) { IPC_MESSAGE_HANDLER(MediaKeysHostMsg_CreateSession, OnCreateSession) IPC_MESSAGE_HANDLER(MediaKeysHostMsg_UpdateSession, OnUpdateSession) IPC_MESSAGE_HANDLER(MediaKeysHostMsg_ReleaseSession, OnReleaseSession) -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) IPC_MESSAGE_HANDLER(MediaPlayerHostMsg_NotifyExternalSurface, OnNotifyExternalSurface) -#endif +#endif // defined(VIDEO_HOLE) IPC_MESSAGE_UNHANDLED(handled = false) IPC_END_MESSAGE_MAP() return handled; @@ -409,7 +409,7 @@ void BrowserMediaPlayerManager::OnSessionError( routing_id(), media_keys_id, session_id, error_code, system_code)); } -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) void BrowserMediaPlayerManager::AttachExternalVideoSurface(int player_id, jobject surface) { MediaPlayerAndroid* player = GetPlayer(player_id); @@ -435,7 +435,7 @@ void BrowserMediaPlayerManager::OnNotifyExternalSurface( if (view) view->NotifyExternalSurface(player_id, is_request, rect); } -#endif +#endif // defined(VIDEO_HOLE) void BrowserMediaPlayerManager::DisableFullscreenEncryptedMediaPlayback() { if (fullscreen_player_id_ == -1) @@ -536,12 +536,12 @@ void BrowserMediaPlayerManager::OnReleaseResources(int player_id) { if (player_id == fullscreen_player_id_) fullscreen_player_is_released_ = true; -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) WebContentsViewAndroid* view = static_cast<WebContentsViewAndroid*>(web_contents_->GetView()); if (view) view->NotifyExternalSurface(player_id, false, gfx::RectF()); -#endif +#endif // defined(VIDEO_HOLE) } void BrowserMediaPlayerManager::OnDestroyPlayer(int player_id) { diff --git a/content/browser/media/android/browser_media_player_manager.h b/content/browser/media/android/browser_media_player_manager.h index 36ed061..d407e7e 100644 --- a/content/browser/media/android/browser_media_player_manager.h +++ b/content/browser/media/android/browser_media_player_manager.h @@ -104,10 +104,10 @@ class CONTENT_EXPORT BrowserMediaPlayerManager media::MediaKeys::KeyError error_code, int system_code) OVERRIDE; -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) void AttachExternalVideoSurface(int player_id, jobject surface); void DetachExternalVideoSurface(int player_id); -#endif +#endif // defined(VIDEO_HOLE) // Called to disble the current fullscreen playback if the video is encrypted. // TODO(qinmin): remove this once we have the new fullscreen mode. @@ -145,10 +145,10 @@ class CONTENT_EXPORT BrowserMediaPlayerManager void OnReleaseSession(int media_keys_id, uint32 session_id); void OnSetMediaKeys(int player_id, int media_keys_id); -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) virtual void OnNotifyExternalSurface( int player_id, bool is_request, const gfx::RectF& rect); -#endif +#endif // defined(VIDEO_HOLE) // Adds a given player to the list. void AddPlayer(media::MediaPlayerAndroid* player); diff --git a/content/browser/web_contents/web_contents_view_android.cc b/content/browser/web_contents/web_contents_view_android.cc index f66b5ec..21fcf54 100644 --- a/content/browser/web_contents/web_contents_view_android.cc +++ b/content/browser/web_contents/web_contents_view_android.cc @@ -54,13 +54,13 @@ void WebContentsViewAndroid::SetContentViewCore( } } -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) void WebContentsViewAndroid::NotifyExternalSurface( int player_id, bool is_request, const gfx::RectF& rect) { if (content_view_core_) content_view_core_->NotifyExternalSurface(player_id, is_request, rect); } -#endif +#endif // defined(VIDEO_HOLE) gfx::NativeView WebContentsViewAndroid::GetNativeView() const { return content_view_core_ ? content_view_core_->GetViewAndroid() : NULL; diff --git a/content/browser/web_contents/web_contents_view_android.h b/content/browser/web_contents/web_contents_view_android.h index 849ca6d..48cfee9 100644 --- a/content/browser/web_contents/web_contents_view_android.h +++ b/content/browser/web_contents/web_contents_view_android.h @@ -29,11 +29,11 @@ class WebContentsViewAndroid : public WebContentsViewPort, // by the UI frontend. void SetContentViewCore(ContentViewCoreImpl* content_view_core); -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) void NotifyExternalSurface(int player_id, bool is_request, const gfx::RectF& rect); -#endif +#endif // defined(VIDEO_HOLE) // WebContentsView implementation -------------------------------------------- virtual gfx::NativeView GetNativeView() const OVERRIDE; diff --git a/content/common/media/media_player_messages_android.h b/content/common/media/media_player_messages_android.h index 2477837..5addcf1 100644 --- a/content/common/media/media_player_messages_android.h +++ b/content/common/media/media_player_messages_android.h @@ -270,14 +270,13 @@ IPC_MESSAGE_CONTROL2(MediaPlayerHostMsg_DurationChanged, int /* demuxer_client_id */, base::TimeDelta /* duration */) -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) // Notify the player about the external surface, requesting it if necessary. IPC_MESSAGE_ROUTED3(MediaPlayerHostMsg_NotifyExternalSurface, int /* player_id */, bool /* is_request */, gfx::RectF /* rect */) - -#endif +#endif // defined(VIDEO_HOLE) // Messages for encrypted media extensions API ------------------------------ // TODO(xhwang): Move the following messages to a separate file. diff --git a/content/renderer/media/android/renderer_media_player_manager.cc b/content/renderer/media/android/renderer_media_player_manager.cc index 4767b17..9af1c31 100644 --- a/content/renderer/media/android/renderer_media_player_manager.cc +++ b/content/renderer/media/android/renderer_media_player_manager.cc @@ -390,7 +390,7 @@ bool RendererMediaPlayerManager::IsInFullscreen(blink::WebFrame* frame) { return fullscreen_frame_ == frame; } -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) void RendererMediaPlayerManager::RequestExternalSurface( int player_id, const gfx::RectF& geometry) { @@ -426,6 +426,6 @@ void RendererMediaPlayerManager::RetrieveGeometryChanges( } } } -#endif +#endif // defined(VIDEO_HOLE) } // namespace content diff --git a/content/renderer/media/android/renderer_media_player_manager.h b/content/renderer/media/android/renderer_media_player_manager.h index 0965611..cf1a48f3 100644 --- a/content/renderer/media/android/renderer_media_player_manager.h +++ b/content/renderer/media/android/renderer_media_player_manager.h @@ -80,13 +80,13 @@ class RendererMediaPlayerManager : public RenderViewObserver { // Requests the player to exit fullscreen. void ExitFullscreen(int player_id); -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) // Requests an external surface for out-of-band compositing. void RequestExternalSurface(int player_id, const gfx::RectF& geometry); // RenderViewObserver overrides. virtual void DidCommitCompositorFrame() OVERRIDE; -#endif +#endif // defined(VIDEO_HOLE) // Encrypted media related methods. void InitializeCDM(int media_keys_id, @@ -132,10 +132,10 @@ class RendererMediaPlayerManager : public RenderViewObserver { // Gets the pointer to ProxyMediaKeys given the |media_keys_id|. ProxyMediaKeys* GetMediaKeys(int media_keys_id); -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) // Gets the list of media players with video geometry changes. void RetrieveGeometryChanges(std::map<int, gfx::RectF>* changes); -#endif +#endif // defined(VIDEO_HOLE) private: // Message handlers. diff --git a/content/renderer/media/android/webmediaplayer_android.cc b/content/renderer/media/android/webmediaplayer_android.cc index 6a0d2fc..be5b6dc 100644 --- a/content/renderer/media/android/webmediaplayer_android.cc +++ b/content/renderer/media/android/webmediaplayer_android.cc @@ -139,12 +139,13 @@ WebMediaPlayerAndroid::WebMediaPlayerAndroid( external_surface_threshold_ = -1; } } +#endif // defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) // Defer stream texture creation until we are sure it's necessary. - stream_id_ = 0; needs_establish_peer_ = false; current_frame_ = VideoFrame::CreateBlackFrame(gfx::Size(1, 1)); -#endif +#endif // defined(VIDEO_HOLE) TryCreateStreamTextureProxyIfNeeded(); if (blink::WebRuntimeFeatures::isPrefixedEncryptedMediaEnabled()) { @@ -324,15 +325,17 @@ void WebMediaPlayerAndroid::DidLoadMediaInfo( } void WebMediaPlayerAndroid::play() { -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) if (hasVideo() && needs_external_surface_ && !manager_->IsInFullscreen(frame_)) { DCHECK(!needs_establish_peer_); manager_->RequestExternalSurface(player_id_, last_computed_rect_); } +#endif // defined(VIDEO_HOLE) +#if defined(GOOGLE_TV) if (audio_renderer_ && paused()) audio_renderer_->Play(); -#endif +#endif // defined(GOOGLE_TV) TryCreateStreamTextureProxyIfNeeded(); if (hasVideo() && needs_establish_peer_) @@ -726,11 +729,15 @@ void WebMediaPlayerAndroid::OnVideoSizeChanged(int width, int height) { if (natural_size_.width == width && natural_size_.height == height) return; +#if defined(VIDEO_HOLE) + bool has_surface_size_restriction = false; #if defined(GOOGLE_TV) - if ((external_surface_threshold_ >= 0 && - external_surface_threshold_ <= width * height) || - // Use H/W surface for MSE as the content is protected. - media_source_delegate_) { + has_surface_size_restriction = external_surface_threshold_ >= 0 && + external_surface_threshold_ <= width * height; +#endif // defined(GOOGLE_TV) + // Use H/W surface for MSE as the content might be protected. + // TODO(qinmin): Change this so that only EME needs the H/W surface + if (media_source_delegate_ || has_surface_size_restriction) { needs_external_surface_ = true; if (!paused() && !manager_->IsInFullscreen(frame_)) manager_->RequestExternalSurface(player_id_, last_computed_rect_); @@ -743,7 +750,7 @@ void WebMediaPlayerAndroid::OnVideoSizeChanged(int width, int height) { EstablishSurfaceTexturePeer(); } } -#endif +#endif // defined(VIDEO_HOLE) natural_size_.width = width; natural_size_.height = height; @@ -790,10 +797,10 @@ void WebMediaPlayerAndroid::OnDidExitFullscreen() { if (!paused() && needs_establish_peer_) EstablishSurfaceTexturePeer(); -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) if (!paused() && needs_external_surface_) manager_->RequestExternalSurface(player_id_, last_computed_rect_); -#endif +#endif // defined(VIDEO_HOLE) frame_->view()->willExitFullScreen(); frame_->view()->didExitFullScreen(); @@ -859,9 +866,9 @@ void WebMediaPlayerAndroid::OnPlayerReleased() { if (!needs_external_surface_) needs_establish_peer_ = true; -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) last_computed_rect_ = gfx::RectF(); -#endif +#endif // defined(VIDEO_HOLE) } void WebMediaPlayerAndroid::ReleaseMediaResources() { @@ -1008,8 +1015,8 @@ void WebMediaPlayerAndroid::DrawRemotePlaybackIcon() { void WebMediaPlayerAndroid::ReallocateVideoFrame() { if (needs_external_surface_) { - // VideoFrame::CreateHoleFrame is only defined under GOOGLE_TV. -#if defined(GOOGLE_TV) + // VideoFrame::CreateHoleFrame is only defined under VIDEO_HOLE. +#if defined(VIDEO_HOLE) if (!natural_size_.isEmpty()) { scoped_refptr<VideoFrame> new_frame = VideoFrame::CreateHoleFrame(natural_size_); @@ -1018,8 +1025,8 @@ void WebMediaPlayerAndroid::ReallocateVideoFrame() { client_->repaint(); } #else - NOTIMPLEMENTED() << "Hole punching not supported outside of Google TV"; -#endif + NOTIMPLEMENTED() << "Hole punching not supported without VIDEO_HOLE flag"; +#endif // defined(VIDEO_HOLE) } else if (!is_remote_ && texture_id_) { scoped_refptr<VideoFrame> new_frame = VideoFrame::WrapNativeTexture( make_scoped_ptr(new VideoFrame::MailboxHolder( @@ -1145,7 +1152,7 @@ void WebMediaPlayerAndroid::UpdatePlayingState(bool is_playing) { delegate_->DidPause(this); } -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) bool WebMediaPlayerAndroid::RetrieveGeometryChange(gfx::RectF* rect) { if (!video_weblayer_) return false; diff --git a/content/renderer/media/android/webmediaplayer_android.h b/content/renderer/media/android/webmediaplayer_android.h index fd6555e..40582ca 100644 --- a/content/renderer/media/android/webmediaplayer_android.h +++ b/content/renderer/media/android/webmediaplayer_android.h @@ -191,12 +191,12 @@ class WebMediaPlayerAndroid // Detach the player from its manager. void Detach(); -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) // Retrieve geometry of the media player (i.e. location and size of the video // frame) if changed. Returns true only if the geometry has been changed since // the last call. bool RetrieveGeometryChange(gfx::RectF* rect); -#endif +#endif // defined(VIDEO_HOLE) virtual MediaKeyException generateKeyRequest( const blink::WebString& key_system, @@ -256,11 +256,6 @@ class WebMediaPlayerAndroid // Requesting whether the surface texture peer needs to be reestablished. void SetNeedsEstablishPeer(bool needs_establish_peer); -#if defined(GOOGLE_TV) - // Request external surface for out-of-band composition. - void RequestExternalSurface(); -#endif - private: void DrawRemotePlaybackIcon(); void ReallocateVideoFrame(); @@ -401,15 +396,17 @@ class WebMediaPlayerAndroid scoped_ptr<webkit::WebLayerImpl> video_weblayer_; +#if defined(VIDEO_HOLE) + // A rectangle represents the geometry of video frame, when computed last + // time. + gfx::RectF last_computed_rect_; +#endif // defined(VIDEO_HOLE) + #if defined(GOOGLE_TV) // Pixel threshold for external surface usage. Negative value means that the // threshold is not defined, so that external surface is never used. int external_surface_threshold_; - // A rectangle represents the geometry of video frame, when computed last - // time. - gfx::RectF last_computed_rect_; - // Media Stream related fields. media::Demuxer* demuxer_; base::Closure destroy_demuxer_cb_; diff --git a/media/base/video_frame.cc b/media/base/video_frame.cc index 2593bf3..8a4eb3c 100644 --- a/media/base/video_frame.cc +++ b/media/base/video_frame.cc @@ -54,10 +54,10 @@ std::string VideoFrame::FormatToString(VideoFrame::Format format) { return "I420"; case VideoFrame::NATIVE_TEXTURE: return "NATIVE_TEXTURE"; -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) case VideoFrame::HOLE: return "HOLE"; -#endif +#endif // defined(VIDEO_HOLE) case VideoFrame::YV12A: return "YV12A"; case VideoFrame::YV12J: @@ -227,8 +227,8 @@ scoped_refptr<VideoFrame> VideoFrame::CreateBlackFrame(const gfx::Size& size) { return CreateColorFrame(size, kBlackY, kBlackUV, kBlackUV, kZero); } -#if defined(GOOGLE_TV) -// This block and other blocks wrapped around #if defined(GOOGLE_TV) is not +#if defined(VIDEO_HOLE) +// This block and other blocks wrapped around #if defined(VIDEO_HOLE) is not // maintained by the general compositor team. Please contact the following // people instead: // @@ -243,15 +243,15 @@ scoped_refptr<VideoFrame> VideoFrame::CreateHoleFrame( VideoFrame::HOLE, size, gfx::Rect(size), size, base::TimeDelta(), false)); return frame; } -#endif +#endif // defined(VIDEO_HOLE) // static size_t VideoFrame::NumPlanes(Format format) { switch (format) { case VideoFrame::NATIVE_TEXTURE: -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) case VideoFrame::HOLE: -#endif +#endif // defined(VIDEO_HOLE) return 0; case VideoFrame::YV12: case VideoFrame::YV16: @@ -328,9 +328,9 @@ size_t VideoFrame::PlaneAllocationSize(Format format, case VideoFrame::UNKNOWN: case VideoFrame::NATIVE_TEXTURE: case VideoFrame::HISTOGRAM_MAX: -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) case VideoFrame::HOLE: -#endif +#endif // defined(VIDEO_HOLE) break; } NOTREACHED() << "Unsupported video frame format/plane: " diff --git a/media/base/video_frame.h b/media/base/video_frame.h index 8a63cd3..b51bfe9 100644 --- a/media/base/video_frame.h +++ b/media/base/video_frame.h @@ -44,9 +44,9 @@ class MEDIA_EXPORT VideoFrame : public base::RefCountedThreadSafe<VideoFrame> { YV16 = 2, // 16bpp YVU planar 1x1 Y, 2x1 VU samples I420 = 3, // 12bpp YVU planar 1x1 Y, 2x2 UV samples. YV12A = 4, // 20bpp YUVA planar 1x1 Y, 2x2 VU, 1x1 A samples. -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) HOLE = 5, // Hole frame. -#endif +#endif // defined(VIDEO_HOLE) NATIVE_TEXTURE = 6, // Native texture. Pixel-format agnostic. YV12J = 7, // JPEG color range version of YV12 HISTOGRAM_MAX, // Must always be greatest. @@ -188,10 +188,10 @@ class MEDIA_EXPORT VideoFrame : public base::RefCountedThreadSafe<VideoFrame> { // equivalent of RGB(0,0,0). static scoped_refptr<VideoFrame> CreateBlackFrame(const gfx::Size& size); -#if defined(GOOGLE_TV) +#if defined(VIDEO_HOLE) // Allocates a hole frame. static scoped_refptr<VideoFrame> CreateHoleFrame(const gfx::Size& size); -#endif +#endif // defined(VIDEO_HOLE) static size_t NumPlanes(Format format); |