Remove unnecessary code from HTMLOptGroupElement.

Remove unnecessary code from HTMLOptGroupElement.

* We don't need to call setRecalcListItems() in childrenChanged() because
  HTMLOptionElement::insertedInto() and removedFrom() do it.

* We don't need to call setRecalcListItems() in parseAttribute(). OPTGROUP
  attributes don't affect HTMLSelectElement::m_listItems.
  The code exists since http://trac.webkit.org/changeset/4 .

BUG=624220
Committed: https://crrev.com/8c8740e3119f92898875bbf06cb46537ac832533
Cr-Commit-Position: refs/heads/master@{#402722}

[tkent, 2016-06-29 03:24:09]

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

[tkent, 2016-06-29 03:24:10]

keishi, would you review this please?

[tkent, 2016-06-29 03:43:37]

Description was changed from

==========
Remove unnecessary code from HTMLOptGroupElement.

* We don't need to call setRecalcListItems() in childrenChanged() becuase
  HTMLOptionElement::insertedInto() and removedFrom() do it.

* We don't need to call setRecalcListItems() in parseAttribute(). Optgroup
  attributes don't affect HTMLSelectElement::m_listItems.

BUG=624220
==========

to

==========
Remove unnecessary code from HTMLOptGroupElement.

* We don't need to call setRecalcListItems() in childrenChanged() becuase
  HTMLOptionElement::insertedInto() and removedFrom() do it.

* We don't need to call setRecalcListItems() in parseAttribute(). Optgroup
  attributes don't affect HTMLSelectElement::m_listItems.
  The code exists since http://trac.webkit.org/changeset/4 .

BUG=624220
==========

[keishi, 2016-06-29 04:44:41]

LGTM

[tkent, 2016-06-29 04:48:41]

Description was changed from

==========
Remove unnecessary code from HTMLOptGroupElement.

* We don't need to call setRecalcListItems() in childrenChanged() becuase
  HTMLOptionElement::insertedInto() and removedFrom() do it.

* We don't need to call setRecalcListItems() in parseAttribute(). Optgroup
  attributes don't affect HTMLSelectElement::m_listItems.
  The code exists since http://trac.webkit.org/changeset/4 .

BUG=624220
==========

to

==========
Remove unnecessary code from HTMLOptGroupElement.

* We don't need to call setRecalcListItems() in childrenChanged() because
  HTMLOptionElement::insertedInto() and removedFrom() do it.

* We don't need to call setRecalcListItems() in parseAttribute(). OPTGROUP
  attributes don't affect HTMLSelectElement::m_listItems.
  The code exists since http://trac.webkit.org/changeset/4 .

BUG=624220
==========

[tkent, 2016-06-29 04:48:57]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2016-06-29 04:49:21]

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-06-29 04:53:39]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-29 04:54:54]

Description was changed from

==========
Remove unnecessary code from HTMLOptGroupElement.

* We don't need to call setRecalcListItems() in childrenChanged() because
  HTMLOptionElement::insertedInto() and removedFrom() do it.

* We don't need to call setRecalcListItems() in parseAttribute(). OPTGROUP
  attributes don't affect HTMLSelectElement::m_listItems.
  The code exists since http://trac.webkit.org/changeset/4 .

BUG=624220
==========

to

==========
Remove unnecessary code from HTMLOptGroupElement.

* We don't need to call setRecalcListItems() in childrenChanged() because
  HTMLOptionElement::insertedInto() and removedFrom() do it.

* We don't need to call setRecalcListItems() in parseAttribute(). OPTGROUP
  attributes don't affect HTMLSelectElement::m_listItems.
  The code exists since http://trac.webkit.org/changeset/4 .

BUG=624220

Committed: https://crrev.com/8c8740e3119f92898875bbf06cb46537ac832533
Cr-Commit-Position: refs/heads/master@{#402722}
==========

[commit-bot: I haz the power, 2016-06-29 04:54:55]

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