TaskScheduler: Add static_assert in TaskTraits::operator==.

TaskScheduler: Add static_assert in TaskTraits::operator==.

Add a static_assert in TaskTraits::operator== to ensure that no member
can be added to TaskTraits without updating operator==.

BUG=553459
Committed: https://crrev.com/7e0a1ba8e64f9dc10769ad5709750a2c8b88c84c
Cr-Commit-Position: refs/heads/master@{#387954}

[fdoray, 2016-04-15 18:56:13]

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

[commit-bot: I haz the power, 2016-04-15 18:56:47]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1897483002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1897483002/1

[commit-bot: I haz the power, 2016-04-15 20:00:05]

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

[commit-bot: I haz the power, 2016-04-15 20:00:06]

Dry run: This issue passed the CQ dry run.

[fdoray, 2016-04-15 21:07:14]

fdoray@chromium.org changed reviewers:
+ gab@chromium.org, robliao@chromium.org

[fdoray, 2016-04-15 21:07:14]

Can you review this CL? Thanks.

[robliao, 2016-04-15 21:36:13]

https://codereview.chromium.org/1897483002/diff/1/base/task_scheduler/task_tr...
File base/task_scheduler/task_traits.cc (right):

https://codereview.chromium.org/1897483002/diff/1/base/task_scheduler/task_tr...
base/task_scheduler/task_traits.cc:43: "");
We should have a message here. Maybe "TaskTraits members changed. Update
operator==."?

[fdoray, 2016-04-18 13:07:58]

robliao@: Done. Can you take another look?

https://codereview.chromium.org/1897483002/diff/1/base/task_scheduler/task_tr...
File base/task_scheduler/task_traits.cc (right):

https://codereview.chromium.org/1897483002/diff/1/base/task_scheduler/task_tr...
base/task_scheduler/task_traits.cc:43: "");
On 2016/04/15 21:36:12, robliao wrote:
> We should have a message here. Maybe "TaskTraits members changed. Update
> operator==."?

Done.

[gab, 2016-04-18 15:52:17]

gab@chromium.org changed reviewers:
- gab@chromium.org

[gab, 2016-04-18 15:52:17]

Leaving this one to Rob :-)

[robliao, 2016-04-18 16:58:11]

lgtm

[fdoray, 2016-04-18 17:00:09]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2016-04-18 17:00:28]

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

[commit-bot: I haz the power, 2016-04-18 18:12:23]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-18 18:13:46]

Description was changed from

==========
TaskScheduler: Add static_assert in TaskTraits::operator==.

Add a static_assert in TaskTraits::operator== to ensure that no member
can be added to TaskTraits without updating operator==.

BUG=553459
==========

to

==========
TaskScheduler: Add static_assert in TaskTraits::operator==.

Add a static_assert in TaskTraits::operator== to ensure that no member
can be added to TaskTraits without updating operator==.

BUG=553459

Committed: https://crrev.com/7e0a1ba8e64f9dc10769ad5709750a2c8b88c84c
Cr-Commit-Position: refs/heads/master@{#387954}
==========

[commit-bot: I haz the power, 2016-04-18 18:13:47]

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