Replace Glob implementation.

Replace Glob implementation.

This makes MDL analysis another 5% faster.

R=brianwilkerson@google.com
BUG=

Committed: https://github.com/dart-lang/sdk/commit/508b70bdabe4a2ca37435eea5686eebe8e77d2b4

[Brian Wilkerson, 2015-11-15 16:27:47]

https://codereview.chromium.org/1452473002/diff/1/pkg/analyzer/lib/src/util/g...
File pkg/analyzer/lib/src/util/glob.dart (right):

https://codereview.chromium.org/1452473002/diff/1/pkg/analyzer/lib/src/util/g...
pkg/analyzer/lib/src/util/glob.dart:18: class Glob {
Why not use the implementation from quiver?

https://codereview.chromium.org/1452473002/diff/1/pkg/analyzer/test/src/util/...
File pkg/analyzer/test/src/util/glob_test.dart (right):

https://codereview.chromium.org/1452473002/diff/1/pkg/analyzer/test/src/util/...
pkg/analyzer/test/src/util/glob_test.dart:60: class GlobWindowsTest {
None of the paths in these tests contain drive letters.

[scheglov, 2015-11-15 18:42:12]

PTAL

https://codereview.chromium.org/1452473002/diff/1/pkg/analyzer/lib/src/util/g...
File pkg/analyzer/lib/src/util/glob.dart (right):

https://codereview.chromium.org/1452473002/diff/1/pkg/analyzer/lib/src/util/g...
pkg/analyzer/lib/src/util/glob.dart:18: class Glob {
On 2015/11/15 16:27:47, Brian Wilkerson wrote:
> Why not use the implementation from quiver?

We do, partially.
But we need to solve the following problems:

1. It does not automatically replace Windows path separator with Posix during
matching.
    In general it is not required for wildcards implementation, but I think it
is useful for globs.

2. It is case sensitive. Again, it is useful for blogs.

Also, at some point we might decide to use an optimized implementation for
**/*.dart - just use endsWith(".dart") for example. I want to run couple
micro-benchmarks later.

https://codereview.chromium.org/1452473002/diff/1/pkg/analyzer/test/src/util/...
File pkg/analyzer/test/src/util/glob_test.dart (right):

https://codereview.chromium.org/1452473002/diff/1/pkg/analyzer/test/src/util/...
pkg/analyzer/test/src/util/glob_test.dart:60: class GlobWindowsTest {
On 2015/11/15 16:27:47, Brian Wilkerson wrote:
> None of the paths in these tests contain drive letters.

Done.

[Brian Wilkerson, 2015-11-15 19:25:10]

LGTM

[scheglov, 2015-11-15 20:17:22]

Committed patchset #2 (id:20001) manually as
508b70bdabe4a2ca37435eea5686eebe8e77d2b4 (presubmit successful).