Remove LayoutSVGResourceContainer::m_id

Remove LayoutSVGResourceContainer::m_id

By passing the old and new values to the idChanged() method, the only
"reload" of the value can be eliminated, and other instances can use
getIdAttribute() on the element. This makes the m_id field unused, so
it can be removed.

BUG=454767
Review-Url: https://codereview.chromium.org/2714153002
Cr-Commit-Position: refs/heads/master@{#452838}
Committed: https://chromium.googlesource.com/chromium/src/+/33a7a547b95490ef9f24c8a24d9fa35800df1595

[fs, 2017-02-24 12:01:03]

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

[commit-bot: I haz the power, 2017-02-24 12:01:20]

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

[fs, 2017-02-24 13:07:03]

fs@opera.com changed reviewers:
+ schenney@chromium.org

[commit-bot: I haz the power, 2017-02-24 13:28:13]

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

[commit-bot: I haz the power, 2017-02-24 13:28:14]

Dry run: This issue passed the CQ dry run.

[Stephen Chennney, 2017-02-24 15:59:34]

On 2017/02/24 13:07:03, fs wrote:

lgtm

[fs, 2017-02-24 16:00:12]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2017-02-24 16:00:45]

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-02-24 16:06:49]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487952012027400, "parent_rev":
"14a2bd9d0775c20be8cb46288b58552108458981", "commit_rev":
"33a7a547b95490ef9f24c8a24d9fa35800df1595"}

[commit-bot: I haz the power, 2017-02-24 16:07:29]

Description was changed from

==========
Remove LayoutSVGResourceContainer::m_id

By passing the old and new values to the idChanged() method, the only
"reload" of the value can be eliminated, and other instances can use
getIdAttribute() on the element. This makes the m_id field unused, so
it can be removed.

BUG=454767
==========

to

==========
Remove LayoutSVGResourceContainer::m_id

By passing the old and new values to the idChanged() method, the only
"reload" of the value can be eliminated, and other instances can use
getIdAttribute() on the element. This makes the m_id field unused, so
it can be removed.

BUG=454767

Review-Url: https://codereview.chromium.org/2714153002
Cr-Commit-Position: refs/heads/master@{#452838}
Committed:
https://chromium.googlesource.com/chromium/src/+/33a7a547b95490ef9f24c8a24d9f...
==========

[commit-bot: I haz the power, 2017-02-24 16:07:30]

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