Chromium Code Reviews

Issue 1105010: Alter the "get themes" action on Mac to behave like other platforms; fix bugs... (Closed)

Created:
10 years, 9 months ago by Avi (use Gerrit)
Modified:
9 years, 7 months ago
Reviewers:
pink (ping after 24hrs)
CC:
chromium-reviews, John Grabowski, pam+watch_chromium.org, ben+cc_chromium.org
Visibility:
Public.

Description

Alter the "get themes" action on Mac to behave like other platforms; fix bugs. BUG=http://crbug.com/38002 TEST=as in bug Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=42374

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Stats (+3 lines, -11 lines)
M chrome/browser/cocoa/preferences_window_controller.mm View 1 chunk +3 lines, -11 lines 1 comment

Messages

Total messages: 3 (0 generated)
Avi (use Gerrit)
10 years, 9 months ago (2010-03-22 18:44:55 UTC) #1
Avi (use Gerrit)
http://codereview.chromium.org/1105010/diff/1/2 File chrome/browser/cocoa/preferences_window_controller.mm (right): http://codereview.chromium.org/1105010/diff/1/2#newcode1315 chrome/browser/cocoa/preferences_window_controller.mm:1315: Browser* browser = BrowserList::GetLastActive(); BTW, you don't have to ...
10 years, 9 months ago (2010-03-23 16:24:11 UTC) #2
pink (ping after 24hrs)
10 years, 9 months ago (2010-03-23 17:41:18 UTC) #3
lgtm

Powered by Google App Engine