[Mac] Position the avatar and fullscreen buttons when adding the first profile.

[Mac] Position the avatar and fullscreen buttons when adding the first profile.

BUG=138586
TEST=On Lion or higher, run with a fresh profile. Go to settings and add a new profile. The avatar button in the original window is to the right of the fullscreen button.
TBR=gyp

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=149559

[sail, 2012-08-02 00:12:40]

LGTM
Awesome test!

https://chromiumcodereview.appspot.com/10823130/diff/1/chrome/browser/ui/coco...
File chrome/browser/ui/cocoa/browser/avatar_button_controller.mm (right):

https://chromiumcodereview.appspot.com/10823130/diff/1/chrome/browser/ui/coco...
chrome/browser/ui/cocoa/browser/avatar_button_controller.mm:43: // Declare a
private 10.7+ private API.
remove one of the "private"s here?

https://chromiumcodereview.appspot.com/10823130/diff/1/chrome/browser/ui/coco...
File chrome/browser/ui/cocoa/browser_window_controller_browsertest.mm (right):

https://chromiumcodereview.appspot.com/10823130/diff/1/chrome/browser/ui/coco...
chrome/browser/ui/cocoa/browser_window_controller_browsertest.mm:54:
static_cast<BrowserWindowController*>([window windowController]);
you can also use browser()->window()->cocoa_controller()

https://chromiumcodereview.appspot.com/10823130/diff/1/chrome/browser/ui/coco...
chrome/browser/ui/cocoa/browser_window_controller_browsertest.mm:98:
EXPECT_LT(NSMinX(fullscreen_frame), NSMinX(avatar_frame));
I don't think you need this line

[Robert Sesek, 2012-08-02 00:23:40]

https://chromiumcodereview.appspot.com/10823130/diff/1/chrome/browser/ui/coco...
File chrome/browser/ui/cocoa/browser/avatar_button_controller.mm (right):

https://chromiumcodereview.appspot.com/10823130/diff/1/chrome/browser/ui/coco...
chrome/browser/ui/cocoa/browser/avatar_button_controller.mm:43: // Declare a
private 10.7+ private API.
On 2012/08/02 00:12:40, sail wrote:
> remove one of the "private"s here?

Done.

https://chromiumcodereview.appspot.com/10823130/diff/1/chrome/browser/ui/coco...
File chrome/browser/ui/cocoa/browser_window_controller_browsertest.mm (right):

https://chromiumcodereview.appspot.com/10823130/diff/1/chrome/browser/ui/coco...
chrome/browser/ui/cocoa/browser_window_controller_browsertest.mm:54:
static_cast<BrowserWindowController*>([window windowController]);
On 2012/08/02 00:12:40, sail wrote:
> you can also use browser()->window()->cocoa_controller()

Cool!

https://chromiumcodereview.appspot.com/10823130/diff/1/chrome/browser/ui/coco...
chrome/browser/ui/cocoa/browser_window_controller_browsertest.mm:98:
EXPECT_LT(NSMinX(fullscreen_frame), NSMinX(avatar_frame));
On 2012/08/02 00:12:40, sail wrote:
> I don't think you need this line

I wanted to have the fullscreen_frame MinX if it failed.

[commit-bot: I haz the power, 2012-08-02 00:24:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rsesek@chromium.org/10823130/8002

[commit-bot: I haz the power, 2012-08-02 00:24:25]

Presubmit check for 10823130-8002 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    chrome

[commit-bot: I haz the power, 2012-08-02 00:25:02]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rsesek@chromium.org/10823130/8002

[commit-bot: I haz the power, 2012-08-02 01:55:18]

Change committed as 149559