[ios] Cleaned up comments and tests for CookieStoreIOS synchronization.

[ios] Cleaned up comments and tests for CookieStoreIOS synchronization.

In UIWebView world CookieStoreIOS could dynamically change its state
and either be synchronized with NSHTTPCookieStorage or unsynchronized.

In WKWebView world CookieStoreIOS never changes its state. It is either
always synchronized (for sign in) or always unsynchronized (for regular
browsing).

Notable changes:
- updated comments to reflect that sync state is not dynamic
- removed RoundTripTest which tested non-existing dynamic synchronization flow
- removed other tests which tests dynamic synchronization

BUG=676144
Committed: https://crrev.com/d649da83761a5469b5d8b8a294682128ccfcbdee
Cr-Commit-Position: refs/heads/master@{#440851}

[Eugene But (OOO till 7-30), 2016-12-28 01:14:36]

Description was changed from

==========
[ios] Cleaned up comments and tests for CookieStoreIOS synchronization.

In UIWebView world CookieStoreIOS could dynamically change its state
and either be synchronized with NSHTTPCookieStorage or unsynchronized.

In WKWebView world CookieStoreIOS never changes its state. It is either
always synchronized (for sign in) or always unsynchronized (for regular
browsing).

Updated comments to reflect that sync state is not dynamic. Also removed
RoundTripTest which tested non-existing dynamic synchronization flow.

BUG=676144
==========

to

==========
[ios] Cleaned up comments and tests for CookieStoreIOS synchronization.

In UIWebView world CookieStoreIOS could dynamically change its state
and either be synchronized with NSHTTPCookieStorage or unsynchronized.

In WKWebView world CookieStoreIOS never changes its state. It is either
always synchronized (for sign in) or always unsynchronized (for regular
browsing).

Notable changes:
- updated comments to reflect that sync state is not dynamic
- removed RoundTripTest which tested non-existing dynamic synchronization flow
- removed other tests which tests dynamic synchronization

BUG=676144
==========

[Eugene But (OOO till 7-30), 2016-12-28 01:17:34]

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

[commit-bot: I haz the power, 2016-12-28 01:17:44]

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

[commit-bot: I haz the power, 2016-12-28 01:26:25]

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

[commit-bot: I haz the power, 2016-12-28 01:26:26]

Dry run: This issue passed the CQ dry run.

[Eugene But (OOO till 7-30), 2016-12-28 02:50:38]

eugenebut@chromium.org changed reviewers:
+ marq@chromium.org

[marq (ping after 24h), 2016-12-28 09:40:46]

lgtm

[Eugene But (OOO till 7-30), 2016-12-28 16:36:43]

The CQ bit was checked by eugenebut@chromium.org

[commit-bot: I haz the power, 2016-12-28 16:36:56]

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

[commit-bot: I haz the power, 2016-12-28 16:39:58]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1482943003751880,
"parent_rev": "61d5b0e98e75f27efd749b16bcaa3eddb57baa4c", "commit_rev":
"c557a58a28837f2397c1104a733d9073757c3423"}

[commit-bot: I haz the power, 2016-12-28 16:40:24]

Description was changed from

==========
[ios] Cleaned up comments and tests for CookieStoreIOS synchronization.

In UIWebView world CookieStoreIOS could dynamically change its state
and either be synchronized with NSHTTPCookieStorage or unsynchronized.

In WKWebView world CookieStoreIOS never changes its state. It is either
always synchronized (for sign in) or always unsynchronized (for regular
browsing).

Notable changes:
- updated comments to reflect that sync state is not dynamic
- removed RoundTripTest which tested non-existing dynamic synchronization flow
- removed other tests which tests dynamic synchronization

BUG=676144
==========

to

==========
[ios] Cleaned up comments and tests for CookieStoreIOS synchronization.

In UIWebView world CookieStoreIOS could dynamically change its state
and either be synchronized with NSHTTPCookieStorage or unsynchronized.

In WKWebView world CookieStoreIOS never changes its state. It is either
always synchronized (for sign in) or always unsynchronized (for regular
browsing).

Notable changes:
- updated comments to reflect that sync state is not dynamic
- removed RoundTripTest which tested non-existing dynamic synchronization flow
- removed other tests which tests dynamic synchronization

BUG=676144

Review-Url: https://codereview.chromium.org/2604763003
==========

[commit-bot: I haz the power, 2016-12-28 16:40:24]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2017-01-02 15:49:04]

Description was changed from

==========
[ios] Cleaned up comments and tests for CookieStoreIOS synchronization.

In UIWebView world CookieStoreIOS could dynamically change its state
and either be synchronized with NSHTTPCookieStorage or unsynchronized.

In WKWebView world CookieStoreIOS never changes its state. It is either
always synchronized (for sign in) or always unsynchronized (for regular
browsing).

Notable changes:
- updated comments to reflect that sync state is not dynamic
- removed RoundTripTest which tested non-existing dynamic synchronization flow
- removed other tests which tests dynamic synchronization

BUG=676144

Review-Url: https://codereview.chromium.org/2604763003
==========

to

==========
[ios] Cleaned up comments and tests for CookieStoreIOS synchronization.

In UIWebView world CookieStoreIOS could dynamically change its state
and either be synchronized with NSHTTPCookieStorage or unsynchronized.

In WKWebView world CookieStoreIOS never changes its state. It is either
always synchronized (for sign in) or always unsynchronized (for regular
browsing).

Notable changes:
- updated comments to reflect that sync state is not dynamic
- removed RoundTripTest which tested non-existing dynamic synchronization flow
- removed other tests which tests dynamic synchronization

BUG=676144

Committed: https://crrev.com/d649da83761a5469b5d8b8a294682128ccfcbdee
Cr-Commit-Position: refs/heads/master@{#440851}
==========

[commit-bot: I haz the power, 2017-01-02 15:49:04]

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