Fix sizing issues in the tab modal signin flow.

chrome/browser/ui/signin_view_controller_delegate.h

Index: chrome/browser/ui/signin_view_controller_delegate.h
diff --git a/chrome/browser/ui/signin_view_controller_delegate.h b/chrome/browser/ui/signin_view_controller_delegate.h
index 85ada3281fd8ccb07e4bd70637a66a20a8e5f11e..3d381efb21fbd5e75e57cdac285845452511fcd2 100644
--- a/chrome/browser/ui/signin_view_controller_delegate.h
+++ b/chrome/browser/ui/signin_view_controller_delegate.h
@@ -36,6 +36,14 @@ class SigninViewControllerDelegate : public content::WebContentsDelegate {
 
   void CloseModalSignin();
 
+  // Either navigates back in the signin flow if the history state allows it or
+  // closes the flow otherwise.
+  void PerformNavigation();
+
+  // This will be called by the base class to request a resize of the native
+  // view hosting the content to |height|.
+  virtual void ResizeNativeView(int height) = 0;

[sky, 2016/04/04 15:04:02]

Document units of height.

[anthonyvd, 2016/04/04 20:35:24]

Done now on the base class.

+
  protected:
   SigninViewControllerDelegate(SigninViewController* signin_view_controller,
                                content::WebContents* web_contents);
@@ -48,10 +56,6 @@ class SigninViewControllerDelegate : public content::WebContentsDelegate {
   void LoadingStateChanged(content::WebContents* source,
                            bool to_different_document) override;
 
-  // Handles the web ui message sent when the navigation button is clicked by
-  // the user, requesting either a back navigation or closing the dialog.
-  void HandleNavigationButtonClicked(const base::ListValue* args);
-
   // This will be called by this base class when the tab-modal window must be
   // closed. This should close the platform-specific window that is currently
   // showing the sign in flow or the sync confirmation dialog.