update web_components

pkg/polymer/test/run.sh

 #!/bin/bash
 # Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 # for details. All rights reserved. Use of this source code is governed by a
 # BSD-style license that can be found in the LICENSE file.
 
 # Usage: run from a Dart SVN checkout after building.
 
 # bail on error
 set -e
 
 # Note: test.dart needs to be run from the root of the Dart checkout
 DIR=$( cd $( dirname "${BASH_SOURCE[0]}" ) && pwd )

[blois, 2014/02/12 02:29:31]

should be readonly.

 pushd $DIR/../../.. > /dev/null

[blois, 2014/02/12 02:29:31]

should be quoted

 
 echo "*** Running unit tests for Polymer.dart and its dependencies."
 
-SUITES="pkg/polymer samples/third_party/todomvc"
+# Note: you can't combine two different pkg/ selectors in the same command.
+# Also, test.py gets less happy the more tests that are run.
+# So we run all of the filters separately.
+for SUITES in pkg/observe html/custom pkg/template_binding pkg/polymer samples/third_party/todomvc

[blois, 2014/02/12 02:29:31]

line length

[blois, 2014/02/12 02:29:31]

for loop variables (SUITES) should be lower_cased.

+do
+  ANALYZER_CONFIG="-m release -r none -c dartanalyzer $*"

[blois, 2014/02/12 02:29:31]

should use readonly:
readonly ANALYZER_CONFIG="-m release -r none -c dartanalyzer $*"

+  CONFIG="-m release -r vm,drt,ff,chrome,d8,jsshell -c none,dart2js --checked $*"
 
-ANALYZER_CONFIG="-m release -r none -c dartanalyzer $*"
-CONFIG="-m release -r vm,drt,ff,chrome,d8,jsshell -c none,dart2js --checked $*"
+  CMD="./tools/test.py $ANALYZER_CONFIG $SUITES"

[blois, 2014/02/12 02:29:31]

should expand variables using:
${ANALYZER}

[blois, 2014/02/12 02:29:31]

CMD is being assigned to multiple times, should not be in caps.

+  echo "*** $CMD"
+  $CMD
 
-CMD="./tools/test.py $ANALYZER_CONFIG $SUITES"
-echo "*** $CMD"
-$CMD
-
-CMD="xvfb-run ./tools/test.py $CONFIG $SUITES"
-echo "*** $CMD"
-$CMD
+  CMD="xvfb-run ./tools/test.py $CONFIG $SUITES"
+  echo "*** $CMD"
+  $CMD
+done
 
 echo -e "All tests pass"