Add CRWSessionEntry isEqual

Add CRWSessionEntry isEqual

EGTest have DCHECKs on CRWSessionEntry equality (e.g.
TabHistoryPopupControllerTestCase testTabHistoryMenu).
This fails because entry can be different even if they have the same
navigation item.

This is a tentative fix by the sheriff to make the tree green.
Please review this CL and revert it later if there is a better solution.

BUG=691469
Review-Url: https://codereview.chromium.org/2690973002
Cr-Commit-Position: refs/heads/master@{#449923}
Committed: https://chromium.googlesource.com/chromium/src/+/b8d957c64f413bde7d4e32bdf375bf622446e7ff

[Olivier, 2017-02-13 08:42:50]

olivierrobin@chromium.org changed reviewers:
+ eugenebut@chromium.org, kkhorimoto@chromium.org, sdefresne@chromium.org

[sdefresne, 2017-02-13 10:12:35]

lgtm to fix the tree, but Eugene/Kurt should review this post landing too.

[Olivier, 2017-02-13 10:13:00]

The CQ bit was checked by olivierrobin@chromium.org

[commit-bot: I haz the power, 2017-02-13 10:13:09]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Olivier, 2017-02-13 10:14:55]

On 2017/02/13 10:13:09, commit-bot: I haz the power wrote:
> CQ is trying da patch. Follow status at
>  
>
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

Thanks.
crbug.com/691469 is a RBB assigned to Eugene.

[Olivier, 2017-02-13 10:15:52]

Description was changed from

==========
Add CRWSessionEntry isEqual

EGTest have DCHECKs on CRWSessionEntry equality (e.g.
TabHistoryPopupControllerTestCase testTabHistoryMenu.
This fails because entry can be different event if they have the same
navigation item.

This is a tentative fix by the sheriff to make the tree green.
Please review this CL and revert it later if there is a better solution.

BUG=691469
==========

to

==========
Add CRWSessionEntry isEqual

EGTest have DCHECKs on CRWSessionEntry equality (e.g.
TabHistoryPopupControllerTestCase testTabHistoryMenu).
This fails because entry can be different even if they have the same
navigation item.

This is a tentative fix by the sheriff to make the tree green.
Please review this CL and revert it later if there is a better solution.

BUG=691469
==========

[commit-bot: I haz the power, 2017-02-13 10:26:05]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486980780867900, "parent_rev":
"2fd0fddb7497752f6e6abf8e3c7b996466cd62e9", "commit_rev":
"b8d957c64f413bde7d4e32bdf375bf622446e7ff"}

[commit-bot: I haz the power, 2017-02-13 10:27:02]

Description was changed from

==========
Add CRWSessionEntry isEqual

EGTest have DCHECKs on CRWSessionEntry equality (e.g.
TabHistoryPopupControllerTestCase testTabHistoryMenu).
This fails because entry can be different even if they have the same
navigation item.

This is a tentative fix by the sheriff to make the tree green.
Please review this CL and revert it later if there is a better solution.

BUG=691469
==========

to

==========
Add CRWSessionEntry isEqual

EGTest have DCHECKs on CRWSessionEntry equality (e.g.
TabHistoryPopupControllerTestCase testTabHistoryMenu).
This fails because entry can be different even if they have the same
navigation item.

This is a tentative fix by the sheriff to make the tree green.
Please review this CL and revert it later if there is a better solution.

BUG=691469

Review-Url: https://codereview.chromium.org/2690973002
Cr-Commit-Position: refs/heads/master@{#449923}
Committed:
https://chromium.googlesource.com/chromium/src/+/b8d957c64f413bde7d4e32bdf375...
==========

[commit-bot: I haz the power, 2017-02-13 10:27:03]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/b8d957c64f413bde7d4e32bdf375...

[Eugene But (OOO till 7-30), 2017-02-13 16:52:38]

lgtm for fixing the tree. Thank you for doing this.