DevTools: do not consider ? a shortcut while editing.

Source/devtools/front_end/ui/UIUtils.js

Index: Source/devtools/front_end/ui/UIUtils.js
diff --git a/Source/devtools/front_end/ui/UIUtils.js b/Source/devtools/front_end/ui/UIUtils.js
index 11941cd2858e3b5f16654223026434a510ef8658..63964438eab5da53382af638345cacca1c3c22da 100644
--- a/Source/devtools/front_end/ui/UIUtils.js
+++ b/Source/devtools/front_end/ui/UIUtils.js
@@ -179,15 +179,7 @@ WebInspector.isBeingEdited = function(node)
     if (element.classList.contains("text-prompt") || element.nodeName === "INPUT" || element.nodeName === "TEXTAREA")
         return true;
 
-    if (!WebInspector.__editingCount)
-        return false;
-
-    while (element) {
-        if (element.__editing)
-            return true;
-        element = element.parentElementOrShadowHost();
-    }
-    return false;
+    return !!WebInspector.__editingCount;

[vsevik, 2014/10/21 11:18:15]

Looks like this method becomes a generic isEditing now. I think the checks below
should be performed for document.activeElement in this case and the call sites
should be updated to not pass any parameters (giving a possibility to check that
none of them is broken now).

 }
 
 /**