Theme the text of the download item so it's readable with dark themes....

Theme the text of the download item so it's readable with dark themes.

BUG=http://crbug.com/21559
TEST=as in bug

[Nico, 2009-11-06 23:29:36]

LGTM if the UI folks are ok with us looking different from win / linux here.

[Nico, 2009-11-08 09:37:14]

http://codereview.chromium.org/377015/diff/1/2
File chrome/browser/cocoa/download_item_controller.mm (right):

http://codereview.chromium.org/377015/diff/1/2#newcode90
Line 90: object:nil];
Oh, and you need to removeObserver:self in dealloc. That's important.

[Avi (use Gerrit), 2009-11-08 17:44:31]

UX comment? Just theming text to make it readable.

[Avi (use Gerrit), 2009-11-08 17:45:26]

http://codereview.chromium.org/377015/diff/1/2
File chrome/browser/cocoa/download_item_controller.mm (right):

http://codereview.chromium.org/377015/diff/1/2#newcode90
Line 90: object:nil];
On 2009/11/08 09:37:14, Nico wrote:
> Oh, and you need to removeObserver:self in dealloc. That's important.

Um, I already _do_. Well, _I_ don't, but it's already there so it doesn't need
to be added. Look at line 100.

[Avi (use Gerrit), 2009-11-09 19:49:55]

Cole says it's OK for now so we can see the warning. If we want to do some
lightening of the background that'd be great.