Cleanup: remove prefix chrome_ of chrome/browser/media/chrome* files

Cleanup: remove prefix chrome_ of chrome/browser/media/chrome* files

Note how the name of the tests inside the files
respect the new naming, and not the chrome_ prefixed.

Done in two moves:
$ for i in `ls chrome_*`; do git mv $i `echo $i | sed -e 's/chrome_//'`; done
$ tools/git/mass-rename.py

No new code whatsoever.

BUG=592104
Committed: https://crrev.com/d8863236eda7fe4a3bfae3bbc39d10bbf7be2a30
Cr-Commit-Position: refs/heads/master@{#380662}

[mcasas, 2016-03-04 21:48:11]

Description was changed from

==========
Cleanup: remove prefix chrome_ of chrome/browser/media/chrome* files

BUG=592104
==========

to

==========
Cleanup: remove prefix chrome_ of chrome/browser/media/chrome* files

Note how the name of the tests inside the files
respect the new naming, and not the chrome_ prefixed.

Done in two moves:
$ for i in `ls chrome_*`; do git mv $i `echo $i | sed -e 's/chrome_//'`; done
$ tools/git/mass-rename.py

No new code whatsoever.

BUG=592104
==========

[mcasas, 2016-03-04 21:48:37]

mcasas@chromium.org changed reviewers:
+ phoglund@chromium.org

[mcasas, 2016-03-04 21:48:38]

phoglund@ PTAL/Owners RS

[phoglund_chromium, 2016-03-07 10:48:19]

phoglund@chromium.org changed reviewers:
+ jam@chromium.org

[phoglund_chromium, 2016-03-07 10:48:20]

chrome_*_browsertest.cc used to be the convention for browser tests (as opposed
to content browsertests, which were just *_browsertest.cc). I can't see that
pattern anymore in codesearch, so I guess the convention has gone away. jam@, is
that correct?

If jam@ has no objections I'm on board with this.

[mcasas, 2016-03-08 21:36:04]

jam@ ping

[mcasas, 2016-03-09 16:57:49]

On 2016/03/08 21:36:04, mcasas wrote:
> jam@ ping

jam@ ping

[mcasas, 2016-03-10 17:37:43]

mcasas@chromium.org changed reviewers:
+ dalecurtis@chromium.org

[mcasas, 2016-03-10 17:37:44]

dalecurtis@ could you please answer phoglund@s micro question in #5 ?

jam@ is unresponsive

[DaleCurtis, 2016-03-10 17:47:22]

dalecurtis@chromium.org changed reviewers:
+ thakis@chromium.org

[DaleCurtis, 2016-03-10 17:47:23]

I can't speak for the chrome/ reviewers, so +thakis - My personal preference is
to not duplicate chrome_ in the file name since by virtue of being in chrome/ it
must be a chrome browsertest (I assume).

[Nico, 2016-03-10 18:12:18]

lgtm if you want to do this and chrome/browser/media people are onboard. I agree
the prefix is not necessary, but it also doesn't hurt that much and this adds a
bit of churn, so I'm +-0 on this I guess :-)

[mcasas, 2016-03-10 18:35:48]

On 2016/03/10 18:12:18, Nico wrote:
> lgtm if you want to do this and chrome/browser/media people are onboard. I
agree
> the prefix is not necessary, but it also doesn't hurt that much and this adds
a
> bit of churn, so I'm +-0 on this I guess :-)

phoglund@ says OK if other OK
dalecurtis@ says mild OK 
thakis@ says is happy if others are happy
==> I interpret this as a go-ahead :)

Also note that this convention is not followed elsewhere,
which means these files are effectively left behind.

[mcasas, 2016-03-10 19:40:13]

The CQ bit was checked by mcasas@chromium.org

[mcasas, 2016-03-10 19:40:14]

The patchset sent to the CQ was uploaded after l-g-t-m from thakis@chromium.org
Link to the patchset: https://codereview.chromium.org/1762413002/#ps20001
(title: "reran the commands")

[commit-bot: I haz the power, 2016-03-10 19:41:02]

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

[commit-bot: I haz the power, 2016-03-10 20:58:05]

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

[commit-bot: I haz the power, 2016-03-10 20:58:07]

Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[mcasas, 2016-03-10 21:20:54]

The CQ bit was checked by mcasas@chromium.org

[commit-bot: I haz the power, 2016-03-10 21:21:58]

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

[commit-bot: I haz the power, 2016-03-10 22:33:12]

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

[commit-bot: I haz the power, 2016-03-10 22:33:13]

Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[mcasas, 2016-03-10 23:11:53]

The CQ bit was checked by mcasas@chromium.org

[commit-bot: I haz the power, 2016-03-10 23:12:44]

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

[DaleCurtis, 2016-03-10 23:21:15]

lgtm

[commit-bot: I haz the power, 2016-03-11 00:21:03]

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

[commit-bot: I haz the power, 2016-03-11 00:21:05]

Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[mcasas, 2016-03-11 01:04:41]

The CQ bit was checked by mcasas@chromium.org

[commit-bot: I haz the power, 2016-03-11 01:05:14]

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

[commit-bot: I haz the power, 2016-03-11 02:10:39]

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

[commit-bot: I haz the power, 2016-03-11 02:10:40]

Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[phoglund_chromium, 2016-03-11 09:01:50]

Yeah, sounds like people are mostly onboard. Ship it

lgtm

[mcasas, 2016-03-11 16:56:43]

The CQ bit was checked by mcasas@chromium.org

[commit-bot: I haz the power, 2016-03-11 16:57:05]

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

[commit-bot: I haz the power, 2016-03-11 17:36:07]

Description was changed from

==========
Cleanup: remove prefix chrome_ of chrome/browser/media/chrome* files

Note how the name of the tests inside the files
respect the new naming, and not the chrome_ prefixed.

Done in two moves:
$ for i in `ls chrome_*`; do git mv $i `echo $i | sed -e 's/chrome_//'`; done
$ tools/git/mass-rename.py

No new code whatsoever.

BUG=592104
==========

to

==========
Cleanup: remove prefix chrome_ of chrome/browser/media/chrome* files

Note how the name of the tests inside the files
respect the new naming, and not the chrome_ prefixed.

Done in two moves:
$ for i in `ls chrome_*`; do git mv $i `echo $i | sed -e 's/chrome_//'`; done
$ tools/git/mass-rename.py

No new code whatsoever.

BUG=592104
==========

[commit-bot: I haz the power, 2016-03-11 17:36:09]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-11 17:37:33]

Description was changed from

==========
Cleanup: remove prefix chrome_ of chrome/browser/media/chrome* files

Note how the name of the tests inside the files
respect the new naming, and not the chrome_ prefixed.

Done in two moves:
$ for i in `ls chrome_*`; do git mv $i `echo $i | sed -e 's/chrome_//'`; done
$ tools/git/mass-rename.py

No new code whatsoever.

BUG=592104
==========

to

==========
Cleanup: remove prefix chrome_ of chrome/browser/media/chrome* files

Note how the name of the tests inside the files
respect the new naming, and not the chrome_ prefixed.

Done in two moves:
$ for i in `ls chrome_*`; do git mv $i `echo $i | sed -e 's/chrome_//'`; done
$ tools/git/mass-rename.py

No new code whatsoever.

BUG=592104

Committed: https://crrev.com/d8863236eda7fe4a3bfae3bbc39d10bbf7be2a30
Cr-Commit-Position: refs/heads/master@{#380662}
==========

[commit-bot: I haz the power, 2016-03-11 17:37:34]

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