Disable LogChannelIDAndCookieStores NOTREACHED.

Disable LogChannelIDAndCookieStores NOTREACHED.

The NOTREACHED is actually reached on iOS. Disable it for now.

Also implement on iOS the ChannelIDService tracking logic as it was not
done in http://crrev.com/1818603002.

BUG=599049
Committed: https://crrev.com/6daab6767d6470bbeadbdc2b7cacf4d6ba4709f7
Cr-Commit-Position: refs/heads/master@{#384001}

[bzanotti, 2016-03-30 09:58:07]

bzanotti@chromium.org changed reviewers:
+ mmenke@chromium.org

[bzanotti, 2016-03-30 09:58:08]

Please take a look.

[mmenke, 2016-03-30 14:55:18]

On 2016/03/30 09:58:08, bzanotti wrote:
> Please take a look.

LGTM, though we'll certainly want to enable this again sooner rather than later.

[mmenke, 2016-03-30 14:55:56]

mmenke@chromium.org changed reviewers:
+ nharper@chromium.org

[mmenke, 2016-03-30 14:55:57]

[+nharper]

[bzanotti, 2016-03-30 14:56:56]

On 2016/03/30 14:55:18, mmenke wrote:
> On 2016/03/30 09:58:08, bzanotti wrote:
> > Please take a look.
> 
> LGTM, though we'll certainly want to enable this again sooner rather than
later.

I agree with you. droger@ is OOO today, but I'll make sure to talk with him
tomorrow to make sure this is fixed.

[bzanotti, 2016-03-30 15:05:17]

bzanotti@chromium.org changed reviewers:
+ sdefresne@chromium.org

[bzanotti, 2016-03-30 15:05:17]

Sylvain: Please do an OWNERS review of 	ios/chrome/browser/browser_state/.

[sdefresne, 2016-03-30 15:10:38]

lgtm

[bzanotti, 2016-03-30 15:36:02]

The CQ bit was checked by bzanotti@chromium.org

[commit-bot: I haz the power, 2016-03-30 15:38:59]

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

[commit-bot: I haz the power, 2016-03-30 16:43:52]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-30 16:46:47]

Description was changed from

==========
Disable LogChannelIDAndCookieStores NOTREACHED.

The NOTREACHED is actually reached on iOS. Disable it for now.

Also implement on iOS the ChannelIDService tracking logic as it was not
done in http://crrev.com/1818603002.

BUG=599049
==========

to

==========
Disable LogChannelIDAndCookieStores NOTREACHED.

The NOTREACHED is actually reached on iOS. Disable it for now.

Also implement on iOS the ChannelIDService tracking logic as it was not
done in http://crrev.com/1818603002.

BUG=599049

Committed: https://crrev.com/6daab6767d6470bbeadbdc2b7cacf4d6ba4709f7
Cr-Commit-Position: refs/heads/master@{#384001}
==========

[commit-bot: I haz the power, 2016-03-30 16:46:48]

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