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

Issue 2866963002: mac: Set background color from theme before NSView is added to window (Closed)

Created:
3 years, 7 months ago by ccameron
Modified:
3 years, 7 months ago
Reviewers:
Avi (use Gerrit)
CC:
chromium-reviews, mac-reviews_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

mac: Set background color from theme before NSView is added to window Update the color based on the window theme before the view is added to the window, not after. Fix the color for incognito windows (added a TODO and filed a bug because some refactoring will be needed there). BUG=470669 Review-Url: https://codereview.chromium.org/2866963002 Cr-Commit-Position: refs/heads/master@{#469903} Committed: https://chromium.googlesource.com/chromium/src/+/3209ebcc6b2239b1f9cb806665d60a1096fcf53b

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+22 lines, -8 lines) Patch
M chrome/browser/ui/cocoa/tab_contents/tab_contents_controller.mm View 5 chunks +22 lines, -8 lines 0 comments Download

Dependent Patchsets:

Messages

Total messages: 12 (8 generated)
ccameron
ptal -- should have been obvious, but there were lots of other white flash sources ...
3 years, 7 months ago (2017-05-07 17:45:04 UTC) #4
Avi (use Gerrit)
lgtm
3 years, 7 months ago (2017-05-07 17:48:37 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2866963002/1
3 years, 7 months ago (2017-05-07 18:26:49 UTC) #9
commit-bot: I haz the power
3 years, 7 months ago (2017-05-07 18:32:01 UTC) #12
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/3209ebcc6b2239b1f9cb806665d6...

Powered by Google App Engine
This is Rietveld 408576698