DevTools: do not consider ? a shortcut while editing.

Source/devtools/front_end/ui/UIUtils.js

 /*
  * Copyright (C) 2011 Google Inc.  All rights reserved.
  * Copyright (C) 2006, 2007, 2008 Apple Inc.  All rights reserved.
  * Copyright (C) 2007 Matt Lilek (pewtermoose@gmail.com).
  * Copyright (C) 2009 Joseph Pecoraro
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
@@ skipping 161 matching lines @@
  * @return {boolean}
  */
 WebInspector.isBeingEdited = function(node)
 {
     if (!node || node.nodeType !== Node.ELEMENT_NODE)
         return false;
     var element = /** {!Element} */ (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).

 }
 
 /**
  * @param {!Element} element
  * @param {boolean} value
  * @return {boolean}
  */
 WebInspector.markBeingEdited = function(element, value)
 {
     if (value) {
@@ skipping 882 matching lines @@
     __proto__: WebInspector.Object.prototype
 }
 
 WebInspector.initializeUIUtils = function()
 {
     window.addEventListener("focus", WebInspector._windowFocused, false);
     window.addEventListener("blur", WebInspector._windowBlurred, false);
     document.addEventListener("focus", WebInspector._focusChanged, true);
     document.addEventListener("blur", WebInspector._documentBlurred, true);
 }