DevTools: Fix batch updates dropping methods invocations.

Source/devtools/front_end/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 201 matching lines @@
     var direction = null;
     if (event.type === "mousewheel") {
         if (event.wheelDeltaY > 0)
             direction = "Up";
         else if (event.wheelDeltaY < 0)
             direction = "Down";
     } else {
         if (event.keyIdentifier === "Up" || event.keyIdentifier === "PageUp")
             direction = "Up";
         else if (event.keyIdentifier === "Down" || event.keyIdentifier === "PageDown")
-            direction = "Down";        
+            direction = "Down";
     }
     return direction;
 }
 
 /**
  * @param {string} hexString
  * @param {!Event} event
  */
 WebInspector._modifiedHexValue = function(hexString, event)
 {
@@ skipping 32 matching lines @@
 
 /**
  * @param {number} number
  * @param {!Event} event
  */
 WebInspector._modifiedFloatNumber = function(number, event)
 {
     var direction = WebInspector._valueModificationDirection(event);
     if (!direction)
         return number;
-    
+
     var arrowKeyOrMouseWheelEvent = (event.keyIdentifier === "Up" || event.keyIdentifier === "Down" || event.type === "mousewheel");
 
     // Jump by 10 when shift is down or jump by 0.1 when Alt/Option is down.
     // Also jump by 10 for page up and down, or by 100 if shift is held with a page key.
     var changeAmount = 1;
     if (event.shiftKey && !arrowKeyOrMouseWheelEvent)
         changeAmount = 100;
     else if (event.shiftKey || !arrowKeyOrMouseWheelEvent)
         changeAmount = 10;
     else if (event.altKey)
@@ skipping 30 matching lines @@
     if (!selection.rangeCount)
         return false;
 
     var selectionRange = selection.getRangeAt(0);
     if (!selectionRange.commonAncestorContainer.isSelfOrDescendant(element))
         return false;
 
     var originalValue = element.textContent;
     var wordRange = selectionRange.startContainer.rangeOfWord(selectionRange.startOffset, WebInspector.StyleValueDelimiters, element);
     var wordString = wordRange.toString();
-    
+
     if (suggestionHandler && suggestionHandler(wordString))
         return false;
 
     var replacementString;
     var prefix, suffix, number;
 
     var matches;
     matches = /(.*#)([\da-fA-F]+)(.*)/.exec(wordString);
     if (matches && matches.length) {
         prefix = matches[1];
         suffix = matches[3];
         number = WebInspector._modifiedHexValue(matches[2], event);
-        
+
         if (customNumberHandler)
             number = customNumberHandler(number);
 
         replacementString = prefix + number + suffix;
     } else {
         matches = /(.*?)(-?(?:\d+(?:\.\d+)?|\.\d+))(.*)/.exec(wordString);
         if (matches && matches.length) {
             prefix = matches[1];
             suffix = matches[3];
             number = WebInspector._modifiedFloatNumber(parseFloat(matches[2]), event);
-            
+
             // Need to check for null explicitly.
-            if (number === null)                
+            if (number === null)
                 return false;
-            
+
             if (customNumberHandler)
                 number = customNumberHandler(number);
 
             replacementString = prefix + number + suffix;
         }
     }
 
     if (replacementString) {
         var replacementTextNode = document.createTextNode(replacementString);
 
         wordRange.deleteContents();
         wordRange.insertNode(replacementTextNode);
 
         var finalSelectionRange = document.createRange();
         finalSelectionRange.setStart(replacementTextNode, 0);
         finalSelectionRange.setEnd(replacementTextNode, replacementString.length);
 
         selection.removeAllRanges();
         selection.addRange(finalSelectionRange);
 
         event.handled = true;
         event.preventDefault();
-                
+
         if (finishHandler)
             finishHandler(originalValue, replacementString);
 
         return true;
     }
     return false;
 }
 
 /**
  * @param {number} ms
@@ skipping 144 matching lines @@
 
 WebInspector._documentBlurred = function(event)
 {
     // We want to know when currentFocusElement loses focus to nowhere.
     // This is the case when event.relatedTarget is null (no element is being focused)
     // and document.activeElement is reset to default (this is not a window blur).
     if (!event.relatedTarget && document.activeElement === document.body)
       WebInspector.setCurrentFocusElement(null);
 }
 
-WebInspector._textInputTypes = ["text", "search", "tel", "url", "email", "password"].keySet(); 
+WebInspector._textInputTypes = ["text", "search", "tel", "url", "email", "password"].keySet();
 WebInspector._isTextEditingElement = function(element)
 {
     if (element instanceof HTMLInputElement)
         return element.type in WebInspector._textInputTypes;
 
     if (element instanceof HTMLTextAreaElement)
         return true;
 
     return false;
 }
@@ skipping 231 matching lines @@
 
 WebInspector.endBatchUpdate = function()
 {
     if (--WebInspector._coalescingLevel)
         return;
 
     var handlers = WebInspector._postUpdateHandlers;
     delete WebInspector._postUpdateHandlers;
 
     window.requestAnimationFrame(function() {
-        if (WebInspector._coalescingLevel)
-            return;
         var keys = handlers.keys();
         for (var i = 0; i < keys.length; ++i) {
             var object = keys[i];
-            var methods = handlers.get(object).keys();
+            var methods = handlers.get(object).items();
             for (var j = 0; j < methods.length; ++j)
                 methods[j].call(object);
         }
     });
 }
 
 /**
  * @param {!Object} object
  * @param {function()} method
  */
 WebInspector.invokeOnceAfterBatchUpdate = function(object, method)
 {
     if (!WebInspector._coalescingLevel) {
-        window.requestAnimationFrame(function() {
-            if (!WebInspector._coalescingLevel)
-                method.call(object);
-        });
+        window.requestAnimationFrame(method.bind(object));
         return;
     }
 
     var methods = WebInspector._postUpdateHandlers.get(object);
     if (!methods) {
-        methods = new Map();
+        methods = new Set();
         WebInspector._postUpdateHandlers.put(object, methods);
     }
-    methods.put(method);
+    methods.add(method);
 }
 
 ;(function() {
 
 function windowLoaded()
 {
     window.addEventListener("focus", WebInspector._windowFocused, false);
     window.addEventListener("blur", WebInspector._windowBlurred, false);
     document.addEventListener("focus", WebInspector._focusChanged, true);
     document.addEventListener("blur", WebInspector._documentBlurred, true);
     window.removeEventListener("DOMContentLoaded", windowLoaded, false);
 }
 
 window.addEventListener("DOMContentLoaded", windowLoaded, false);
 
 })();