Allow setting breakpoints specifying the suffix of a file path

Allow setting breakpoints specifying the suffix of a file path

In this version, the VM does not set a breakpoint if the url given
matches the suffix of more than one loaded script.

Partial matches must start with a path separator or with the character immediately following a path separator.

E.g. d.dart will match /tmp/d.dart, but not /tmp/ed.dart.

R=asiva@google.com

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

[hausner, 2014-09-24 20:24:49]

Here's an improved version that requires that path elements must fully match,
eliminating false ambiguous matches.

[hausner, 2014-09-24 23:20:42]

hausner@google.com changed reviewers:
+ asiva@google.com

[hausner, 2014-09-24 23:21:06]

This is ready for review.

[siva, 2014-09-26 20:02:48]

lgtm

https://codereview.chromium.org/602853002/diff/20001/runtime/vm/debugger.cc
File runtime/vm/debugger.cc (right):

https://codereview.chromium.org/602853002/diff/20001/runtime/vm/debugger.cc#n...
runtime/vm/debugger.cc:1907:
GrowableObjectArray::Handle(GrowableObjectArray::New());
....Handle(isolate_, GrowableObjectArray::New());

since you already have the isolate value.

[hausner, 2014-09-26 20:11:30]

Committed patchset #3 (id:40001) manually as r40736 (presubmit successful).

[hausner, 2014-09-26 20:19:12]

Thank you.

https://codereview.chromium.org/602853002/diff/20001/runtime/vm/debugger.cc
File runtime/vm/debugger.cc (right):

https://codereview.chromium.org/602853002/diff/20001/runtime/vm/debugger.cc#n...
runtime/vm/debugger.cc:1907:
GrowableObjectArray::Handle(GrowableObjectArray::New());
On 2014/09/26 20:02:48, siva wrote:
> ....Handle(isolate_, GrowableObjectArray::New());
> 
> since you already have the isolate value.

Done.