components: Provide an unified target to generate components_tests_resources.pak.

components: Provide an unified target to generate components_tests_resources.pak.

Instead of having separate actions for each test suite in components, provide a single target that can be used by both test targets.

BUG=451827
R=blundell@chromium.org,jam@chromium.org
TEST=components_unittests && components_browsertests

Committed: https://crrev.com/4bb3f4788c97127aee4d05efed3cec261c43cfbd
Cr-Commit-Position: refs/heads/master@{#315387}

[tfarina, 2015-02-05 17:16:57]

PTAL!

[jam, 2015-02-05 21:24:57]

lgtm

[blundell, 2015-02-06 08:25:31]

lgtm

https://codereview.chromium.org/900223003/diff/1/components/dom_distiller/sta...
File components/dom_distiller/standalone/content_extractor.cc (right):

https://codereview.chromium.org/900223003/diff/1/components/dom_distiller/sta...
components/dom_distiller/standalone/content_extractor.cc:1: // Copyright 2014
The Chromium Authors. All rights reserved.
Could you file a bug against cjhopman to move this file to
content_extractor_browsertest.cc? As it is, I had to look to make sure this
wasn't production code that would now be hardcoding a reference to the test
resources.

https://codereview.chromium.org/900223003/diff/1/components/dom_distiller/sta...
components/dom_distiller/standalone/content_extractor.cc:162: void
AddComponentsResources() {
s/AddComponentsResources/AddComponentsTestResources

[tfarina, 2015-02-09 18:44:03]

New patchsets have been uploaded after l-g-t-m from
jam@chromium.org,blundell@chromium.org

[tfarina, 2015-02-09 18:44:13]

https://codereview.chromium.org/900223003/diff/1/components/dom_distiller/sta...
File components/dom_distiller/standalone/content_extractor.cc (right):

https://codereview.chromium.org/900223003/diff/1/components/dom_distiller/sta...
components/dom_distiller/standalone/content_extractor.cc:1: // Copyright 2014
The Chromium Authors. All rights reserved.
On 2015/02/06 08:25:31, blundell wrote:
> Could you file a bug against cjhopman to move this file to
> content_extractor_browsertest.cc? As it is, I had to look to make sure this
> wasn't production code that would now be hardcoding a reference to the test
> resources.

Done. https://crbug.com/456886

https://codereview.chromium.org/900223003/diff/1/components/dom_distiller/sta...
components/dom_distiller/standalone/content_extractor.cc:162: void
AddComponentsResources() {
On 2015/02/06 08:25:31, blundell wrote:
> s/AddComponentsResources/AddComponentsTestResources

Done.

[tfarina, 2015-02-09 18:44:29]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2015-02-09 18:44:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/900223003/20001

[commit-bot: I haz the power, 2015-02-09 21:00:13]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-02-09 21:00:42]

Patchset 2 (id:??) landed as
https://crrev.com/4bb3f4788c97127aee4d05efed3cec261c43cfbd
Cr-Commit-Position: refs/heads/master@{#315387}