Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(106)

Unified Diff: pkg/analysis_server/test/physical_resource_provider_test.dart

Issue 312743002: Use 'Directory.watch' on Windows in pkg/watcher, instead of pooling. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Fixes Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | pkg/pkg.status » ('j') | pkg/watcher/lib/src/directory_watcher/windows.dart » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/analysis_server/test/physical_resource_provider_test.dart
diff --git a/pkg/analysis_server/test/physical_resource_provider_test.dart b/pkg/analysis_server/test/physical_resource_provider_test.dart
index f2af810a7b48daf3deccc73854daec8b31b764a8..a74e326083b30fbfc3212070f6f899f0f4631105 100644
--- a/pkg/analysis_server/test/physical_resource_provider_test.dart
+++ b/pkg/analysis_server/test/physical_resource_provider_test.dart
@@ -32,10 +32,8 @@ main() {
group('Watch', () {
Future delayed(computation()) {
Bob Nystrom 2014/06/04 15:48:29 Is this delay still needed? If so, leave a brief c
Anders Johnsen 2014/06/10 06:47:14 Done.
- // On Windows, watching the filesystem is accomplished by polling once
- // per second. So wait 2 seconds to give time for polling to reliably
- // occur.
- return new Future.delayed(new Duration(seconds: 2), computation);
+ return new Future.delayed(
+ new Duration(milliseconds: 100), computation);
}
watchingFolder(String path, test(List<WatchEvent> changesReceived)) {
« no previous file with comments | « no previous file | pkg/pkg.status » ('j') | pkg/watcher/lib/src/directory_watcher/windows.dart » ('J')

Powered by Google App Engine
This is Rietveld 408576698