[Part 5 of 6] Move TaskManagerTester to the task_management directory.

Move TaskManagerTester to the task_management directory.

[splitting apart task_manager_browsertest_util, part 5 of 6]

BUG=606963
Committed: https://crrev.com/22bcc7727da7c2134dfa1a902c1f0080c145ad3e
Cr-Commit-Position: refs/heads/master@{#392662}

[ncarter (slow), 2016-05-06 22:28:32]

Description was changed from

==========
Move TaskManagerTester to the task_management directory.

[splitting apart task_manager_browsertest_util, part 5]

BUG=606963
==========

to

==========
Move TaskManagerTester to the task_management directory.

[splitting apart task_manager_browsertest_util, part 5 of 6]

BUG=606963
==========

[ncarter (slow), 2016-05-06 22:29:06]

nick@chromium.org changed reviewers:
+ afakhry@chromium.org

[ncarter (slow), 2016-05-06 22:29:07]

Ahmed, please review (this is part 5)

[afakhry, 2016-05-07 00:23:59]

https://codereview.chromium.org/1956943002/diff/20001/chrome/browser/task_man...
File chrome/browser/task_manager/legacy_task_manager_tester.h (right):

https://codereview.chromium.org/1956943002/diff/20001/chrome/browser/task_man...
chrome/browser/task_manager/legacy_task_manager_tester.h:16: class
LegacyTaskManagerTester : public task_management::TaskManagerTester {
I'll have to wait until I get why you made that extra hierarchy level. :)

https://codereview.chromium.org/1956943002/diff/20001/chrome/browser/task_man...
File chrome/browser/task_manager/task_manager_browsertest_util.cc (right):

https://codereview.chromium.org/1956943002/diff/20001/chrome/browser/task_man...
chrome/browser/task_manager/task_manager_browsertest_util.cc:49:
&ResourceChangeObserver::OnResourceChange, base::Unretained(this)));
mmm, I see you already made it here! :)

[ncarter (slow), 2016-05-07 22:23:09]

https://codereview.chromium.org/1956943002/diff/20001/chrome/browser/task_man...
File chrome/browser/task_manager/legacy_task_manager_tester.h (right):

https://codereview.chromium.org/1956943002/diff/20001/chrome/browser/task_man...
chrome/browser/task_manager/legacy_task_manager_tester.h:16: class
LegacyTaskManagerTester : public task_management::TaskManagerTester {
On 2016/05/07 00:23:59, afakhry wrote:
> I'll have to wait until I get why you made that extra hierarchy level. :)

Mostly it's me being lazy, and wanting to not have to declare all the methods in
the header, and this let me hide everything except the factory function
LegacyTaskManagerTesterImpl in the cc file.

[ncarter (slow), 2016-05-09 20:26:35]

https://codereview.chromium.org/1956943002/diff/20001/chrome/browser/task_man...
File chrome/browser/task_manager/task_manager_browsertest_util.cc (right):

https://codereview.chromium.org/1956943002/diff/20001/chrome/browser/task_man...
chrome/browser/task_manager/task_manager_browsertest_util.cc:49:
&ResourceChangeObserver::OnResourceChange, base::Unretained(this)));
On 2016/05/07 00:23:59, afakhry wrote:
> mmm, I see you already made it here! :)

Acknowledged.

[afakhry, 2016-05-09 20:28:05]

lgtm

[ncarter (slow), 2016-05-10 18:34:57]

The CQ bit was checked by nick@chromium.org

[commit-bot: I haz the power, 2016-05-10 18:38:48]

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

[commit-bot: I haz the power, 2016-05-10 18:43:51]

Description was changed from

==========
Move TaskManagerTester to the task_management directory.

[splitting apart task_manager_browsertest_util, part 5 of 6]

BUG=606963
==========

to

==========
Move TaskManagerTester to the task_management directory.

[splitting apart task_manager_browsertest_util, part 5 of 6]

BUG=606963
==========

[commit-bot: I haz the power, 2016-05-10 18:43:52]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-05-10 18:45:06]

Description was changed from

==========
Move TaskManagerTester to the task_management directory.

[splitting apart task_manager_browsertest_util, part 5 of 6]

BUG=606963
==========

to

==========
Move TaskManagerTester to the task_management directory.

[splitting apart task_manager_browsertest_util, part 5 of 6]

BUG=606963

Committed: https://crrev.com/22bcc7727da7c2134dfa1a902c1f0080c145ad3e
Cr-Commit-Position: refs/heads/master@{#392662}
==========

[commit-bot: I haz the power, 2016-05-10 18:45:07]

Patchset 3 (id:??) landed as
https://crrev.com/22bcc7727da7c2134dfa1a902c1f0080c145ad3e
Cr-Commit-Position: refs/heads/master@{#392662}