Wrap Directory.watch on Mac OS for the watcher package.

[email protected]
BUG=14428

Review URL: https://codereview.chromium.org//66163002

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@30168 260f80e4-7a28-3924-810f-c04153c831b5

Author: nex3

pkg/pkg.status

@@ -330,5 +330,11 @@ unittest/test/mock_test: StaticWarning, OK # testing unimplemented members
 [ $runtime == vm && ($system == windows || $system == macos) ]
 watcher/test/*/linux_test: Skip
 
+[ $runtime == vm && ($system == windows || $system == linux) ]
+watcher/test/*/mac_os_test: Skip
+
 [ $runtime == vm && $system == linux ]
 watcher/test/*/linux_test: Pass, Slow # Issue 14606
+
+[ $runtime == vm && $system == macos ]
+watcher/test/no_subscribers/mac_os_test: Fail # Issue 14793 (see test file for details)

pkg/watcher/lib/src/constructable_file_system_event.dart

@@ -0,0 +1,60 @@
+// Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
+// for details. All rights reserved. Use of this source code is governed by a
+// BSD-style license that can be found in the LICENSE file.
+
+library watcher.constructable_file_system_event;
+
+import 'dart:io';
+
+abstract class _ConstructableFileSystemEvent implements FileSystemEvent {
+  final bool isDirectory;
+  final String path;
+  final int type;
+
+  _ConstructableFileSystemEvent(this.path, this.isDirectory);
+}
+
+class ConstructableFileSystemCreateEvent extends _ConstructableFileSystemEvent
+    implements FileSystemCreateEvent {
+  final type = FileSystemEvent.CREATE;
+
+  ConstructableFileSystemCreateEvent(String path, bool isDirectory)
+      : super(path, isDirectory);
+
+  String toString() => "FileSystemCreateEvent('$path')";
+}
+
+class ConstructableFileSystemDeleteEvent extends _ConstructableFileSystemEvent
+    implements FileSystemDeleteEvent {
+  final type = FileSystemEvent.DELETE;
+
+  ConstructableFileSystemDeleteEvent(String path, bool isDirectory)
+      : super(path, isDirectory);
+
+  String toString() => "FileSystemDeleteEvent('$path')";
+}
+
+class ConstructableFileSystemModifyEvent extends _ConstructableFileSystemEvent
+    implements FileSystemModifyEvent {
+  final bool contentChanged;
+  final type = FileSystemEvent.MODIFY;
+
+  ConstructableFileSystemModifyEvent(String path, bool isDirectory,
+          this.contentChanged)
+      : super(path, isDirectory);
+
+  String toString() =>
+    "FileSystemModifyEvent('$path', contentChanged=$contentChanged)";
+}
+
+class ConstructableFileSystemMoveEvent extends _ConstructableFileSystemEvent
+    implements FileSystemMoveEvent {
+  final String destination;
+  final type = FileSystemEvent.MOVE;
+
+  ConstructableFileSystemMoveEvent(String path, bool isDirectory,
+          this.destination)
+      : super(path, isDirectory);
+
+  String toString() => "FileSystemMoveEvent('$path', '$destination')";
+}

pkg/watcher/lib/src/directory_watcher.dart

@@ -9,6 +9,7 @@ import 'dart:io';
 
 import 'watch_event.dart';
 import 'directory_watcher/linux.dart';
+import 'directory_watcher/mac_os.dart';
 import 'directory_watcher/polling.dart';
 
 /// Watches the contents of a directory and emits [WatchEvent]s when something
@@ -54,6 +55,7 @@ abstract class DirectoryWatcher {
   /// watchers.
   factory DirectoryWatcher(String directory, {Duration pollingDelay}) {
     if (Platform.isLinux) return new LinuxDirectoryWatcher(directory);
+    if (Platform.isMacOS) return new MacOSDirectoryWatcher(directory);
     return new PollingDirectoryWatcher(directory, pollingDelay: pollingDelay);
   }
 }

pkg/watcher/lib/src/directory_watcher/linux.dart

@@ -12,8 +12,6 @@ import '../utils.dart';
 import '../watch_event.dart';
 import 'resubscribable.dart';
 
-import 'package:stack_trace/stack_trace.dart';
-
 /// Uses the inotify subsystem to watch for filesystem events.
 ///
 /// Inotify doesn't suport recursively watching subdirectories, nor does