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

Unified Diff: content/browser/android/overscroll_controller_android.cc

Issue 1084533004: [Android] Disable the refresh effect for fullscreen HTML5 scenarios (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 8 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 | « content/browser/android/overscroll_controller_android.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/android/overscroll_controller_android.cc
diff --git a/content/browser/android/overscroll_controller_android.cc b/content/browser/android/overscroll_controller_android.cc
index 02e2c01383eb6ac8df2773b62e5b6a614db82343..d73cd22247cab5c23cf7caed53ea3628ad5ab35c 100644
--- a/content/browser/android/overscroll_controller_android.cc
+++ b/content/browser/android/overscroll_controller_android.cc
@@ -97,6 +97,13 @@ OverscrollControllerAndroid::OverscrollControllerAndroid(
refresh_effect_(CreateRefreshEffect(content_view_core)),
is_fullscreen_(false) {
DCHECK(compositor_);
+ // Fullscreen state is only relevant for the refresh effect.
+ if (refresh_effect_) {
+ WebContentsImpl* web_contents =
+ static_cast<WebContentsImpl*>(content_view_core->GetWebContents());
+ is_fullscreen_ = web_contents->IsFullscreenForCurrentTab();
+ Observe(web_contents);
+ }
}
OverscrollControllerAndroid::~OverscrollControllerAndroid() {
@@ -268,6 +275,7 @@ void OverscrollControllerAndroid::Disable() {
void OverscrollControllerAndroid::DidToggleFullscreenModeForTab(
bool entered_fullscreen) {
+ DCHECK(refresh_effect_);
if (is_fullscreen_ == entered_fullscreen)
return;
is_fullscreen_ = entered_fullscreen;
« no previous file with comments | « content/browser/android/overscroll_controller_android.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698