Add HiDPI webstore icons. Use them on NTP and extensions domui.

Add HiDPI webstore icons. Use them on NTP and extensions domui.

BUG=134372
TEST=none
TBR=sky

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=145332

[flackr, 2012-06-30 11:45:03]

LGTM

http://codereview.chromium.org/10704053/diff/1/chrome/browser/browser_resourc...
File chrome/browser/browser_resources.grd (right):

http://codereview.chromium.org/10704053/diff/1/chrome/browser/browser_resourc...
chrome/browser/browser_resources.grd:25: <structure
name="IDR_NEW_TAB_SEARCH_THEME_CSS"
file="resources\ntp_search\new_tab_theme.css" flattenhtml="true"
type="chrome_html" />
At some point we should move the rest of the html/css/js here as I'd previously
done with options and shared resources. There should be no regressions, and then
we can just land resources as we get them.

[Nico, 2012-06-30 15:24:17]

Does the grit exception on all the trybots look familiar to you?

Any thoughts on http://code.google.com/p/chromium/issues/detail?id=134988 ?  I'm
concerned that whatever causes that might affect this CL too.

[flackr, 2012-06-30 15:27:28]

+benrg, looks like it's failing on chrome_scaled_image, have you seen this
before?

On 2012/06/30 15:24:17, Nico wrote:
> Does the grit exception on all the trybots look familiar to you?
> 
> Any thoughts on http://code.google.com/p/chromium/issues/detail?id=134988 ? 
I'm
> concerned that whatever causes that might affect this CL too.

[flackr, 2012-06-30 15:27:28]

+benrg, looks like it's failing on chrome_scaled_image, have you seen this
before?

On 2012/06/30 15:24:17, Nico wrote:
> Does the grit exception on all the trybots look familiar to you?
> 
> Any thoughts on http://code.google.com/p/chromium/issues/detail?id=134988 ? 
I'm
> concerned that whatever causes that might affect this CL too.

[oshima, 2012-06-30 15:37:52]

I'm guessing that this is because trybot can't handle png file changes?


On 2012/06/30 15:27:28, flackr wrote:
> +benrg, looks like it's failing on chrome_scaled_image, have you seen this
> before?
> 
> On 2012/06/30 15:24:17, Nico wrote:
> > Does the grit exception on all the trybots look familiar to you?
> > 
> > Any thoughts on http://code.google.com/p/chromium/issues/detail?id=134988 ? 
> I'm
> > concerned that whatever causes that might affect this CL too.

[flackr, 2012-06-30 15:40:51]

Ah yes of course! Sorry I completely overlooked that. I usually land assets
ahead of the CL when I want to use the CQ / trybots.

On 2012/06/30 15:37:52, oshima wrote:
> I'm guessing that this is because trybot can't handle png file changes?
> 
> 
> On 2012/06/30 15:27:28, flackr wrote:
> > +benrg, looks like it's failing on chrome_scaled_image, have you seen this
> > before?
> > 
> > On 2012/06/30 15:24:17, Nico wrote:
> > > Does the grit exception on all the trybots look familiar to you?
> > > 
> > > Any thoughts on http://code.google.com/p/chromium/issues/detail?id=134988
? 
> > I'm
> > > concerned that whatever causes that might affect this CL too.

[benrg, 2012-07-02 18:37:17]

The failure in _MakeBraceGlob (which is supposed to produce a comprehensible
error message) is a bug in my code. Thanks for pointing this out.

[commit-bot: I haz the power, 2012-07-03 17:42:07]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/thakis@chromium.org/10704053/7002

[commit-bot: I haz the power, 2012-07-03 17:42:11]

Presubmit check for 10704053-7002 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    chrome/browser/history

[Nico, 2012-07-03 17:42:43]

sky: OWNERS

[commit-bot: I haz the power, 2012-07-03 17:42:59]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/thakis@chromium.org/10704053/7002