Create headerHeightForTab in TabHeadersDelegate.

ios/chrome/browser/ui/fullscreen_controller.h

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 14dbd25c6d5fa42a6f583bd6eadb50a25e4878b2..f841ca4087af6e0c08a459fcbe8346484d41e489 100644
--- a/ios/chrome/browser/ui/fullscreen_controller.h
+++ b/ios/chrome/browser/ui/fullscreen_controller.h
@@ -17,6 +17,7 @@ const NSTimeInterval kToolbarAnimationDuration = 0.3;
 
 @class CRWWebViewScrollViewProxy;
 @class FullScreenController;
+@class Tab;
 
 namespace web {
 class NavigationManager;
@@ -49,10 +50,15 @@ extern NSString* const kSetupForTestingWillCloseAllTabsNotification;
      changeTopContentPadding:(BOOL)changeTopContentPadding
            scrollingToOffset:(CGFloat)contentOffset;
 
-// Called to retrieve the current height of the header. Only called from
+// Called to retrieve the height of the header view for the tab model's current
+// tab. Only called from -setHeaderVisible:, so that method needs to be
+// explicitly called when the height changes.
+- (CGFloat)headerHeightForCurrentTab;
+
+// Called to retrieve the height of the header view for |tab|. Only called from
 // -setHeaderVisible:, so that method needs to be explicitly called when the
 // height changes.
-- (CGFloat)headerHeight;
+- (CGFloat)headerHeightForTab:(Tab*)tab;

[noyau (Ping after 24h), 2017/01/09 15:09:31]

This method should not be in this protocol, it is never called by the fullscreen
controller, and the full screen controller should not know anything whatsoever
about Tab.

[Olivier, 2017/01/09 17:23:18]

Right. The delegate was called from other places.
I create a specific delegate to handle headers.

 
 // Tests if the session ID matches the current tab.
 - (BOOL)isTabWithIDCurrent:(NSString*)sessionID;