Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(121)

Issue 155444: Mac pop-up window happier-ness.... (Closed)

Created:
11 years, 5 months ago by John Grabowski
Modified:
9 years, 6 months ago
CC:
chromium-reviews_googlegroups.com, John Grabowski, Ben Goodger (Google)
Visibility:
Public.

Description

Mac pop-up window happier-ness. BUG=http://crbug.com/15727, http://crbug.com/16063, http://crbug.com/16329 TEST=A few things. Bookmark bar test (16063): - open bookmark bar, then open popup. Popup should not have bmb. - close bookmark bar, then open popup. Popup should not have bmb. - In main window, toggle bmb a few times. Popup should not change. Tab at bottom bug (16329): - Open 'details' popup in gmail (next to 'last acct activity') - make sure no tab strip at the bottom of the window General (15727): - Open a popup. Make sure no toolbar (back/fwd buttons et al) Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=20773

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Total comments: 4

Patch Set 4 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+78 lines, -2 lines) Patch
M chrome/browser/cocoa/bookmark_bar_controller.mm View 1 2 3 1 chunk +8 lines, -2 lines 0 comments Download
M chrome/browser/cocoa/bookmark_bar_controller_unittest.mm View 1 2 3 1 chunk +6 lines, -0 lines 0 comments Download
M chrome/browser/cocoa/browser_window_controller.h View 1 2 3 2 chunks +2 lines, -0 lines 0 comments Download
M chrome/browser/cocoa/tab_window_controller.mm View 1 2 3 1 chunk +6 lines, -0 lines 0 comments Download
M chrome/browser/cocoa/toolbar_controller.h View 1 2 3 2 chunks +6 lines, -0 lines 0 comments Download
M chrome/browser/cocoa/toolbar_controller.mm View 1 2 3 3 chunks +33 lines, -0 lines 0 comments Download
M chrome/browser/cocoa/toolbar_controller_unittest.mm View 1 2 3 1 chunk +17 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
John Grabowski
11 years, 5 months ago (2009-07-13 21:18:23 UTC) #1
pink (ping after 24hrs)
LGTM http://codereview.chromium.org/155444/diff/22/1015 File chrome/browser/cocoa/bookmark_bar_controller_unittest.mm (right): http://codereview.chromium.org/155444/diff/22/1015#newcode64 Line 64: // Awkwardness to look like we've been ...
11 years, 5 months ago (2009-07-14 14:22:09 UTC) #2
John Grabowski
11 years, 5 months ago (2009-07-14 18:39:49 UTC) #3
All comment feedback applied.

http://codereview.chromium.org/155444/diff/22/1014
File chrome/browser/cocoa/toolbar_controller.h (right):

http://codereview.chromium.org/155444/diff/22/1014#newcode113
Line 113: // around).  This This changes the behavior of other methods, like
On 2009/07/14 14:22:09, pink wrote:
> s/This This/This. What does it mean to turn off the toolbar but have a
location
> bar?

It means "yes URL bar, no buttons (stop/referesh/back/fwd), no bookmark bar,
etc".... like windows.

Powered by Google App Engine
This is Rietveld 408576698