pkg/watcher: prepare for 0.9.3

pkg/watcher: prepare for 0.9.3

added changelog
updated pubspec
code formatting
removed unused imports

R=ajohnsen@google.com

Committed: https://code.google.com/p/dart/source/detail?r=38862

[kevmoo, 2014-08-02 18:59:50]

We've gone back and forth a lot of this. Can we land what we have so far and
ponder further optimizations in the next release?

[Anders Johnsen, 2014-08-04 06:59:58]

lgtm (but please wait for https://codereview.chromium.org/428493004/ to land -
you are more than welcome to land this for me, if approved).

https://codereview.chromium.org/441483004/diff/2/pkg/watcher/test/directory_w...
File pkg/watcher/test/directory_watcher/linux_test.dart (right):

https://codereview.chromium.org/441483004/diff/2/pkg/watcher/test/directory_w...
pkg/watcher/test/directory_watcher/linux_test.dart:12: void main() {
I don't really see this this as an improvement.

[kevmoo, 2014-08-04 15:55:16]

Committed patchset #5 manually as 38862 (presubmit successful).

[kevmoo, 2014-08-04 16:06:01]

https://codereview.chromium.org/441483004/diff/2/pkg/watcher/test/directory_w...
File pkg/watcher/test/directory_watcher/linux_test.dart (right):

https://codereview.chromium.org/441483004/diff/2/pkg/watcher/test/directory_w...
pkg/watcher/test/directory_watcher/linux_test.dart:12: void main() {
On 2014/08/04 06:59:58, Anders Johnsen wrote:
> I don't really see this this as an improvement.

Eh. I think it's good to follow our own style guide. :-)

[nweiz, 2014-08-04 17:48:41]

https://codereview.chromium.org/441483004/diff/60001/pkg/watcher/pubspec.yaml
File pkg/watcher/pubspec.yaml (right):

https://codereview.chromium.org/441483004/diff/60001/pkg/watcher/pubspec.yaml...
pkg/watcher/pubspec.yaml:9: sdk: '>=1.0.0 <2.0.0'
Is this constraint accurate? There were a number of workarounds in this code for
bugs in earlier versions of dart:io's [Directory.watch] that have since been
removed as the bugs were fixed.

[kevmoo, 2014-08-08 21:57:06]

On 2014/08/04 17:48:41, nweiz wrote:
> https://codereview.chromium.org/441483004/diff/60001/pkg/watcher/pubspec.yaml
> File pkg/watcher/pubspec.yaml (right):
> 
>
https://codereview.chromium.org/441483004/diff/60001/pkg/watcher/pubspec.yaml...
> pkg/watcher/pubspec.yaml:9: sdk: '>=1.0.0 <2.0.0'
> Is this constraint accurate? There were a number of workarounds in this code
for
> bugs in earlier versions of dart:io's [Directory.watch] that have since been
> removed as the bugs were fixed.

Added a bug to track this: https://code.google.com/p/dart/issues/detail?id=20434

I'm not sure how to determine what the new min should be. There were a number of
CLS + reverts to watcher recently, but the net result was no change to the
pubspec min version.