Changes in smoke in preparation of polymer's codegen:

These are 3 changes in smoke in preparation of polymer's codegen. I happened to make them at once, but let me know if you prefer that I split them up a bit:

(a) add filter to query. This makes it easier to support *Changed  without having to generate metadata about lots of useless methods. We might remove *Changed in the future, but I still think this is a nice feature for smoke.

(b) changed recorder.dart to automatically include getters/setters for queries. It is the most common use case that you query a member because you want both the metadata and to read/write/invoke it.

(c) add a new smoke implementation to make it easier to debug inconsistencies between static vs mirrors implementations. (I used this while debugging stuffin polymer)

R=jmesserly@google.com

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

[Siggi Cherem (dart-lang), 2014-03-19 00:07:01]

https://codereview.chromium.org/204143002/diff/20001/pkg/polymer/lib/polymer....
File pkg/polymer/lib/polymer.dart (left):

https://codereview.chromium.org/204143002/diff/20001/pkg/polymer/lib/polymer....
pkg/polymer/lib/polymer.dart:79: import 'package:smoke/mirrors.dart' as smoke;
this was used to call "useMirrors" in loader, that logic is now under
mirrors_loader.dart

[Siggi Cherem (dart-lang), 2014-03-19 00:09:53]

https://codereview.chromium.org/204143002/diff/20001/pkg/smoke/pubspec.yaml
File pkg/smoke/pubspec.yaml (right):

https://codereview.chromium.org/204143002/diff/20001/pkg/smoke/pubspec.yaml#n...
pkg/smoke/pubspec.yaml:2: version: 0.1.0-dev
this could also be -pre.1.dev, but I figure if our intent is to never publish
-dev, we can use it as a default anytime we are developing something that has
not been published

thoughts?

[Jennifer Messerly, 2014-03-19 00:18:54]

lgtm

https://codereview.chromium.org/204143002/diff/20001/pkg/smoke/pubspec.yaml
File pkg/smoke/pubspec.yaml (right):

https://codereview.chromium.org/204143002/diff/20001/pkg/smoke/pubspec.yaml#n...
pkg/smoke/pubspec.yaml:2: version: 0.1.0-dev
On 2014/03/19 00:09:53, Siggi Cherem (dart-lang) wrote:
> this could also be -pre.1.dev, but I figure if our intent is to never publish
> -dev, we can use it as a default anytime we are developing something that has
> not been published
> 
> thoughts?

SGTM

[Siggi Cherem (dart-lang), 2014-03-19 16:27:39]

Committed patchset #3 manually as r34117 (presubmit successful).