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

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

Issue 2594793004: Revert 'Bypass UIWebView.scrollView's contentInset implementation with UIScrollView's. (Closed)
Patch Set: 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
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..7b8d652f9a349390649c76dbe7877ddec2880939 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;
+// Update the insets during animation.
Eugene But (OOO till 7-30) 2016/12/21 18:48:43 s/Update/Updates
justincohen 2016/12/21 19:59:45 Done.
+- (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

Powered by Google App Engine
This is Rietveld 408576698