DevTools: refactor Common.Renderer for DOM nodes

third_party/WebKit/Source/devtools/front_end/sdk/DOMModel.js

Index: third_party/WebKit/Source/devtools/front_end/sdk/DOMModel.js
diff --git a/third_party/WebKit/Source/devtools/front_end/sdk/DOMModel.js b/third_party/WebKit/Source/devtools/front_end/sdk/DOMModel.js
index b6d4378f881608d8dfd771da4ddf8d60ced25743..12b4b67207c6fa5ddd310807e1a53f1e2281c5ce 100644
--- a/third_party/WebKit/Source/devtools/front_end/sdk/DOMModel.js
+++ b/third_party/WebKit/Source/devtools/front_end/sdk/DOMModel.js
@@ -1847,13 +1847,15 @@ SDK.DOMModel = class extends SDK.SDKModel {
 
   /**
    * @param {!SDK.RemoteObject} object
-   * @param {function(?SDK.DOMNode)} callback
+   * @return {!Promise<?SDK.DOMNode>}
    */
-  pushObjectAsNodeToFrontend(object, callback) {
-    if (object.isNode())
-      this.pushNodeToFrontend(/** @type {string} */ (object.objectId), callback);
-    else
-      callback(null);
+  pushObjectAsNodeToFrontend(object) {
+    return new Promise(fulfill => {
+      if (object.isNode())
+        this.pushNodeToFrontend(/** @type {string} */ (object.objectId), node => fulfill(node));

[dgozman, 2017/03/21 23:00:49]

nit: can just pass fulfill as a last parameter.

[chenwilliam, 2017/03/22 00:30:38]

Done.

+      else
+        fulfill(null);
+    });
   }
 
   /**