Chromium Code Reviews| Index: content/browser/android/content_video_view.cc |
| diff --git a/content/browser/android/content_video_view.cc b/content/browser/android/content_video_view.cc |
| index 82f496670b9f7d8e8d3e617d055ea1558db992c8..07b7ed97fd5b9465dc5a1595bfaa106597dd396e 100644 |
| --- a/content/browser/android/content_video_view.cc |
| +++ b/content/browser/android/content_video_view.cc |
| @@ -249,31 +249,16 @@ gfx::NativeView ContentVideoView::GetNativeView() { |
| JavaObjectWeakGlobalRef ContentVideoView::CreateJavaObject() { |
| ContentViewCoreImpl* content_view_core = manager_->GetContentViewCore(); |
| JNIEnv* env = AttachCurrentThread(); |
| - bool legacyMode = base::CommandLine::ForCurrentProcess()->HasSwitch( |
| - switches::kDisableOverlayFullscreenVideoSubtitle); |
| return JavaObjectWeakGlobalRef( |
| env, |
| Java_ContentVideoView_createContentVideoView( |
| env, |
| content_view_core->GetContext().obj(), |
| reinterpret_cast<intptr_t>(this), |
| - content_view_core->GetContentVideoViewClient().obj(), |
| - legacyMode).obj()); |
| + content_view_core->GetContentVideoViewClient().obj()).obj()); |
| } |
| void ContentVideoView::CreatePowerSaveBlocker() { |
| - if (!base::CommandLine::ForCurrentProcess()->HasSwitch( |
| - switches::kDisableOverlayFullscreenVideoSubtitle)) { |
| - return; |
| - } |
| - |
| - if (power_save_blocker_) return; |
| - |
| - power_save_blocker_ = PowerSaveBlocker::Create( |
| - PowerSaveBlocker::kPowerSaveBlockPreventDisplaySleep, |
| - "Playing video").Pass(); |
|
mkosiba (inactive)
2014/09/24 15:38:30
intentional? how does the fullscreen html5 video v
Ignacio Solla
2014/09/25 09:36:16
Good catch, there is a bug that we didn't notice,
|
| - static_cast<PowerSaveBlockerImpl*>(power_save_blocker_.get())-> |
| - InitDisplaySleepBlocker(GetNativeView()); |
| } |
| } // namespace content |