DOMTokenList: Fold AddInternal() and RemoveInternal() into toggle().

DOMTokenList: Fold AddInternal() and RemoveInternal() into toggle().

Only toggle() uses AddInternal() and RemoveInternal(). This CL folds them into
toggle(), and update toggle() code to follow the DOM specification.

This CL doesn't change any behavior.

BUG=726214
Review-Url: https://codereview.chromium.org/2909203003
Cr-Commit-Position: refs/heads/master@{#475440}
Committed: https://chromium.googlesource.com/chromium/src/+/eff1abada903178a950e2c2587f227356cff584d

[tkent, 2017-05-30 03:54:35]

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

[tkent, 2017-05-30 03:54:54]

Description was changed from

==========
DOMTokenList: Fold AddInternal() and RemoveInternal() into toggle().

Only toggle() uses AddInternal() and RemoveInternal(). This CL folds them into
toggle(), and update toggle() code to follow the DOM specification.

BUG=726214
==========

to

==========
DOMTokenList: Fold AddInternal() and RemoveInternal() into toggle().

Only toggle() uses AddInternal() and RemoveInternal(). This CL folds them into
toggle(), and update toggle() code to follow the DOM specification.

This CL doesn't change any behavior.

BUG=726214
==========

[commit-bot: I haz the power, 2017-05-30 03:54:55]

Dry run: CQ is trying da patch.

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

[tkent, 2017-05-30 05:52:21]

tkent@chromium.org changed reviewers:
+ kochi@chromium.org

[tkent, 2017-05-30 05:52:22]

kochi@, would you review this please?

[commit-bot: I haz the power, 2017-05-30 06:02:24]

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

[commit-bot: I haz the power, 2017-05-30 06:02:27]

Dry run: This issue passed the CQ dry run.

[kochi, 2017-05-30 06:16:27]

LGTM

The spec itself is very confusing to me, esp. "force"
parameter :)

[tkent, 2017-05-30 06:23:31]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2017-05-30 06:23:40]

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, 2017-05-30 06:27:13]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1496125411278400, "parent_rev":
"2579c854287eab6fa1a3b3162bf7e1a17254cd28", "commit_rev":
"eff1abada903178a950e2c2587f227356cff584d"}

[commit-bot: I haz the power, 2017-05-30 06:27:20]

Description was changed from

==========
DOMTokenList: Fold AddInternal() and RemoveInternal() into toggle().

Only toggle() uses AddInternal() and RemoveInternal(). This CL folds them into
toggle(), and update toggle() code to follow the DOM specification.

This CL doesn't change any behavior.

BUG=726214
==========

to

==========
DOMTokenList: Fold AddInternal() and RemoveInternal() into toggle().

Only toggle() uses AddInternal() and RemoveInternal(). This CL folds them into
toggle(), and update toggle() code to follow the DOM specification.

This CL doesn't change any behavior.

BUG=726214

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

[commit-bot: I haz the power, 2017-05-30 06:27:21]

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