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

Unified Diff: ios/chrome/browser/ui/browser_view_controller.mm

Issue 2599313002: Create headerHeightForTab in TabHeadersDelegate. (Closed)
Patch Set: only fullscreencontroller and sideswipecontroller Created 3 years, 11 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
Index: ios/chrome/browser/ui/browser_view_controller.mm
diff --git a/ios/chrome/browser/ui/browser_view_controller.mm b/ios/chrome/browser/ui/browser_view_controller.mm
index dba26efc255093fb1f866eea27f75fb741f7dd64..4f3aefd2e870fb1955e226eb0851f70a761bfef8 100644
--- a/ios/chrome/browser/ui/browser_view_controller.mm
+++ b/ios/chrome/browser/ui/browser_view_controller.mm
@@ -734,7 +734,7 @@ NSString* const kReadingListSnackbarCategory = @"ReadingListSnackbarCategory";
// Returns the footer view if one exists (e.g. the voice search bar).
- (UIView*)footerView;
// Returns the height of the header view for the tab model's current tab.
-- (CGFloat)headerHeight;
+- (CGFloat)headerHeightForCurrentTab;
// Returns the height of the header view for |tab|.
- (CGFloat)headerHeightForTab:(Tab*)tab;
// Sets the frame for the headers.
@@ -1115,7 +1115,7 @@ class BrowserBookmarkModelBridge : public bookmarks::BookmarkModelObserver {
}
- (BOOL)isToolbarOnScreen {
- return [self headerHeight] - [self currentHeaderOffset] > 0;
+ return [self headerHeightForCurrentTab] - [self currentHeaderOffset] > 0;
}
- (BOOL)isInNewTabAnimation {
@@ -1830,7 +1830,8 @@ class BrowserBookmarkModelBridge : public bookmarks::BookmarkModelObserver {
// the toolbar is not fullscreen.
CGRect contentFrame = [_contentArea frame];
CGFloat marginWithHeader = StatusBarHeight();
- CGFloat overlap = [self headerHeight] != 0 ? marginWithHeader : minY;
+ CGFloat overlap =
+ [self headerHeightForCurrentTab] != 0 ? marginWithHeader : minY;
contentFrame.size.height = CGRectGetMaxY(contentFrame) - overlap;
contentFrame.origin.y = overlap;
[_contentArea setFrame:contentFrame];
@@ -2084,8 +2085,8 @@ class BrowserBookmarkModelBridge : public bookmarks::BookmarkModelObserver {
- (UIImageView*)pageOpenCloseAnimationView {
CGRect frame = [_contentArea bounds];
- frame.size.height = frame.size.height - [self headerHeight];
- frame.origin.y = [self headerHeight];
+ frame.size.height = frame.size.height - [self headerHeightForCurrentTab];
+ frame.origin.y = [self headerHeightForCurrentTab];
UIImageView* pageView =
[[[UIImageView alloc] initWithFrame:frame] autorelease];
@@ -2584,7 +2585,7 @@ class BrowserBookmarkModelBridge : public bookmarks::BookmarkModelObserver {
return _voiceSearchBar;
}
-- (CGFloat)headerHeight {
+- (CGFloat)headerHeightForCurrentTab {
return [self headerHeightForTab:[_model currentTab]];
}
@@ -2621,10 +2622,11 @@ class BrowserBookmarkModelBridge : public bookmarks::BookmarkModelObserver {
if (!headers.size())
return 0.0;
- // Prerender tab does not have a toolbar, return |headerHeight| as promised by
+ // Prerender tab does not have a toolbar, return |headerHeightForCurrentTab|
+ // as promised by
// API documentation.
if ([[[self tabModel] currentTab] isPrerenderTab])
- return [self headerHeight];
+ return [self headerHeightForCurrentTab];
UIView* topHeader = headers[0].view;
return -(topHeader.frame.origin.y - [self headerOffset]);
@@ -2632,7 +2634,7 @@ class BrowserBookmarkModelBridge : public bookmarks::BookmarkModelObserver {
- (CGFloat)footerYForHeaderOffset:(CGFloat)headerOffset {
UIView* footer = [self footerView];
- CGFloat headerHeight = [self headerHeight];
+ CGFloat headerHeight = [self headerHeightForCurrentTab];
if (!footer || headerHeight == 0)
return 0.0;
@@ -2798,13 +2800,13 @@ class BrowserBookmarkModelBridge : public bookmarks::BookmarkModelObserver {
- (CGFloat)overscrollActionsControllerHeaderInset:
(OverscrollActionsController*)controller {
if (controller == [[[self tabModel] currentTab] overscrollActionsController])
- return [self headerHeight];
+ return [self headerHeightForCurrentTab];
else
return 0;
}
- (CGFloat)overscrollHeaderHeight {
- return [self headerHeight] + StatusBarHeight();
+ return [self headerHeightForCurrentTab] + StatusBarHeight();
}
#pragma mark - TabSnapshottingDelegate methods.
@@ -3683,7 +3685,7 @@ class BrowserBookmarkModelBridge : public bookmarks::BookmarkModelObserver {
// This is called from within an animation block.
- (void)toolbarHeightChanged {
- if ([self headerHeight] != 0) {
+ if ([self headerHeightForCurrentTab] != 0) {
// Ensure full screen height is updated.
Tab* currentTab = [_model currentTab];
BOOL visible = self.isToolbarOnScreen;
@@ -3839,7 +3841,7 @@ class BrowserBookmarkModelBridge : public bookmarks::BookmarkModelObserver {
CGRect frame = [_contentArea frame];
if (!fullScreen) {
// Changing the origin here is unnecessary, it's set in page_animation_util.
- frame.size.height -= [self headerHeight];
+ frame.size.height -= [self headerHeightForCurrentTab];
}
CGFloat shortAxis = frame.size.width;
@@ -4736,7 +4738,7 @@ class BrowserBookmarkModelBridge : public bookmarks::BookmarkModelObserver {
}
- (CGFloat)currentHeaderHeight {
- return [self headerHeight] - [self currentHeaderOffset];
+ return [self headerHeightForCurrentTab] - [self currentHeaderOffset];
}
#pragma mark - InfoBarControllerDelegate

Powered by Google App Engine
This is Rietveld 408576698