muzidudu / grit-i18n

Automatically exported from code.google.com/p/grit-i18n
0 stars 0 forks source link

Warn about duplicate resource inclusion in the PAK files #27

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
It would be nice if grit warned about duplicate files in the PAK files 
(included normally or through html_inline).

For example Chrome for Android has the following duplicates now:

Count   | File
--------------------------------------------------------------------------------
-------------------------------------
      2 [ /usr/local/google/ssd1/clankium/src/chrome/app/theme/default_100_percent/common/error_network_generic.png
      2 [ /usr/local/google/ssd1/clankium/src/chrome/app/theme/default_100_percent/common/error_network_offline.png
      2 [ /usr/local/google/ssd1/clankium/src/chrome/app/theme/default_100_percent/extensions_section.png
      2 [ /usr/local/google/ssd1/clankium/src/chrome/app/theme/default_200_percent/extensions_section.png
      2 [ /usr/local/google/ssd1/clankium/src/chrome/browser/resources/about_welcome_android/images/holo_logo_150.png
      2 [ /usr/local/google/ssd1/clankium/src/chrome/browser/resources/ntp_android/images/default_thumbnail.png
      2 [ /usr/local/google/ssd1/clankium/src/chrome/browser/resources/ssl/roadblock_background.png
      2 [ /usr/local/google/ssd1/clankium/src/chrome/browser/resources/ssl/ssl_errors_common.js
      2 [ /usr/local/google/ssd1/clankium/src/chrome/browser/resources/uber/uber_shared.css
      2 [ /usr/local/google/ssd1/clankium/src/chrome/renderer/resources/neterror.css
      2 [ /usr/local/google/ssd1/clankium/src/chrome/renderer/resources/neterror.js
      2 [ /usr/local/google/ssd1/clankium/src/chrome/renderer/resources/plugin_blocked.png
      2 [ /usr/local/google/ssd1/clankium/src/third_party/jstemplate/jstemplate_compiled.js
      2 [ /usr/local/google/ssd1/clankium/src/ui/resources/default_100_percent/common/folder_closed.png
      2 [ /usr/local/google/ssd1/clankium/src/ui/resources/default_100_percent/common/folder_closed_rtl.png
      2 [ /usr/local/google/ssd1/clankium/src/ui/webui/resources/images/star_small.png
      2 [ /usr/local/google/ssd1/clankium/src/ui/webui/resources/js/cr.js
      2 [ /usr/local/google/ssd1/clankium/src/ui/webui/resources/js/i18n_template_no_process.js
      3 [ /usr/local/google/ssd1/clankium/src/chrome/browser/resources/disclosure_triangle_small.png
      3 [ /usr/local/google/ssd1/clankium/src/chrome/browser/resources/ssl/images/roadblock.png
      3 [ /usr/local/google/ssd1/clankium/src/chrome/browser/resources/uber/uber_utils.js
      3 [ /usr/local/google/ssd1/clankium/src/chrome/renderer/resources/plugin_placeholders.css
      3 [ /usr/local/google/ssd1/clankium/src/ui/resources/default_200_percent/close_2_hover.png
      3 [ /usr/local/google/ssd1/clankium/src/ui/resources/default_200_percent/close_2.png
      4 [ /usr/local/google/ssd1/clankium/src/ui/resources/default_100_percent/close_2_hover.png
      4 [ /usr/local/google/ssd1/clankium/src/ui/resources/default_100_percent/close_2.png
      4 [ /usr/local/google/ssd1/clankium/src/ui/resources/default_100_percent/close_2_pressed.png
      4 [ /usr/local/google/ssd1/clankium/src/ui/resources/default_200_percent/close_2_pressed.png
      4 [ /usr/local/google/ssd1/clankium/src/ui/webui/resources/images/select.png
      6 [ /usr/local/google/ssd1/clankium/src/chrome/browser/resources/about_welcome_android/images/holo_logo_200.png
      6 [ /usr/local/google/ssd1/clankium/src/ui/webui/resources/js/util.js
      9 [ /usr/local/google/ssd1/clankium/src/ui/webui/resources/js/assert.js

Original issue reported on code.google.com by auri...@chromium.org on 3 Mar 2014 at 7:21