[Part 3 of 6] Switch ColumnSpecifier to enum class. This makes straddling namespaces easier.

Switch ColumnSpecifier to enum class. This makes straddling namespaces easier.

[splitting apart task_manager_browsertest_util, part 3 of 6]

BUG=606963
Committed: https://crrev.com/8e3d8ebf7f9f6f5f3782ffb4cbe4c77f64eccd7e
Cr-Commit-Position: refs/heads/master@{#392651}

[ncarter (slow), 2016-05-06 21:45:18]

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

[ncarter (slow), 2016-05-06 21:45:19]

Ahmed, please review (this is part 3)

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

Description was changed from

==========
Switch ColumnSpecifier to enum class. This makes straddling namespaces easier.

[splitting apart task_manager_browsertest_util, part 3]

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

to

==========
Switch ColumnSpecifier to enum class. This makes straddling namespaces easier.

[splitting apart task_manager_browsertest_util, part 3 of 6]

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

[afakhry, 2016-05-07 00:00:25]

lgtm. I like that splitting into several smaller CLs.

[ncarter (slow), 2016-05-09 16:47:24]

The CQ bit was checked by nick@chromium.org

[commit-bot: I haz the power, 2016-05-09 16:47:48]

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

[commit-bot: I haz the power, 2016-05-09 16:47:49]

This CL has an open dependency (Issue 1956753003 Patch 40001). Please resolve
the dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[ncarter (slow), 2016-05-09 21:35:57]

The CQ bit was checked by nick@chromium.org

[ncarter (slow), 2016-05-09 21:35:57]

The patchset sent to the CQ was uploaded after l-g-t-m from afakhry@chromium.org
Link to the patchset: https://codereview.chromium.org/1960703002/#ps60001
(title: "Rebase")

[commit-bot: I haz the power, 2016-05-09 21:36:23]

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

[commit-bot: I haz the power, 2016-05-09 21:36:24]

This CL has an open dependency (Issue 1956753003 Patch 60001). Please resolve
the dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[ncarter (slow), 2016-05-10 16:50:32]

The CQ bit was checked by nick@chromium.org

[commit-bot: I haz the power, 2016-05-10 16:51:08]

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

[commit-bot: I haz the power, 2016-05-10 18:19:13]

Description was changed from

==========
Switch ColumnSpecifier to enum class. This makes straddling namespaces easier.

[splitting apart task_manager_browsertest_util, part 3 of 6]

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

to

==========
Switch ColumnSpecifier to enum class. This makes straddling namespaces easier.

[splitting apart task_manager_browsertest_util, part 3 of 6]

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

[commit-bot: I haz the power, 2016-05-10 18:19:14]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-05-10 18:21:20]

Description was changed from

==========
Switch ColumnSpecifier to enum class. This makes straddling namespaces easier.

[splitting apart task_manager_browsertest_util, part 3 of 6]

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

to

==========
Switch ColumnSpecifier to enum class. This makes straddling namespaces easier.

[splitting apart task_manager_browsertest_util, part 3 of 6]

BUG=606963

Committed: https://crrev.com/8e3d8ebf7f9f6f5f3782ffb4cbe4c77f64eccd7e
Cr-Commit-Position: refs/heads/master@{#392651}
==========

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

Patchset 4 (id:??) landed as
https://crrev.com/8e3d8ebf7f9f6f5f3782ffb4cbe4c77f64eccd7e
Cr-Commit-Position: refs/heads/master@{#392651}