DevTools: Throttle file system refreshes on focus.

DevTools: Throttle file system refreshes on focus.

BUG=380860
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=177109

[vsevik, 2014-06-26 07:55:58]

PTAL

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

lgtm

https://chromiumcodereview.appspot.com/353743008/diff/1/Source/devtools/front...
File Source/devtools/front_end/sdk/Workspace.js (right):

https://chromiumcodereview.appspot.com/353743008/diff/1/Source/devtools/front...
Source/devtools/front_end/sdk/Workspace.js:446:
this._projectDelegate.refresh(path);
forgot to pass |callback| in the refresh

https://chromiumcodereview.appspot.com/353743008/diff/1/Source/devtools/front...
File Source/devtools/front_end/sdk/WorkspaceController.js (right):

https://chromiumcodereview.appspot.com/353743008/diff/1/Source/devtools/front...
Source/devtools/front_end/sdk/WorkspaceController.js:68:
barrier.callWhenDone(callback);
FYI: this differs from what it used to be, but I believe this is intentional.

[vsevik, 2014-06-26 16:59:24]

>
https://chromiumcodereview.appspot.com/353743008/diff/1/Source/devtools/front...
> Source/devtools/front_end/sdk/Workspace.js:446:
> this._projectDelegate.refresh(path);
> forgot to pass |callback| in the refresh
Done. 

>
https://chromiumcodereview.appspot.com/353743008/diff/1/Source/devtools/front...
> Source/devtools/front_end/sdk/WorkspaceController.js:68:
> barrier.callWhenDone(callback);
> FYI: this differs from what it used to be, but I believe this is intentional.

This is intended.

[vsevik, 2014-06-26 16:59:32]

The CQ bit was checked by vsevik@chromium.org

[commit-bot: I haz the power, 2014-06-26 17:02:40]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/vsevik@chromium.org/353743008/20001

[commit-bot: I haz the power, 2014-06-26 23:04:13]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_blink_compile_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/android_blink_compile_db...)
  android_blink_compile_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/android_blink_compile_re...)
  android_chromium_gn_compile_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/android_chromium_gn_comp...)
  blink_presubmit on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/8644)
  linux_blink_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/1...)
  linux_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/1...)
  linux_chromium_gn_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_chromium_gn_rel/bu...)
  mac_blink_compile_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_compile_dbg/bu...)
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/12984)
  linux_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/linux_gpu/builds/...)
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/22085)

[commit-bot: I haz the power, 2014-06-26 23:05:34]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-06-26 23:05:35]

Try jobs failed on following builders:
  mac_blink_compile_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_compile_dbg/bu...)
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/13006)
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/22222)

[vsevik, 2014-06-27 12:15:32]

The CQ bit was checked by vsevik@chromium.org

[commit-bot: I haz the power, 2014-06-27 12:16:20]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/vsevik@chromium.org/353743008/40001

[commit-bot: I haz the power, 2014-06-27 15:24:50]

Change committed as 177109