Remove make-css-file-arrays.pl.

Remove make-css-file-arrays.pl.

Use make-file-arrays.py instead. 
Now we don't need perl to build Blink.  We still need perl to run layout
tests.

- We remove action_useragentstylesheets.py wrapper too.
- make-file-arrays.py supports file names including '-' for view-source.css.
- make-file-arrays.py removes unnecessary EOL characters in order to make the
  behaivor same as make-css-file-arrays.pl.
- Change generated variable names for consistency.

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

[tkent, 2014-06-30 07:09:17]

brettw, jamesr,
This contains GN changes.
Would you review this please?

https://codereview.chromium.org/361693004/diff/20001/Source/build/scripts/mak...
File Source/build/scripts/make-file-arrays.py (left):

https://codereview.chromium.org/361693004/diff/20001/Source/build/scripts/mak...
Source/build/scripts/make-file-arrays.py:65: # Don't accidentally match URLs
(http://...)
This is a bogus comment.  leftover of
http://src.chromium.org/viewvc/blink?view=revision&revision=172748

[brettw, 2014-06-30 16:47:50]

.gn LGTM

[tkent, 2014-06-30 23:26:42]

Brett, thank you!.

haraken, would you review non-GN part?

[haraken, 2014-07-01 00:47:46]

LGTM with nits.

https://codereview.chromium.org/361693004/diff/20001/Source/build/scripts/mak...
File Source/build/scripts/make-file-arrays.py (right):

https://codereview.chromium.org/361693004/diff/20001/Source/build/scripts/mak...
Source/build/scripts/make-file-arrays.py:70: trailing_space = re.compile(r'[
\t]+$', re.MULTILINE)

Shall we use [\s\t]+ for all of repeating_space, leading_space and
trailing_space consistently?

https://codereview.chromium.org/361693004/diff/20001/Source/core/BUILD.gn
File Source/core/BUILD.gn (right):

https://codereview.chromium.org/361693004/diff/20001/Source/core/BUILD.gn#new...
Source/core/BUILD.gn:1037: script = "../build/scripts/make-file-arrays.py"

This should be make_file_arrays.py.

https://codereview.chromium.org/361693004/diff/20001/Source/core/css/CSSDefau...
File Source/core/css/CSSDefaultStyleSheets.cpp (right):

https://codereview.chromium.org/361693004/diff/20001/Source/core/css/CSSDefau...
Source/core/css/CSSDefaultStyleSheets.cpp:107: String defaultRules =
String(htmlCss, sizeof(htmlCss)) +
RenderTheme::theme().extraDefaultStyleSheet();

htmlCss => htmlCSS

The same comment for other parts in this file.

[tkent, 2014-07-01 00:52:49]

Thank you for review.

https://codereview.chromium.org/361693004/diff/20001/Source/core/BUILD.gn
File Source/core/BUILD.gn (right):

https://codereview.chromium.org/361693004/diff/20001/Source/core/BUILD.gn#new...
Source/core/BUILD.gn:1037: script = "../build/scripts/make-file-arrays.py"
On 2014/07/01 00:47:45, haraken wrote:
> 
> This should be make_file_arrays.py.

I don't think so.

https://codereview.chromium.org/361693004/diff/20001/Source/core/css/CSSDefau...
File Source/core/css/CSSDefaultStyleSheets.cpp (right):

https://codereview.chromium.org/361693004/diff/20001/Source/core/css/CSSDefau...
Source/core/css/CSSDefaultStyleSheets.cpp:107: String defaultRules =
String(htmlCss, sizeof(htmlCss)) +
RenderTheme::theme().extraDefaultStyleSheet();
On 2014/07/01 00:47:46, haraken wrote:
> 
> htmlCss => htmlCSS
> 
> The same comment for other parts in this file.

This is the existing naming rule of make-file-arrays.py.  Fixing it is not in
the scope of this CL.

[haraken, 2014-07-01 01:00:00]

https://codereview.chromium.org/361693004/diff/20001/Source/core/BUILD.gn
File Source/core/BUILD.gn (right):

https://codereview.chromium.org/361693004/diff/20001/Source/core/BUILD.gn#new...
Source/core/BUILD.gn:1037: script = "../build/scripts/make-file-arrays.py"
On 2014/07/01 00:52:49, tkent wrote:
> On 2014/07/01 00:47:45, haraken wrote:
> > 
> > This should be make_file_arrays.py.
> 
> I don't think so.

All other files in build/scripts/ have xxx_yyy.py instead of xxx-yyy.py. You can
fix it in a follow-up.

[tkent, 2014-07-01 01:05:46]

https://codereview.chromium.org/361693004/diff/20001/Source/build/scripts/mak...
File Source/build/scripts/make-file-arrays.py (right):

https://codereview.chromium.org/361693004/diff/20001/Source/build/scripts/mak...
Source/build/scripts/make-file-arrays.py:70: trailing_space = re.compile(r'[
\t]+$', re.MULTILINE)
On 2014/07/01 00:47:45, haraken wrote:
> 
> Shall we use [\s\t]+ for all of repeating_space, leading_space and
> trailing_space consistently?

Done.

https://codereview.chromium.org/361693004/diff/20001/Source/core/BUILD.gn
File Source/core/BUILD.gn (right):

https://codereview.chromium.org/361693004/diff/20001/Source/core/BUILD.gn#new...
Source/core/BUILD.gn:1037: script = "../build/scripts/make-file-arrays.py"
On 2014/07/01 00:59:59, haraken wrote:
> On 2014/07/01 00:52:49, tkent wrote:
> > On 2014/07/01 00:47:45, haraken wrote:
> > > 
> > > This should be make_file_arrays.py.
> > 
> > I don't think so.
> 
> All other files in build/scripts/ have xxx_yyy.py instead of xxx-yyy.py. You
can
> fix it in a follow-up.

yeah, it's not in the scope of this CL.

[tkent, 2014-07-01 01:07:01]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2014-07-01 01:08:15]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/tkent@chromium.org/361693004/40001

[commit-bot: I haz the power, 2014-07-01 02:42:28]

Change committed as 177264