Avoid duplicating the target name "svg".

Avoid duplicating the target name "svg".

It has not been possible to build just the svg code with
ninja -C out/Default svg
because of two other, minor, build targets named svg.
This commit renames those svg_layout and svg_style.

Review-Url: https://codereview.chromium.org/2783253002
Cr-Commit-Position: refs/heads/master@{#461073}
Committed: https://chromium.googlesource.com/chromium/src/+/e4218f89ae50b22efd8c98b97f57fa138a4dfe19

[Daniel Bratell, 2017-03-30 16:08:34]

The CQ bit was checked by bratell@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2017-03-30 16:09:19]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Daniel Bratell, 2017-03-30 16:11:11]

bratell@opera.com changed reviewers:
+ fs@opera.com, pdr@chromium.org

[Daniel Bratell, 2017-03-30 16:11:12]

PTAL

[pdr., 2017-03-30 17:50:49]

On 2017/03/30 at 16:11:12, bratell wrote:
> PTAL

LGTM if it works.

Can you explain the usecase for this? What are you building just svg for?

[commit-bot: I haz the power, 2017-03-30 17:50:56]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-30 17:50:56]

Dry run: This issue passed the CQ dry run.

[Daniel Bratell, 2017-03-31 07:51:10]

On 2017/03/30 17:50:49, pdr. wrote:
> On 2017/03/30 at 16:11:12, bratell wrote:
> > PTAL
> 
> LGTM if it works.
> 
> Can you explain the usecase for this? What are you building just svg for?

I'm still experimenting with unity builds and when I mentioned to fs that I
could easily build just "html" or "layout" or "style", "svg" never work he told
me that yes, that was how it was and you have to write the whole path if you
just want to test compile the svg files and that seemed unnecessary to me.

The usecase would be that you have changed some svg code and quickly want to
check that it compiles and don't want to wait for a full build (which can be an
hour or more) before moving on. It also makes me feel better to not have name
collisions in the build system. :-)

[Daniel Bratell, 2017-03-31 07:51:44]

The CQ bit was checked by bratell@opera.com

[commit-bot: I haz the power, 2017-03-31 07:51:58]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-31 07:56:49]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490946704073370, "parent_rev":
"6fbccafc5d50e35d8f8360559829a8b5ece100dc", "commit_rev":
"e4218f89ae50b22efd8c98b97f57fa138a4dfe19"}

[commit-bot: I haz the power, 2017-03-31 07:57:40]

Description was changed from

==========
Avoid duplicating the target name "svg".

It has not been possible to build just the svg code with
ninja -C out/Default svg
because of two other, minor, build targets named svg.
This commit renames those svg_layout and svg_style.
==========

to

==========
Avoid duplicating the target name "svg".

It has not been possible to build just the svg code with
ninja -C out/Default svg
because of two other, minor, build targets named svg.
This commit renames those svg_layout and svg_style.

Review-Url: https://codereview.chromium.org/2783253002
Cr-Commit-Position: refs/heads/master@{#461073}
Committed:
https://chromium.googlesource.com/chromium/src/+/e4218f89ae50b22efd8c98b97f57...
==========

[commit-bot: I haz the power, 2017-03-31 07:57:42]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/e4218f89ae50b22efd8c98b97f57...