Refactor BrowserActivationWaiter as test support and move PopupBlockerBrowserTest to interactive_ui…

Refactor BrowserActivationWaiter as test support and move PopupBlockerBrowserTest to interactive_uitest

(1) Making BrowserActivationWaiter a common test support
(2) move PopupBlockerBrowserTest to interactive_uitest as there exists one test that is relying on native widget activation.

BUG=658295
TEST=utilizing the test support in PopupBlockerBrowserTest

Committed: https://crrev.com/91ff5a8662a658007afd53608db8f965bb2b03d7
Cr-Commit-Position: refs/heads/master@{#426878}

[Qiang(Joe) Xu, 2016-10-21 06:23:12]

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

[commit-bot: I haz the power, 2016-10-21 06:23:30]

Dry run: 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, 2016-10-21 07:30:14]

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

[commit-bot: I haz the power, 2016-10-21 07:30:14]

Dry run: This issue passed the CQ dry run.

[Qiang(Joe) Xu, 2016-10-21 17:04:21]

Description was changed from

==========
Refactor BrowserActivationWaiter as test support and move
PopupBlockerBrowserTest to interactive_uitest

BUG=
==========

to

==========
Refactor BrowserActivationWaiter as test support and move
PopupBlockerBrowserTest to interactive_uitest

(1) Making BrowserActivationWaiter a common test support
(2) move PopupBlockerBrowserTest to interactive_uitest as there exists one test
that is relying on native widget activation.

BUG=658295
TEST=utilizing the test support in PopupBlockerBrowserTest
==========

[Qiang(Joe) Xu, 2016-10-21 17:04:21]

warx@chromium.org changed reviewers:
+ avi@chromium.org, sky@chromium.org

[Qiang(Joe) Xu, 2016-10-21 17:05:40]

Hi avi@, sky@, could you please take a look? Thanks!

[Avi (use Gerrit), 2016-10-21 17:27:10]

lgtm

[sky, 2016-10-21 20:08:40]

LGTM

[Qiang(Joe) Xu, 2016-10-21 20:11:45]

The CQ bit was checked by warx@chromium.org

[commit-bot: I haz the power, 2016-10-21 20:12:24]

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, 2016-10-21 20:35:41]

Description was changed from

==========
Refactor BrowserActivationWaiter as test support and move
PopupBlockerBrowserTest to interactive_uitest

(1) Making BrowserActivationWaiter a common test support
(2) move PopupBlockerBrowserTest to interactive_uitest as there exists one test
that is relying on native widget activation.

BUG=658295
TEST=utilizing the test support in PopupBlockerBrowserTest
==========

to

==========
Refactor BrowserActivationWaiter as test support and move
PopupBlockerBrowserTest to interactive_uitest

(1) Making BrowserActivationWaiter a common test support
(2) move PopupBlockerBrowserTest to interactive_uitest as there exists one test
that is relying on native widget activation.

BUG=658295
TEST=utilizing the test support in PopupBlockerBrowserTest
==========

[commit-bot: I haz the power, 2016-10-21 20:35:43]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-21 20:37:53]

Description was changed from

==========
Refactor BrowserActivationWaiter as test support and move
PopupBlockerBrowserTest to interactive_uitest

(1) Making BrowserActivationWaiter a common test support
(2) move PopupBlockerBrowserTest to interactive_uitest as there exists one test
that is relying on native widget activation.

BUG=658295
TEST=utilizing the test support in PopupBlockerBrowserTest
==========

to

==========
Refactor BrowserActivationWaiter as test support and move
PopupBlockerBrowserTest to interactive_uitest

(1) Making BrowserActivationWaiter a common test support
(2) move PopupBlockerBrowserTest to interactive_uitest as there exists one test
that is relying on native widget activation.

BUG=658295
TEST=utilizing the test support in PopupBlockerBrowserTest

Committed: https://crrev.com/91ff5a8662a658007afd53608db8f965bb2b03d7
Cr-Commit-Position: refs/heads/master@{#426878}
==========

[commit-bot: I haz the power, 2016-10-21 20:37:54]

Patchset 1 (id:??) landed as
https://crrev.com/91ff5a8662a658007afd53608db8f965bb2b03d7
Cr-Commit-Position: refs/heads/master@{#426878}