[DevTools] Do not inherit SDK.DOMNode from SDK.SDKObject

[DevTools] Do not inherit SDK.DOMNode from SDK.SDKObject

It exposes DOMModel already, there is no need for the target.
There is also no need to extend from Common.Object, have an extra field for nothing and provide event listening capabilities.

BUG=none

Review-Url: https://codereview.chromium.org/2778283002
Cr-Commit-Position: refs/heads/master@{#460601}
Committed: https://chromium.googlesource.com/chromium/src/+/f367e006d3be29eadb50d4d161d63dcc24df4620

[dgozman, 2017-03-28 17:55:37]

dgozman@chromium.org changed reviewers:
+ pfeldman@chromium.org

[dgozman, 2017-03-28 17:55:38]

What do you think about this?

[dgozman, 2017-03-28 17:55:41]

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

[commit-bot: I haz the power, 2017-03-28 17:56:08]

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, 2017-03-28 19:21:50]

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

[commit-bot: I haz the power, 2017-03-28 19:21:51]

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

[pfeldman, 2017-03-29 21:57:00]

lgtm, but why?

[dgozman, 2017-03-29 22:41:43]

Description was changed from

==========
[DevTools] Do not inherit SDK.DOMNode from SDK.SDKObject

It exposes DOMModel already, there is no need for the target.

BUG=none
==========

to

==========
[DevTools] Do not inherit SDK.DOMNode from SDK.SDKObject

It exposes DOMModel already, there is no need for the target.
There is also no need to extend from Common.Object, have an extra field for
nothing and provide event listening capabilities.

BUG=none
==========

[dgozman, 2017-03-29 22:41:49]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2017-03-29 22:42:35]

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-03-30 00:41:48]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490827309101400, "parent_rev":
"fb1ec92eac2f8a6db6d357249563ffa3fca43058", "commit_rev":
"f367e006d3be29eadb50d4d161d63dcc24df4620"}

[commit-bot: I haz the power, 2017-03-30 00:42:48]

Description was changed from

==========
[DevTools] Do not inherit SDK.DOMNode from SDK.SDKObject

It exposes DOMModel already, there is no need for the target.
There is also no need to extend from Common.Object, have an extra field for
nothing and provide event listening capabilities.

BUG=none
==========

to

==========
[DevTools] Do not inherit SDK.DOMNode from SDK.SDKObject

It exposes DOMModel already, there is no need for the target.
There is also no need to extend from Common.Object, have an extra field for
nothing and provide event listening capabilities.

BUG=none

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

[commit-bot: I haz the power, 2017-03-30 00:42:50]

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