PPAPI Testing: Clean up location of ppapi::StripTestPrefixes

PPAPI Testing: Clean up location of ppapi::StripTestPrefixes

BUG=478309
Committed: https://crrev.com/a88b609ba9a3df0f5cb9950a0564c9ee463f6f69
Cr-Commit-Position: refs/heads/master@{#329874}

[tommycli, 2015-05-13 22:38:38]

The CQ bit was checked by tommycli@chromium.org to run a CQ dry run

[tommycli, 2015-05-13 22:38:50]

tommycli@chromium.org changed reviewers:
+ phajdan.jr@chromium.org, raymes@chromium.org

[commit-bot: I haz the power, 2015-05-13 22:40:17]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1140953003/1

[tommycli, 2015-05-13 22:40:28]

phajdan.jr, raymes: PTAL small CL thanks!

[commit-bot: I haz the power, 2015-05-13 22:48:44]

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

[commit-bot: I haz the power, 2015-05-13 22:48:45]

Dry run: Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[tommycli, 2015-05-14 00:09:53]

The CQ bit was checked by tommycli@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-05-14 00:11:14]

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

[commit-bot: I haz the power, 2015-05-14 01:11:41]

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

[commit-bot: I haz the power, 2015-05-14 01:11:42]

Dry run: This issue passed the CQ dry run.

[raymes, 2015-05-14 01:53:47]

https://codereview.chromium.org/1140953003/diff/20001/ppapi/tests/test_case.cc
File ppapi/tests/test_case.cc (right):

https://codereview.chromium.org/1140953003/diff/20001/ppapi/tests/test_case.c...
ppapi/tests/test_case.cc:20: std::string StripPrefix(const std::string&
test_name) {
Can we remove this function and just call the version in
ppapi/shared_impl/test_utils.cc?

[raymes, 2015-05-14 01:55:01]

lgtm

https://codereview.chromium.org/1140953003/diff/20001/ppapi/tests/test_case.cc
File ppapi/tests/test_case.cc (right):

https://codereview.chromium.org/1140953003/diff/20001/ppapi/tests/test_case.c...
ppapi/tests/test_case.cc:20: std::string StripPrefix(const std::string&
test_name) {
On 2015/05/14 01:53:47, raymes wrote:
> Can we remove this function and just call the version in
> ppapi/shared_impl/test_utils.cc?

Ahh we can't do this because of DEPS.

[Paweł Hajdan Jr., 2015-05-14 10:05:50]

LGTM

[tommycli, 2015-05-14 17:44:23]

https://codereview.chromium.org/1140953003/diff/20001/ppapi/tests/test_case.cc
File ppapi/tests/test_case.cc (right):

https://codereview.chromium.org/1140953003/diff/20001/ppapi/tests/test_case.c...
ppapi/tests/test_case.cc:20: std::string StripPrefix(const std::string&
test_name) {
On 2015/05/14 01:55:01, raymes wrote:
> On 2015/05/14 01:53:47, raymes wrote:
> > Can we remove this function and just call the version in
> > ppapi/shared_impl/test_utils.cc?
> 
> Ahh we can't do this because of DEPS.

Yes, I tried very hard. (see PS1 heh)

[tommycli, 2015-05-14 17:44:29]

The CQ bit was checked by tommycli@chromium.org

[commit-bot: I haz the power, 2015-05-14 17:44:55]

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

[commit-bot: I haz the power, 2015-05-14 17:54:09]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-05-14 17:54:59]

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