[Mac] Prefs under the hood work:...

chrome/browser/cocoa/preferences_window_controller.mm

 // Copyright (c) 2009 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/cocoa/preferences_window_controller.h"
 
 #include <algorithm>
 #include "app/l10n_util.h"
 #include "base/mac_util.h"
 #include "base/string_util.h"
@@ skipping 202 matching lines @@
     if (view != label) {
       origin.y += nonLabelShift;
     }
     [view setFrameOrigin:origin];
   }
 
   // Return how much the group grew.
   return localVerticalShift + nonLabelShift;
 }
 
+// Compare function for -[NSArray sortedArrayUsingFunction:context:] that
+// sorts the views in Y order bottom up.
+NSInteger CompareFrameY(id view1, id view2, void* context) {
+  CGFloat y1 = NSMinY([view1 frame]);
+  CGFloat y2 = NSMinY([view2 frame]);
+  if (y1 < y2)
+    return NSOrderedAscending;
+  else if (y1 > y2)
+    return NSOrderedDescending;
+  else
+    return NSOrderedSame;
+}
+
+// Helper to tweak the layout of the "Under the Hood" content by autosizing all
+// the views and moving things up vertically.  Special case the two controls for
+// download location as they are horizontal, and should fill the row.
+CGFloat AutoSizeUnderTheHoodContent(NSView* view,
+                                    NSPathControl* downloadLocationControl,
+                                    NSButton* downloadLocationButton) {
+  CGFloat verticalShift = 0.0;
+
+  // Loop bottom up through the views sizing and shifting.
+  NSArray* views = [view subviews];
+  views = [views sortedArrayUsingFunction:CompareFrameY context:NULL];

[Mark Mentovai, 2009/11/03 17:35:44]

Rather than assigning the same crap into the same variable, why not just do:

NSArray* views = [[view subviews] sortedArrayUsingFunction:...]

I think that's preferable to overwriting a variable with something that depends
on the variable itself.

+  for (NSView* view in views) {
+    NSSize delta = WrapOrSizeToFit(view);
+    DCHECK_GE(delta.height, 0.0) << "Should NOT shrink in height";
+    if (verticalShift) {
+      NSPoint origin = [view frame].origin;
+      origin.y += verticalShift;
+      [view setFrameOrigin:origin];
+    }
+    verticalShift += delta.height;
+    // The Download Location controls go in a row with the button aligned to the

[Mark Mentovai, 2009/11/03 17:35:44]

blank before?

+    // right edge and the path control using all the rest of the space.
+    if (view == downloadLocationButton) {
+      NSPoint origin = [downloadLocationButton frame].origin;
+      origin.x -= delta.width;
+      [downloadLocationButton setFrameOrigin:origin];
+      NSSize controlSize = [downloadLocationControl frame].size;
+      controlSize.width -= delta.width;
+      [downloadLocationControl setFrameSize:controlSize];
+    }
+  }
+
+  return verticalShift;
+}
+
 }  // namespace
 
 //-------------------------------------------------------------------------
 
 @interface PreferencesWindowController(Private)
 // Callback when preferences are changed. |prefName| is the name of the
 // pref that has changed.
 - (void)prefChanged:(std::wstring*)prefName;
 // Record the user performed a certain action and save the preferences.
 - (void)recordUserAction:(const wchar_t*)action;
@@ skipping 150 matching lines @@
   widest = std::max(widest, NSWidth(underTheHoodFrame));
   NSWindow* prefsWindow = [self window];
   NSRect frame = [prefsWindow frame];
   frame.size.width = widest;
   [prefsWindow setFrame:frame display:NO];
 
   // The Under the Hood prefs is a scroller, it shouldn't get any border, so it
   // gets resized to the as wide as the window ends up.
   underTheHoodFrame.size.width = widest;
   [underTheHoodView_ setFrame:underTheHoodFrame];
-  // Widen the Under the Hood content so things can rewrap
-  NSSize advancedContentSize = [advancedView_ frame].size;
-  advancedContentSize.width = [advancedScroller_ contentSize].width;
-  [advancedView_ setFrameSize:advancedContentSize];
+
+  // Widen the Under the Hood content so things can rewrap to the full width
+  NSSize underTheHoodContentSize = [underTheHoodContentView_ frame].size;
+  underTheHoodContentSize.width = [underTheHoodScroller_ contentSize].width;
+  [underTheHoodContentView_ setFrameSize:underTheHoodContentSize];
+
+  // Now that Under the Hood is the right wide, spin through auto sizing things

[Mark Mentovai, 2009/11/03 17:35:44]

hahaha

Now that Under the Hood is the right width, auto-size to the new width to get
the final height.

I think that's what your comment means, it's kind of nonsense now (and not just
because of "teh.")

+  // to that new width to get teh final height.
+  verticalShift = AutoSizeUnderTheHoodContent(underTheHoodContentView_,
+                                              downloadLocationControl_,
+                                              downloadLocationButton_);
+  [GTMUILocalizerAndLayoutTweaker
+      resizeViewWithoutAutoResizingSubViews:underTheHoodContentView_
+                                      delta:NSMakeSize(0.0, verticalShift)];
+  underTheHoodContentSize = [underTheHoodContentView_ frame].size;
 
   // Adjust the view origins so they show up centered.
   CenterViewForWidth(basicsView_, widest);
   CenterViewForWidth(personalStuffView_, widest);
   CenterViewForWidth(underTheHoodView_, widest);
 
-  // Put the advanced view into the scroller and scroll it to the top.
-  [advancedScroller_ setDocumentView:advancedView_];
-  [advancedView_ scrollPoint:NSMakePoint(0, advancedContentSize.height)];
+  // Put the Under the Hood content view into the scroller and scroll it to the
+  // top.
+  [underTheHoodScroller_ setDocumentView:underTheHoodContentView_];
+  [underTheHoodContentView_ scrollPoint:
+      NSMakePoint(0, underTheHoodContentSize.height)];
 
   // Get the last visited page from local state.
   OptionsPage page = static_cast<OptionsPage>(lastSelectedPage_.GetValue());
   if (page == OPTIONS_PAGE_DEFAULT)
     page = OPTIONS_PAGE_GENERAL;
 
   NSUInteger pageIndex = (NSUInteger)page;
   if (pageIndex >= [[toolbar_ items] count])
     pageIndex = 0;
   NSToolbarItem* firstItem = [[toolbar_ items] objectAtIndex:pageIndex];
@@ skipping 938 matching lines @@
   [[NSNotificationCenter defaultCenter]
       postNotificationName:kUserDoneEditingPrefsNotification
                     object:self];
 }
 
 - (void)controlTextDidEndEditing:(NSNotification*)notification {
   [customPagesSource_ validateURLs];
 }
 
 @end