Added .scss transform

Added .scss transform
Initial CSS transform

BUG=

[terry, 2013-09-06 18:05:46]

Added .scss transform and test for transforms.

[Siggi Cherem (dart-lang), 2013-09-08 20:31:33]

magical =)

Thanks Terry, the new transformer code looks good.

My main comment below is to pull out the changes to deploy.dart for now, but
otherwise looks good.

https://chromiumcodereview.appspot.com/23576012/diff/7001/pkg/csslib/lib/pars...
File pkg/csslib/lib/parser.dart (right):

https://chromiumcodereview.appspot.com/23576012/diff/7001/pkg/csslib/lib/pars...
pkg/csslib/lib/parser.dart:10: import 'package:barback/barback.dart';
note barback.dart depends on dart:io, so you might want to pull the transformer
code to be on a separate library.

(I thought so far you needed to have the parser independent of dart:io so it
could be used in the client-side.)

https://chromiumcodereview.appspot.com/23576012/diff/7001/pkg/csslib/lib/tran...
File pkg/csslib/lib/transform.dart (right):

https://chromiumcodereview.appspot.com/23576012/diff/7001/pkg/csslib/lib/tran...
pkg/csslib/lib/transform.dart:21: new Future.value((input.id.extension ==
'.scss'));
replace the above two lines with:

/** Only run this transformer on .scss files. */                                
                  
final String allowedExtensions = ".scss"; 

(it's a new improvement from barback :-))

https://chromiumcodereview.appspot.com/23576012/diff/7001/pkg/csslib/test/tra...
File pkg/csslib/test/transform/common.dart (right):

https://chromiumcodereview.appspot.com/23576012/diff/7001/pkg/csslib/test/tra...
pkg/csslib/test/transform/common.dart:13: String idToString(AssetId id) =>
'${id.package}|${id.path}';
please add a TODO about deleting this file (since it's a copy of
polymer/test/transform/common.dart) and mention dartbug.com/12519

https://chromiumcodereview.appspot.com/23576012/diff/7001/pkg/csslib/test/tra...
pkg/csslib/test/transform/common.dart:101: const INTEROP_TAG = '<script
src="packages/browser/interop.js"></script>\n';
delete these two constants (you probably don't need them)

https://chromiumcodereview.appspot.com/23576012/diff/7001/pkg/polymer/lib/dep...
File pkg/polymer/lib/deploy.dart (left):

https://chromiumcodereview.appspot.com/23576012/diff/7001/pkg/polymer/lib/dep...
pkg/polymer/lib/deploy.dart:6: * Temporary deploy command used to create a
version of the app that can be
Let's revert the changes to lib/deploy.dart bin/deploy.dart for now. We could
move them to a separate CL, but I'm not sure yet what is the right design here.
It might help if we first meet and brainstorm about it in person.

Long story short: So far pub's convention says to put tools under 'bin', but
these tools are not visible if you are installing this package via pub. Right
now the best I've found is to create a build.dart on each user's directory,
which internally calls this script. I think you are right that calling it 'main'
might be confusing, so maybe renaming 'main' to 'deploy' might make more sense?

This is currently on the top of my list for this week.

https://chromiumcodereview.appspot.com/23576012/diff/7001/pkg/polymer/lib/src...
File pkg/polymer/lib/src/transform.dart (right):

https://chromiumcodereview.appspot.com/23576012/diff/7001/pkg/polymer/lib/src...
pkg/polymer/lib/src/transform.dart:25: [new ScssTransformer()],
we should be able to run this in parallel with other phases. So maybe add it to
the same list as Observable?

(I'm thinking that eventually the code-extractor will extract the .css from the
html

[Jennifer Messerly, 2014-06-06 17:31:08]

is this still needed?