check_dups: Add a script to check for the same resource in multiple pak files.

check_dups: Add a script to check for the same resource in multiple pak files.

The usage is:
  check_dups.py first.pak second.pak ...

The script looks at each resource pack, and throws an exception if a resource
with the same id exists in more than one resource pack. This can be useful to
avoid accidentally increasing the install-size.

BUG=tbd

[sadrul, 2015-04-02 22:17:29]

sadrul@chromium.org changed reviewers:
+ mnissler@chromium.org

[sadrul, 2015-04-02 22:17:29]

Hi! The corresponding CL in chrome that uses this script:
https://codereview.chromium.org/1055153002/

The tests run OK for me locally. Are there trybots I can run the patch on?

[Mattias Nissler (ping if slow), 2015-04-07 10:12:38]

mnissler@chromium.org changed reviewers:
+ thakis@chromium.org

[Mattias Nissler (ping if slow), 2015-04-07 10:12:38]

Passing this on to thakis@, who's a much more competent reviewer for this CL.

https://codereview.chromium.org/1056993003/diff/80001/grit/format/check_dups.py
File grit/format/check_dups.py (right):

https://codereview.chromium.org/1056993003/diff/80001/grit/format/check_dups....
grit/format/check_dups.py:18: 
nit: two blank lines between top-levels (here and elsewhere)

https://codereview.chromium.org/1056993003/diff/80001/grit/format/check_dups....
grit/format/check_dups.py:36: def main():
Hm, a new program doesn't really fit with grit's command abstraction.

[sadrul, 2015-04-07 20:09:30]

https://codereview.chromium.org/1056993003/diff/80001/grit/format/check_dups.py
File grit/format/check_dups.py (right):

https://codereview.chromium.org/1056993003/diff/80001/grit/format/check_dups....
grit/format/check_dups.py:18: 
On 2015/04/07 10:12:38, Mattias Nissler wrote:
> nit: two blank lines between top-levels (here and elsewhere)

Done.

https://codereview.chromium.org/1056993003/diff/80001/grit/format/check_dups....
grit/format/check_dups.py:36: def main():
On 2015/04/07 10:12:38, Mattias Nissler wrote:
> Hm, a new program doesn't really fit with grit's command abstraction.

I am not very familiar with grit. Are there existing patterns of usage I should
look at to make this more consistent with current usage?

[Mattias Nissler (ping if slow), 2015-04-08 08:29:52]

https://codereview.chromium.org/1056993003/diff/80001/grit/format/check_dups.py
File grit/format/check_dups.py (right):

https://codereview.chromium.org/1056993003/diff/80001/grit/format/check_dups....
grit/format/check_dups.py:36: def main():
On 2015/04/07 20:09:30, sadrul wrote:
> On 2015/04/07 10:12:38, Mattias Nissler wrote:
> > Hm, a new program doesn't really fit with grit's command abstraction.
> 
> I am not very familiar with grit. Are there existing patterns of usage I
should
> look at to make this more consistent with current usage?

Let's wait and hear what thakis@ says.

[sadrul, 2015-04-09 18:56:48]

> Let's wait and hear what thakis@ says.

thakis@ ping

[Nico, 2015-04-09 22:26:18]

I'm mostly away until next week wednesday. We already have a bunch of similar
scripts in the Cr repo (tools/check_grd_for_unused_strings.py,
tools/resources/*). Maybe it should go there?

[sadrul, 2015-05-13 18:51:35]

This is no longer useful since https://codereview.chromium.org/1115033003/ has
landed.