DevTools: Throttle file system refreshes on focus.

Source/devtools/front_end/sdk/WorkspaceController.js

 /*
  * Copyright (C) 2013 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 18 matching lines @@
  */
 
 /**
  * @constructor
  */
 WebInspector.WorkspaceController = function(workspace)
 {
     this._workspace = workspace;
     WebInspector.resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.InspectedURLChanged, this._inspectedURLChanged, this);
     window.addEventListener("focus", this._windowFocused.bind(this), false);
+    this._fileSystemRefreshThrottler = new WebInspector.Throttler(1000);
 }
 
 WebInspector.WorkspaceController.prototype = {
     /**
      * @param {!WebInspector.Event} event
      */
     _inspectedURLChanged: function(event)
     {
         WebInspector.Revision.filterOutStaleRevisions();
     },
 
     /**
      * @param {?Event} event
      */
     _windowFocused: function(event)
     {
-        if (this._fileSystemRefreshTimeout)
-            return;
-        this._fileSystemRefreshTimeout = setTimeout(refreshFileSystems.bind(this), 1000);
+        this._fileSystemRefreshThrottler.schedule(refreshFileSystems.bind(this));
 
         /**
          * @this {WebInspector.WorkspaceController}
+         * @param {!WebInspector.Throttler.FinishCallback} callback
          */
-        function refreshFileSystems()
+        function refreshFileSystems(callback)
         {
-            delete this._fileSystemRefreshTimeout;
+            var barrier = new CallbackBarrier();
             var projects = this._workspace.projects();
             for (var i = 0; i < projects.length; ++i)
-                projects[i].refresh("/");
+                projects[i].refresh("/", barrier.createCallback());
+            barrier.callWhenDone(callback);

[lushnikov, 2014/06/26 16:01:25]

FYI: this differs from what it used to be, but I believe this is intentional.

         }
     }
 }