DevTools: better hasSelection check to include composed trees

third_party/WebKit/Source/devtools/front_end/platform/DOMExtension.js

 /*
  * Copyright (C) 2007 Apple Inc.  All rights reserved.
  * Copyright (C) 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 259 matching lines @@
   // FIXME: crbug.com/447523, use selection.isCollapsed when it is fixed for shadow dom.
   var selection = this.getComponentSelection();
   var range = selection && selection.rangeCount ? selection.getRangeAt(0) : null;
   return range ? range.collapsed : true;
 };
 
 /**
  * @return {boolean}
  */
 Node.prototype.hasSelection = function() {
+  // FIXME: use contains(node, {includeShadow: true}) when it is fixed for shadow dom.
+  var contents = this.querySelectorAll('content');
+  for (var i = 0; i < contents.length; ++i) {
+    var contentElement = contents[i].getDistributedNodes();

[allada, 2016/12/20 02:15:28]

Wouldnt it be easier to read using something like:
for (var content of contents) {
  if (content.getDistributedNodes().some(element => element.hasSelection()))
    return true;
}

Or even do it all in 1 line using two Array.some() callbacks if it does't over
complicate it.

[luoe, 2016/12/20 02:41:03]

Done.

+    for (var j = 0; j < contentElement.length; ++j) {
+      if (contentElement[j].hasSelection())
+        return true;
+    }
+  }
   if (this.isComponentSelectionCollapsed())
     return false;
   return this.getComponentSelection().containsNode(this, true);
 };
 
 /**
  * @return {!Selection}
  */
 Node.prototype.getDeepSelection = function() {
   var activeElement = this.ownerDocument.activeElement;
@@ skipping 528 matching lines @@
   function windowLoaded() {
     window.removeEventListener('DOMContentLoaded', windowLoaded, false);
     callback();
   }
 
   if (document.readyState === 'complete' || document.readyState === 'interactive')
     callback();
   else
     window.addEventListener('DOMContentLoaded', windowLoaded, false);
 }