First pass at NNTP theming....

Fix a bunch of bugs related to NNTP theming.

Basically: 
- Add NNTP Theming

BrowserThemeProvider:
- Add NNTP logo inversion capability
- Only allow a set of images to be themed.
- Tint incognito windows correctly.
(Sorry for the feature-mush)

BUG=17379, 14578, 17593
TEST=Make sure the NNTP is themed correctly.


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

[arv (Not doing code reviews), 2009-07-24 00:25:12]

http://codereview.chromium.org/159305/diff/78/1042
File chrome/browser/browser_theme_provider.cc (right):

http://codereview.chromium.org/159305/diff/78/1042#newcode599
Line 599: if (display_properties_value->GetString(*iter, &val))
Do you intend to submit the commented out code?

http://codereview.chromium.org/159305/diff/78/1042#newcode787
Line 787: int blur_amount = (HasCustomImage(id)) ? 0 : 0;
0 or... 0! Is this a bug?

http://codereview.chromium.org/159305/diff/78/1037
File chrome/browser/resources/new_tab_theme.css (right):

http://codereview.chromium.org/159305/diff/78/1037#newcode5
Line 5: background-repeat:$5;
It would have been nice if we could have named these.

http://codereview.chromium.org/159305/diff/78/1037#newcode66
Line 66: background-color: hsla(213, 0%, 100%, .95);
This would probably be clearer as rgba(255, 255, 255, .95) since it is really
white with some alpha.

Why is this in the theme css?

[Glen Murphy, 2009-07-24 02:16:24]

http://codereview.chromium.org/159305/diff/78/1042
File chrome/browser/browser_theme_provider.cc (right):

http://codereview.chromium.org/159305/diff/78/1042#newcode599
Line 599: if (display_properties_value->GetString(*iter, &val))
On 2009/07/24 00:25:14, arv wrote:
> Do you intend to submit the commented out code?

Fixed.

http://codereview.chromium.org/159305/diff/78/1042#newcode787
Line 787: int blur_amount = (HasCustomImage(id)) ? 0 : 0;
On 2009/07/24 00:25:14, arv wrote:
> 0 or... 0! Is this a bug?

my mistake. fixed.

http://codereview.chromium.org/159305/diff/78/1037
File chrome/browser/resources/new_tab_theme.css (right):

http://codereview.chromium.org/159305/diff/78/1037#newcode5
Line 5: background-repeat:$5;
On 2009/07/24 00:25:14, arv wrote:
> It would have been nice if we could have named these.

It's being tracked; Miranda has a bug on file about it.

http://codereview.chromium.org/159305/diff/78/1037#newcode66
Line 66: background-color: hsla(213, 0%, 100%, .95);
On 2009/07/24 00:25:14, arv wrote:
> This would probably be clearer as rgba(255, 255, 255, .95) since it is really
> white with some alpha.
> 
> Why is this in the theme css?

User error

[arv (Not doing code reviews), 2009-07-24 02:20:33]

LGTM