Migrate all usages of xxd.py to blink_resources.grd and remove xxd.py - Part 1

Migrate all usages of xxd.py to blink_resources.grd and remove xxd.py - Part 1

Blink should make use of blink_resources.grd for the inline resources of inspector.
This removes the dependency upon using xxd.py which embeds these resources as strings.

Part 2: https://crrev.com/403273003
Part 3: https://crrev.com/402033002

BUG=312586
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=178832

[vivekg, 2014-07-21 09:23:06]

PTAL, thank you!

[jochen (gone - plz use gerrit), 2014-07-21 11:28:38]

lgtm

[vivekg, 2014-07-21 11:29:08]

The CQ bit was checked by vivekg@chromium.org

[commit-bot: I haz the power, 2014-07-21 11:30:17]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/vivek.vg@samsung.com/401233003/1

[vivekg, 2014-07-21 11:59:13]

The CQ bit was unchecked by vivekg@chromium.org

[vivekg, 2014-07-21 11:59:52]

I think for inspector related changes, we could modify
WebKit/Source/devtools/scripts/generate_devtools_grd.py instead of
blink_resources.grd.
Will update the patch accordingly.

[vivekg, 2014-07-21 14:59:46]

On 2014/07/21 11:59:52, vivekg_ wrote:
> I think for inspector related changes, we could modify
> WebKit/Source/devtools/scripts/generate_devtools_grd.py instead of
> blink_resources.grd.
> Will update the patch accordingly.


The generate_devtools_grd.py is mostly related to the devtools frontend files.
The files changed by this CL are independent of them. So I think its ok for them
to be part of blink_resources.grd.

@pfeldman, WDYT?

[vivekg, 2014-07-22 11:16:37]

Friendly ping!

[jochen (gone - plz use gerrit), 2014-07-22 11:27:07]

On 2014/07/22 at 11:16:37, vivekg wrote:
> Friendly ping!

I defer to Pavel on this question

[vivekg, 2014-07-24 07:28:39]

On 2014/07/22 11:27:07, jochen wrote:
> On 2014/07/22 at 11:16:37, vivekg wrote:
> > Friendly ping!
> 
> I defer to Pavel on this question

+yurys as well.

[pfeldman, 2014-07-24 07:50:43]

lgtm

https://codereview.chromium.org/401233003/diff/1/public/blink_resources.grd
File public/blink_resources.grd (right):

https://codereview.chromium.org/401233003/diff/1/public/blink_resources.grd#n...
public/blink_resources.grd:15: <include
name="IDR_INJECTED_SCRIPT_CANVAS_MODULE_SOURCE"
file="../Source/core/inspector/InjectedScriptCanvasModuleSource.js"
type="BINDATA"/>
Lets prefix all consts with inspector?
IDR_INSPECTOR_INJECTED_SCRIPT_*

[vivekg, 2014-07-24 09:49:18]

On 2014/07/24 07:50:43, pfeldman wrote:
> lgtm
> 
> https://codereview.chromium.org/401233003/diff/1/public/blink_resources.grd
> File public/blink_resources.grd (right):
> 
>
https://codereview.chromium.org/401233003/diff/1/public/blink_resources.grd#n...
> public/blink_resources.grd:15: <include
> name="IDR_INJECTED_SCRIPT_CANVAS_MODULE_SOURCE"
> file="../Source/core/inspector/InjectedScriptCanvasModuleSource.js"
> type="BINDATA"/>
> Lets prefix all consts with inspector?
> IDR_INSPECTOR_INJECTED_SCRIPT_*

Done. Thank you!

[vivekg, 2014-07-24 09:49:31]

The CQ bit was checked by vivekg@chromium.org

[commit-bot: I haz the power, 2014-07-24 09:50:23]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/vivek.vg@samsung.com/401233003/20001

[commit-bot: I haz the power, 2014-07-24 10:56:45]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  mac_gpu_retina_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu_retina_tr...)

[commit-bot: I haz the power, 2014-07-24 11:22:04]

Change committed as 178832