Adds a sample for polymer internationalization/localization

Adds a sample for polymer internationalization/localization

BUG=
R=jmesserly@google.com, shailentuli@google.com

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

[Alan Knight, 2013-12-04 19:14:35]

I note there's one glitch in this which is that the select doesn't show up with
a value initially, I'm not sure why. And I don't know why Rietveld is showing
diffs for some of these. They're all new files.

[sethladd, 2013-12-04 19:43:33]

Related to https://code.google.com/p/dart/issues/detail?id=13922 ?

[Alan Knight, 2013-12-06 21:50:35]

Ping

[Jennifer Messerly, 2013-12-06 22:19:44]

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
File samples/polymer_intl/build.dart (right):

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
samples/polymer_intl/build.dart:4: build(entryPoints:
['web/polymerinternationalizedmessagesexample.html'],
this could really use a shorter name :)

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
File samples/polymer_intl/web/localized.html (right):

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
samples/polymer_intl/web/localized.html:3: <h1>Polymer Internationalized
Messages Example</h1>
-2 indent

[Jennifer Messerly, 2013-12-06 22:20:00]

On 2013/12/06 22:19:44, John Messerly wrote:
>
https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
> File samples/polymer_intl/build.dart (right):
> 
>
https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
> samples/polymer_intl/build.dart:4: build(entryPoints:
> ['web/polymerinternationalizedmessagesexample.html'],
> this could really use a shorter name :)
> 
>
https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
> File samples/polymer_intl/web/localized.html (right):
> 
>
https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
> samples/polymer_intl/web/localized.html:3: <h1>Polymer Internationalized
> Messages Example</h1>
> -2 indent

p.s. thanks for ping, I didn't see this the first time :)

[shailentuli, 2013-12-06 22:33:38]

Alan, thanks for putting together the sample.  I'm giving it a quick first look.
Sorry, it took so long. 

Three general comments:

1) Every file needs a copyright notice. 

2) The sample needs a README.  It doesn't have to be long, but it should explain
briefly how the user can run the app, and how it is put together.

3) The sample needs at least an analyzer test.

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
File samples/polymer_intl/build.dart (right):

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
samples/polymer_intl/build.dart:4: build(entryPoints:
['web/polymerinternationalizedmessagesexample.html'],
Agree with John. And underscore_case.

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
File samples/polymer_intl/pubspec.yaml (right):

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
samples/polymer_intl/pubspec.yaml:1: name:
PolymerInternationalizedMessagesExample
underscore_case, not camelCase.

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
File samples/polymer_intl/web/messages_fr.dart (right):

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
samples/polymer_intl/web/messages_fr.dart:9: import 'package:intl/intl.dart';
unused import.

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
samples/polymer_intl/web/messages_fr.dart:18: 
Nit: remove extra blank line.

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
File samples/polymer_intl/web/messages_pt.dart (right):

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
samples/polymer_intl/web/messages_pt.dart:9: import 'package:intl/intl.dart';
Unused import.

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
samples/polymer_intl/web/messages_pt.dart:18: 
Nit: extra blank line.

[Jennifer Messerly, 2013-12-06 23:09:26]

On 2013/12/06 22:33:38, shailentuli wrote:
> Alan, thanks for putting together the sample.  I'm giving it a quick first
look.
> Sorry, it took so long. 
> 
> Three general comments:
> 
> 1) Every file needs a copyright notice. 

yeah, I wondered that too. Often times (e.g.
https://github.com/angular/angular.js/blob/master/example, todomvc.com) example
code is left without copyright notices, both to make the files cleaner and to
make it clear to users they are free to start from your code with no
restrictions. However, the existing Dart samples do have copyright notices in
every file header :|

[Alan Knight, 2013-12-06 23:52:31]

OK, added copyright notices (Dan had told me we didn't need them in HTML files,
and they seem a little silly in generated files, but added them anyway), a test,
and a README.

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
File samples/polymer_intl/build.dart (right):

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
samples/polymer_intl/build.dart:4: build(entryPoints:
['web/polymerinternationalizedmessagesexample.html'],
On 2013/12/06 22:33:38, shailentuli wrote:
> Agree with John. And underscore_case.

Done.

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
File samples/polymer_intl/pubspec.yaml (right):

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
samples/polymer_intl/pubspec.yaml:1: name:
PolymerInternationalizedMessagesExample
On 2013/12/06 22:33:38, shailentuli wrote:
> underscore_case, not camelCase.

Done.

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
File samples/polymer_intl/web/localized.html (right):

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
samples/polymer_intl/web/localized.html:3: <h1>Polymer Internationalized
Messages Example</h1>
On 2013/12/06 22:19:44, John Messerly wrote:
> -2 indent

Done.

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
File samples/polymer_intl/web/messages_fr.dart (right):

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
samples/polymer_intl/web/messages_fr.dart:9: import 'package:intl/intl.dart';
On 2013/12/06 22:33:38, shailentuli wrote:
> unused import.

OK. But these files are generated, so it'll come back if someone regenerates
them.

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
samples/polymer_intl/web/messages_fr.dart:18: 
On 2013/12/06 22:33:38, shailentuli wrote:
> Nit: remove extra blank line.

Ditto.

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
File samples/polymer_intl/web/messages_pt.dart (right):

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
samples/polymer_intl/web/messages_pt.dart:9: import 'package:intl/intl.dart';
On 2013/12/06 22:33:38, shailentuli wrote:
> Unused import.

Done.

https://chromiumcodereview.appspot.com/105073003/diff/20001/samples/polymer_i...
samples/polymer_intl/web/messages_pt.dart:18: 
On 2013/12/06 22:33:38, shailentuli wrote:
> Nit: extra blank line.

Done.

[Jennifer Messerly, 2013-12-07 01:43:01]

> Dan had told me we didn't need them in HTML files, and they seem a little
silly in generated files, but added them anyway.

yeah, you don't need them in generated files.

[Jennifer Messerly, 2013-12-07 01:43:56]

lgtm

[shailentuli, 2013-12-07 04:48:15]

lgtm

[Alan Knight, 2013-12-09 20:58:53]

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