DevTools: refactor Common.Renderer for DOM nodes

third_party/WebKit/Source/devtools/front_end/audits/AuditExtensionCategory.js

Index: third_party/WebKit/Source/devtools/front_end/audits/AuditExtensionCategory.js
diff --git a/third_party/WebKit/Source/devtools/front_end/audits/AuditExtensionCategory.js b/third_party/WebKit/Source/devtools/front_end/audits/AuditExtensionCategory.js
index e6e54d2f8b54b1ae01e505746351d90dff23abca..53c6037cd2f4081fdf767462631b7b6744090150 100644
--- a/third_party/WebKit/Source/devtools/front_end/audits/AuditExtensionCategory.js
+++ b/third_party/WebKit/Source/devtools/front_end/audits/AuditExtensionCategory.js
@@ -215,7 +215,19 @@ Audits.AuditExtensionFormatters = {
      * @param {!SDK.RemoteObject} remoteObject
      */
     function onEvaluate(remoteObject) {
-      Common.Renderer.renderPromise(remoteObject).then(appendRenderer).then(remoteObject.release.bind(remoteObject));
+      var domModel = SDK.DOMModel.fromTarget(remoteObject.target());

[pfeldman, 2017/03/01 02:37:44]

if (remoteObject.isNode()) {
   ...
} else {
   ... still render it as an object though.
}

you can try using async/await.

[chenwilliam, 2017/03/02 00:06:54]

Done. If we use async await in our frontend code, I think this means we'll need
to rethink our backwards compatibility test suite right?

+      if (!domModel)
+        return;
+      domModel.pushObjectAsNodeToFrontend(remoteObject, onNodeResolved);
+
+      /**
+       * @param {?SDK.DOMNode} node
+       */
+      function onNodeResolved(node) {
+        if (!node)
+          return;
+        Common.Renderer.renderPromise(node).then(appendRenderer).then(remoteObject.release.bind(remoteObject));
+      }
 
       /**
        * @param {!Element} element