Declare the sequence of phases for the full polymer transform and add tests for

Declare the sequence of phases for the full polymer transform and add tests for
it.

R=jmesserly@google.com

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

[Siggi Cherem (dart-lang), 2013-08-22 18:26:27]

With these changes I got todomvc running on pub-serve on my local machine,
pretty exiciting!

[Jennifer Messerly, 2013-08-23 01:23:11]

lgtm

https://codereview.chromium.org/23011045/diff/6001/pkg/polymer/lib/src/transf...
File pkg/polymer/lib/src/transform/code_extractor.dart (right):

https://codereview.chromium.org/23011045/diff/6001/pkg/polymer/lib/src/transf...
pkg/polymer/lib/src/transform/code_extractor.dart:38: count++;
instead of count, a boolean "htmlChanged" might be more clear.

https://codereview.chromium.org/23011045/diff/6001/pkg/polymer/lib/src/transf...
pkg/polymer/lib/src/transform/code_extractor.dart:45: tag.attributes['src'] =
'$filename.$count.dart';
I forgot the outcome of previous CL discussion, should we have a TODO here to
ensure this filename is unique?

[Siggi Cherem (dart-lang), 2013-08-23 01:45:28]

thx!

https://codereview.chromium.org/23011045/diff/6001/pkg/polymer/lib/src/transf...
File pkg/polymer/lib/src/transform/code_extractor.dart (right):

https://codereview.chromium.org/23011045/diff/6001/pkg/polymer/lib/src/transf...
pkg/polymer/lib/src/transform/code_extractor.dart:38: count++;
On 2013/08/23 01:23:11, John Messerly wrote:
> instead of count, a boolean "htmlChanged" might be more clear.

Done.

https://codereview.chromium.org/23011045/diff/6001/pkg/polymer/lib/src/transf...
pkg/polymer/lib/src/transform/code_extractor.dart:45: tag.attributes['src'] =
'$filename.$count.dart';
On 2013/08/23 01:23:11, John Messerly wrote:
> I forgot the outcome of previous CL discussion, should we have a TODO here to
> ensure this filename is unique?

Added TODO. Yeah, we weren't sure how necessary it is to worry about it right
away, so meanwhile  I opened this bug for barback:
https://code.google.com/p/dart/issues/detail?id=12618

[Siggi Cherem (dart-lang), 2013-08-23 01:45:58]

Committed patchset #2 manually as r26574 (presubmit successful).