Allow DOM.getNodeForLocation to skip to nearest non-shadow ancestor

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..ded89af42a459ff24bd011de839456b12216091d 100644
--- a/third_party/WebKit/Source/devtools/front_end/sdk/DOMModel.js
+++ b/third_party/WebKit/Source/devtools/front_end/sdk/DOMModel.js
@@ -1829,7 +1829,8 @@ SDK.DOMModel = class extends SDK.SDKModel {
    * @param {function(?SDK.DOMNode)} callback
    */
   nodeForLocation(x, y, callback) {
-    this._agent.getNodeForLocation(x, y, mycallback.bind(this));
+    var showUAShadowDOM = Common.moduleSetting('showUAShadowDOM').get();
+    this._agent.getNodeForLocation(x, y, showUAShadowDOM, mycallback.bind(this));

[peleg, 2017/03/22 06:29:03]

Is this what you had in mind for using moduleSetting?

[pfeldman, 2017/03/22 17:32:46]

I meant that you pass the boolean into DOMModel and fetch the setting on the
call site. showUAShadowDOM setting is defined in the elements module sdk does
not depend upon, so we should not use it here. We'll migrate to the scoped
settings at some point, but before that, we just need to be careful.

 
     /**
      * @param {?Protocol.Error} error