DevTools: respect minimum width/height when resizing.

chrome/browser/ui/cocoa/dev_tools_controller.mm

Index: chrome/browser/ui/cocoa/dev_tools_controller.mm
===================================================================
--- chrome/browser/ui/cocoa/dev_tools_controller.mm	(revision 187863)
+++ chrome/browser/ui/cocoa/dev_tools_controller.mm	(working copy)
@@ -229,6 +229,18 @@
   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];
 }