Rearranges the polymer package now that the old compiler is gone.

Rearranges the polymer package now that the old compiler is gone.

R=jmesserly@google.com

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

[Siggi Cherem (dart-lang), 2013-09-12 01:46:38]

no code changes here, just moving things around.

[Jennifer Messerly, 2013-09-12 02:54:21]

hmm, to be honest I kinda liked the organization of having the build steps
separate from the runtime libraries :)

seemed nice that if someone is looking at our code, they can see the runtime
stuff that matches Polymer.js, and the build stuff in another place. Maybe it
should be "lib/src/build" tho?

https://codereview.chromium.org/23757034/diff/1/pkg/polymer/lib/src/code_extr...
File pkg/polymer/lib/src/code_extractor.dart (right):

https://codereview.chromium.org/23757034/diff/1/pkg/polymer/lib/src/code_extr...
pkg/polymer/lib/src/code_extractor.dart:6: library polymer.src.transformers;
library name?

https://codereview.chromium.org/23757034/diff/1/pkg/polymer/lib/src/common.dart
File pkg/polymer/lib/src/common.dart (left):

https://codereview.chromium.org/23757034/diff/1/pkg/polymer/lib/src/common.da...
pkg/polymer/lib/src/common.dart:6: library polymer.src.transform.common;
hmmm, shouldn't this be under transform, since it's all related to build step?

in general, it'd be nice to separate the build/deploy files from the runtime
files.

https://codereview.chromium.org/23757034/diff/1/pkg/polymer/lib/src/import_in...
File pkg/polymer/lib/src/import_inliner.dart (right):

https://codereview.chromium.org/23757034/diff/1/pkg/polymer/lib/src/import_in...
pkg/polymer/lib/src/import_inliner.dart:6: library
polymer.src.transform.import_inliner;
lib name?

https://codereview.chromium.org/23757034/diff/1/pkg/polymer/lib/src/polyfill_...
File pkg/polymer/lib/src/polyfill_injector.dart (right):

https://codereview.chromium.org/23757034/diff/1/pkg/polymer/lib/src/polyfill_...
pkg/polymer/lib/src/polyfill_injector.dart:9: library
polymer.src.transform.polyfill_injector;
ditto on lib name :). I'll stop mentioning it but it seems to happen in a few
other places

[Siggi Cherem (dart-lang), 2013-09-12 16:58:48]

Part of the reason I was moving things up is that besides 'utils.dart', we don't
have anything else under lib/src/ that isn't build/linter related.

I suppose we may be adding more runtime scripts in the near future, so I can see
the benefit of having src/build/. Let's chat later today when we are both around
to get a better idea of what we are planning to do, and then I'll adjust this CL
accordingly.

[Siggi Cherem (dart-lang), 2013-09-12 23:46:56]

PTAL - I just rearranged things as we discussed and did a couple more moves.
Here is the summary:

- moved all transforms under 'build'
lib/src/transform/* -> lib/src/build/*
test/transform/* -> test/build/*

- moved linter and barback_runner under 'build'
lib/src/linter.dart -> lib/src/build/linter.dart
lib/src/barback_runner.dart -> lib/src/build/runner.dart

- moved 'transform.dart' up, as it will become how pub discovers the
transformers:
lib/src/transform.dart -> lib/transformer.dart

- moved utils test out of 'build/':
test/build/utils_test.dart -> test/utils_test.dart

[Jennifer Messerly, 2013-09-12 23:59:12]

lgtm!

[Siggi Cherem (dart-lang), 2013-09-13 00:18:30]

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