Drop [TreatNullAs=NullString] for Document.cookie

Drop [TreatNullAs=NullString] for Document.cookie

https://html.spec.whatwg.org/#document

After this change, setting document.cookie to null will stringify to
"null", so it's simply like setting a cookie by that name. This matches
Firefox Nightly and Edge.

BUG=497307
Committed: https://crrev.com/b05426e99566190787f9532e9af0e8cdf7c1dd9b
Cr-Commit-Position: refs/heads/master@{#360567}

[philipj_slow, 2015-11-18 13:48:03]

The CQ bit was checked by philipj@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2015-11-18 13:48:22]

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

[commit-bot: I haz the power, 2015-11-18 14:56:35]

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

[commit-bot: I haz the power, 2015-11-18 14:56:36]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[philipj_slow, 2015-11-18 15:27:39]

philipj@opera.com changed reviewers:
+ mkwst@chromium.org

[philipj_slow, 2015-11-18 15:27:39]

PTAL and CQ? I think that linux_chromium_rel_ng is flaky, not my failures.

[philipj_slow, 2015-11-19 09:18:37]

Ping? You've posted reviews so I know you're there ;)

[Mike West, 2015-11-19 09:43:15]

Sorry, I missed this. :/

LGTM, matches the spec and everything. Perhaps add a link to
https://html.spec.whatwg.org/#document in the CL description just so folks can
verify?

[philipj_slow, 2015-11-19 12:09:36]

Description was changed from

==========
Drop [TreatNullAs=NullString] for Document.cookie

After this change, setting document.cookie to null will stringify to
"null", so it's simply like setting a cookie by that name. This matches
Firefox Nightly and Edge.

BUG=497307
==========

to

==========
Drop [TreatNullAs=NullString] for Document.cookie

https://html.spec.whatwg.org/#document

After this change, setting document.cookie to null will stringify to
"null", so it's simply like setting a cookie by that name. This matches
Firefox Nightly and Edge.

BUG=497307
==========

[philipj_slow, 2015-11-19 12:09:41]

On 2015/11/19 09:43:15, Mike West wrote:
> Sorry, I missed this. :/
> 
> LGTM, matches the spec and everything. Perhaps add a link to
> https://html.spec.whatwg.org/#document in the CL description just so folks can
> verify?

Done!

[philipj_slow, 2015-11-19 12:09:51]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2015-11-19 12:10:10]

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

[commit-bot: I haz the power, 2015-11-19 12:54:03]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-11-19 12:55:40]

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