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

Unified Diff: ios/chrome/browser/ui/fullscreen_controller.h

Issue 2594793004: Revert 'Bypass UIWebView.scrollView's contentInset implementation with UIScrollView's. (Closed)
Patch Set: Eugene comments Created 4 years 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 | « ios/chrome/browser/ui/browser_view_controller.mm ('k') | ios/chrome/browser/ui/fullscreen_controller.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/chrome/browser/ui/fullscreen_controller.h
diff --git a/ios/chrome/browser/ui/fullscreen_controller.h b/ios/chrome/browser/ui/fullscreen_controller.h
index e7127efdc96a962ab1ccfb69c5fa0f3ca863e8a0..14dbd25c6d5fa42a6f583bd6eadb50a25e4878b2 100644
--- a/ios/chrome/browser/ui/fullscreen_controller.h
+++ b/ios/chrome/browser/ui/fullscreen_controller.h
@@ -106,13 +106,8 @@ extern NSString* const kSetupForTestingWillCloseAllTabsNotification;
// is necessary when programatically scrolling down the y offset.
- (void)shouldSkipNextScrollOffsetForHeader;
-// Update the insets during animation. When |forceUpdate| is set to NO, a faster
-// workaround implemention is used to update the content's offset. That
-// implemention does not cause a full update. |forceUpdate| should only be set
-// to |NO| when -setToolbarInsetsForHeaderOffset is called
-// in quick succession during scroll callbacks.
-- (void)setToolbarInsetsForHeaderOffset:(CGFloat)headerOffset
- forceUpdate:(BOOL)forceUpdate;
+// Updates the insets during animation.
+- (void)setToolbarInsetsForHeaderOffset:(CGFloat)headerOffset;
// Set the content offset of the underlying UIScrollView so that the content
// is not hidden by the header. The header will be moved to its visible position
« no previous file with comments | « ios/chrome/browser/ui/browser_view_controller.mm ('k') | ios/chrome/browser/ui/fullscreen_controller.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698