Rework how ServiceObjects are created and cached in Observatory.

Rework how ServiceObjects are created and cached in Observatory.

We now use a single cache, rather than per-type caches which simplifies the get() logic and makes it easier to add more model-types in the future.

Rework how isolates are created/stored/retrieved.  IsolateList is gone.

Add canCache and immutable to ServiceObjects and use them to inform when an object should be cached or reloaded.

Remove cache.dart, service.dart.

Add a notion of a ServiceObjectOwner.

Cleanups.

Etc.

Etc.

R=johnmccutchan@google.com

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

[Cutch, 2014-03-24 20:28:28]

lgtm with comments.

https://codereview.chromium.org/204983008/diff/40001/runtime/bin/vmservice/cl...
File runtime/bin/vmservice/client/lib/src/service/object.dart (right):

https://codereview.chromium.org/204983008/diff/40001/runtime/bin/vmservice/cl...
runtime/bin/vmservice/client/lib/src/service/object.dart:9: bool
_isServiceMap(ObservableMap m) {
Maybe move these loose functions to the bottom of the file.

https://codereview.chromium.org/204983008/diff/40001/runtime/bin/vmservice/cl...
runtime/bin/vmservice/client/lib/src/service/object.dart:96: // Not a reference.
Update comment here.

https://codereview.chromium.org/204983008/diff/40001/runtime/bin/vmservice/cl...
runtime/bin/vmservice/client/lib/src/service/object.dart:228: if
(id.startsWith('isolates/')) {
Make 'isolates/' into a constant string and put it up near the matchers.

[turnidge, 2014-03-24 21:38:36]

https://codereview.chromium.org/204983008/diff/40001/runtime/bin/vmservice/cl...
File runtime/bin/vmservice/client/lib/src/service/object.dart (right):

https://codereview.chromium.org/204983008/diff/40001/runtime/bin/vmservice/cl...
runtime/bin/vmservice/client/lib/src/service/object.dart:9: bool
_isServiceMap(ObservableMap m) {
On 2014/03/24 20:28:29, Cutch wrote:
> Maybe move these loose functions to the bottom of the file.

Done.

https://codereview.chromium.org/204983008/diff/40001/runtime/bin/vmservice/cl...
runtime/bin/vmservice/client/lib/src/service/object.dart:96: // Not a reference.
On 2014/03/24 20:28:29, Cutch wrote:
> Update comment here.

Done.

https://codereview.chromium.org/204983008/diff/40001/runtime/bin/vmservice/cl...
runtime/bin/vmservice/client/lib/src/service/object.dart:228: if
(id.startsWith('isolates/')) {
On 2014/03/24 20:28:29, Cutch wrote:
> Make 'isolates/' into a constant string and put it up near the matchers.

Done.

[turnidge, 2014-03-24 21:42:38]

Committed patchset #4 manually as r34333 (presubmit successful).