DevTools: respect minimum width/height when resizing.

chrome/browser/ui/cocoa/dev_tools_controller.mm

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #import "chrome/browser/ui/cocoa/dev_tools_controller.h"
 
 #include <algorithm>
 
 #include <Cocoa/Cocoa.h>
 
@@ skipping 211 matching lines @@
 - (CGFloat)splitView:(NSSplitView*)splitView
     constrainSplitPosition:(CGFloat)proposedPosition
                ofSubviewAt:(NSInteger)dividerIndex {
   if (![splitView_ isVertical] &&
       proposedPosition < [splitView_ topContentOffset]) {
     return [splitView_ topContentOffset];
   }
   return proposedPosition;
 }
 
+- (CGFloat)splitView:(NSSplitView *)splitView

[Avi (use Gerrit), 2013/03/19 14:43:23]

no space before *

[dgozman, 2013/03/21 09:55:26]

Done.

[dgozman, 2013/03/21 09:55:26]

Done.

+    constrainMaxCoordinate:(CGFloat)proposedMax
+               ofSubviewAt:(NSInteger)dividerIndex {

[Avi (use Gerrit), 2013/03/19 14:43:23]

If you need to indent the "constrain" line, OK, but align "ofsubviewat" to the
first line.

[dgozman, 2013/03/21 09:55:26]

I did as in method above. Also, style guide suggests to align all but first
parameters.

+  if ([splitView_ isVertical]) {
+    return NSWidth([splitView_ frame]) - [splitView_ dividerThickness] -
+        devToolsWindow_->GetMinimumWidth();
+  } else {
+    return NSHeight([splitView_ frame]) - [splitView_ dividerThickness] -
+        devToolsWindow_->GetMinimumHeight();
+  }
+}
+
 -(void)splitViewWillResizeSubviews:(NSNotification *)notification {
   [[splitView_ window] disableScreenUpdatesUntilFlush];
 }
 
 @end