Move extension_test_message_listener to extensions.

Move extension_test_message_listener to extensions.

This CL is part of the refactoring of webview into extensions/, and the webview test infrastructure depends heavily on this. Webview tests will be added to extensions soon.

BUG=352290
Committed: https://crrev.com/910f2f9803b96e22989b6ac2ea80a50c49b9d20f
Cr-Commit-Position: refs/heads/master@{#295665}

[lfg, 2014-09-18 20:54:33]

lfg@chromium.org changed reviewers:
+ yoz@chromium.org

[lfg, 2014-09-18 20:54:35]

yoz@:

PTAL?

[Yoyo Zhou, 2014-09-18 21:31:49]

LGTM, are there any appropriate BUG=?

[lfg, 2014-09-18 21:41:03]

lfg@chromium.org changed reviewers:
+ jamescook@chromium.org, rockot@chromium.org

[lfg, 2014-09-18 21:41:04]

I'm not sure if there's an appropriate BUG=, I was hoping you could point me to
one :), but perhaps rockot@ or jamescook@ could tell?

I'm doing this patch first because I'll later add some webview tests to
extensions, and the webview test infrastructure heavily depends on
extension_test_message_listener.

[lfg, 2014-09-18 21:42:46]

lfg@chromium.org changed reviewers:
+ sky@chromium.org

[lfg, 2014-09-18 21:42:47]

sky@chromium.org: Please review changes in 

chrome/browser/*

[Yoyo Zhou, 2014-09-18 21:42:58]

On 2014/09/18 21:41:04, lfg wrote:
> I'm not sure if there's an appropriate BUG=, I was hoping you could point me
to
> one :), but perhaps rockot@ or jamescook@ could tell?
> 
> I'm doing this patch first because I'll later add some webview tests to
> extensions, and the webview test infrastructure heavily depends on
> extension_test_message_listener.

Well, do you have a bug for that? You can use it here, so people have some
context for why. (Even though this patch is a good idea in general.)

[lfg, 2014-09-18 21:44:42]

On 2014/09/18 21:42:58, Yoyo Zhou wrote:
> On 2014/09/18 21:41:04, lfg wrote:
> > I'm not sure if there's an appropriate BUG=, I was hoping you could point me
> to
> > one :), but perhaps rockot@ or jamescook@ could tell?
> > 
> > I'm doing this patch first because I'll later add some webview tests to
> > extensions, and the webview test infrastructure heavily depends on
> > extension_test_message_listener.
> 
> Well, do you have a bug for that? You can use it here, so people have some
> context for why. (Even though this patch is a good idea in general.)

Done.

[James Cook, 2014-09-18 21:46:01]

On 2014/09/18 21:42:58, Yoyo Zhou wrote:
> On 2014/09/18 21:41:04, lfg wrote:
> > I'm not sure if there's an appropriate BUG=, I was hoping you could point me
> to
> > one :), but perhaps rockot@ or jamescook@ could tell?
> > 
> > I'm doing this patch first because I'll later add some webview tests to
> > extensions, and the webview test infrastructure heavily depends on
> > extension_test_message_listener.
> 
> Well, do you have a bug for that? You can use it here, so people have some
> context for why. (Even though this patch is a good idea in general.)

How about:

crbug.com/352293	Assigned	hanxi@chromium.org	 	Move webview API to //extensions

Also, please include a one or two line motivation for the change in the commit
comment. Something like "Part of the refactoring of webview into src/extensions.
webview test infrastructure heavily depends on
extension_test_message_listener and tests will be moving there soon".  Or
similar.

[James Cook, 2014-09-18 21:47:20]

On 2014/09/18 21:46:01, James Cook wrote:
> On 2014/09/18 21:42:58, Yoyo Zhou wrote:
> > On 2014/09/18 21:41:04, lfg wrote:
> > > I'm not sure if there's an appropriate BUG=, I was hoping you could point
me
> > to
> > > one :), but perhaps rockot@ or jamescook@ could tell?
> > > 
> > > I'm doing this patch first because I'll later add some webview tests to
> > > extensions, and the webview test infrastructure heavily depends on
> > > extension_test_message_listener.
> > 
> > Well, do you have a bug for that? You can use it here, so people have some
> > context for why. (Even though this patch is a good idea in general.)
> 
> How about:
> 
> crbug.com/352293	Assigned	mailto:hanxi@chromium.org	 	Move webview API to
//extensions
> 
> Also, please include a one or two line motivation for the change in the commit
> comment. Something like "Part of the refactoring of webview into
src/extensions.
> webview test infrastructure heavily depends on
> extension_test_message_listener and tests will be moving there soon".  Or
> similar.

352290 is fine too. But the commit comment still needs a bit more detail.  When
I'm researching old commits it's super handy to see the motivation without
looking up each bug.

[lfg, 2014-09-18 21:49:34]

On 2014/09/18 21:47:20, James Cook wrote:
> On 2014/09/18 21:46:01, James Cook wrote:
> > On 2014/09/18 21:42:58, Yoyo Zhou wrote:
> > > On 2014/09/18 21:41:04, lfg wrote:
> > > > I'm not sure if there's an appropriate BUG=, I was hoping you could
point
> me
> > > to
> > > > one :), but perhaps rockot@ or jamescook@ could tell?
> > > > 
> > > > I'm doing this patch first because I'll later add some webview tests to
> > > > extensions, and the webview test infrastructure heavily depends on
> > > > extension_test_message_listener.
> > > 
> > > Well, do you have a bug for that? You can use it here, so people have some
> > > context for why. (Even though this patch is a good idea in general.)
> > 
> > How about:
> > 
> > crbug.com/352293	Assigned	mailto:hanxi@chromium.org	 	Move webview API to
> //extensions
> > 
> > Also, please include a one or two line motivation for the change in the
commit
> > comment. Something like "Part of the refactoring of webview into
> src/extensions.
> > webview test infrastructure heavily depends on
> > extension_test_message_listener and tests will be moving there soon".  Or
> > similar.
> 
> 352290 is fine too. But the commit comment still needs a bit more detail. 
When
> I'm researching old commits it's super handy to see the motivation without
> looking up each bug.

Done.

[sky, 2014-09-18 23:41:05]

LGTM

[lfg, 2014-09-19 05:02:12]

The CQ bit was checked by lfg@chromium.org

[commit-bot: I haz the power, 2014-09-19 05:02:21]

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

[commit-bot: I haz the power, 2014-09-19 05:31:21]

Committed patchset #5 (id:80001) as 700ea9694972b9061554d420a20870b258d7c277

[commit-bot: I haz the power, 2014-09-19 05:32:09]

Patchset 5 (id:??) landed as
https://crrev.com/910f2f9803b96e22989b6ac2ea80a50c49b9d20f
Cr-Commit-Position: refs/heads/master@{#295665}