Refactored js code and merged to the master

ok.


On Tue, Jul 15, 2014 at 3:30 PM, Brian Stell <notifications@github.com>
wrote:

> In run_time/src/gae_server/chrome/js/script.js:
>
> > @@ -17,20 +17,136 @@
> >  */
> >
> >
> > +/*
> > + * FilesystemHelper class
> > + * @param {Promise} filesystemReady Promise to filesystem is ready
> > + * @param {Boolean} ASSUME_EMPTY Ignore contents of the filesystem
> > + * @returns {FilesystemHelper}
> > + */
> > +function FilesystemHelper(filesystemReady, ASSUME_EMPTY) {
> > +    this.filesystemReady = filesystemReady;
> > +    this.ASSUME_EMPTY = ASSUME_EMPTY;
>
> Uppercase is typically reserved for constants.
>
> https://google-styleguide.googlecode.com/svn/trunk/javascriptguide.xml#Naming
> "Private properties and methods should be named with a trailing underscore"
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/googlei18n/incremental-fonts/pull/62/files#r14967913>.
>