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

Unified Diff: content/browser/android/content_video_view.h

Issue 639413002: [aw] Reuse WebContents PowerSaveBlocker in fullscreen. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@testsForPowerSaver
Patch Set: Nits Created 6 years, 2 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
« no previous file with comments | « android_webview/test/shell/assets/video.mp4 ('k') | content/browser/android/content_video_view.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/android/content_video_view.h
diff --git a/content/browser/android/content_video_view.h b/content/browser/android/content_video_view.h
index b45f38fc0c101e2abd203b06daa58321ac880bb3..c8d49219eac75d660892cddbe634f64c6011f3e1 100644
--- a/content/browser/android/content_video_view.h
+++ b/content/browser/android/content_video_view.h
@@ -19,7 +19,6 @@
namespace content {
class BrowserMediaPlayerManager;
-class PowerSaveBlocker;
// Native mirror of ContentVideoView.java. This class is responsible for
// creating the Java video view and pass all the player status change to
@@ -87,23 +86,10 @@ class ContentVideoView {
// Creates the corresponding ContentVideoView Java object.
JavaObjectWeakGlobalRef CreateJavaObject();
- // Returns the associated NativeView
- gfx::NativeView GetNativeView();
-
- void CreatePowerSaveBlocker();
-
// Object that manages the fullscreen media player. It is responsible for
// handling all the playback controls.
BrowserMediaPlayerManager* manager_;
- // PowerSaveBlock to keep screen on for fullscreen video.
- // There is already blocker when inline video started, and it requires the
- // ContentView's container displayed to take effect; but in WebView, apps
- // could use another container to hold ContentVideoView, and the blocker in
- // ContentView's container can not keep screen on; so we need another blocker
- // here, it is no harm, just an additonal blocker.
- scoped_ptr<PowerSaveBlocker> power_save_blocker_;
-
// Weak reference of corresponding Java object.
JavaObjectWeakGlobalRef j_content_video_view_;
« no previous file with comments | « android_webview/test/shell/assets/video.mp4 ('k') | content/browser/android/content_video_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698