Roll grit to r136.

Roll grit to r136.

BUG=None

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

[flackr, 2013-09-04 20:47:38]

Please review. Rolling grit in chromium to pull in your change.

[Rafał Chłodnicki, 2013-09-04 21:25:31]

lgtm

Failures on Windows look grit related but not sure if have anything to do with
my fix.

[commit-bot: I haz the power, 2013-09-05 16:20:20]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a lowly provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[flackr, 2013-09-05 16:34:47]

+joi, looks like the initial win bot failure was a flake.

[Jói, 2013-09-05 17:06:53]

Yeah, looks fine. LGTM

On Thu, Sep 5, 2013 at 6:34 PM,  <flackr@chromium.org> wrote:
> +joi, looks like the initial win bot failure was a flake.
>
> https://codereview.chromium.org/23658022/

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[commit-bot: I haz the power, 2013-09-05 17:09:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/flackr@chromium.org/23658022/1

[commit-bot: I haz the power, 2013-09-05 17:36:21]

Sorry for I got bad news for ya.
Compile failed with a clobber build on win_x64_rel.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_x64_re...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[Rafał Chłodnicki, 2013-09-05 17:41:04]

On 2013/09/05 17:36:21, I haz the power (commit-bot) wrote:
> Sorry for I got bad news for ya.
> Compile failed with a clobber build on win_x64_rel.
>
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_x64_re...
> Your code is likely broken or HEAD is junk. Please ensure your
> code is not broken then alert the build sheriffs.
> Look at the try server FAQ for more details.

Looks like caused by http://code.google.com/p/grit-i18n/source/detail?r=134
chrome/tools/build/win/make_policy_zip.py needs updating.

[flackr, 2013-09-05 18:48:41]

On 2013/09/05 17:41:04, Rafał Chłodnicki wrote:
> On 2013/09/05 17:36:21, I haz the power (commit-bot) wrote:
> > Sorry for I got bad news for ya.
> > Compile failed with a clobber build on win_x64_rel.
> >
>
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_x64_re...
> > Your code is likely broken or HEAD is junk. Please ensure your
> > code is not broken then alert the build sheriffs.
> > Look at the try server FAQ for more details.
> 
> Looks like caused by http://code.google.com/p/grit-i18n/source/detail?r=134
> chrome/tools/build/win/make_policy_zip.py needs updating.

Commented on the other code review: https://codereview.chromium.org/22923027/.
grd_reader uses the system platform if target_platform isn't passed,
grit_info.Outputs probably should do the same..

[commit-bot: I haz the power, 2013-09-06 05:02:24]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/flackr@chromium.org/23658022/22001

[commit-bot: I haz the power, 2013-09-06 07:36:31]

Change committed as 221628