[Media Router WebUI] Remove unnecessary default property values.

[Media Router WebUI] Remove unnecessary default property values.

This change removes default property values if we don't need them at render time. This cuts down initial load time because we avoid unnecessary function (e.g. observer) calls or data binding in the DOM when we only have a placeholder value. The extra calls/bindings happen when properties are set.

On a Linux machine using a debug build, the UI load time is:
- before change: 3533.77ms (average 20 runs)
- after charge: 3468.72ms (average 20 runs)

There is a 1.8% decrease in initial load time observed.

BUG=549289
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/f02873962bcfd7a419e9d67e237aedfcee8e2262
Cr-Commit-Position: refs/heads/master@{#391173}

[apacible, 2016-04-29 16:19:21]

Description was changed from

==========
default

BUG=
==========

to

==========
[Media Router WebUI] Remove unnecessary default property values.

BUG=549289
==========

[apacible, 2016-04-29 16:29:55]

Patchset #1 (id:1) has been deleted

[apacible, 2016-04-29 16:52:15]

Patchset #1 (id:20001) has been deleted

[apacible, 2016-04-29 17:24:00]

Description was changed from

==========
[Media Router WebUI] Remove unnecessary default property values.

BUG=549289
==========

to

==========
[Media Router WebUI] Remove unnecessary default property values.

This change removes default property values if we don't need them at render
time. This cuts down initial load time because we avoid unnecessary function
(e.g. observer) calls or data binding in the DOM when we only have a placeholder
value. The extra calls/bindings happen when properties are set.

3,533.77

BUG=549289
==========

[apacible, 2016-04-29 17:26:41]

Description was changed from

==========
[Media Router WebUI] Remove unnecessary default property values.

This change removes default property values if we don't need them at render
time. This cuts down initial load time because we avoid unnecessary function
(e.g. observer) calls or data binding in the DOM when we only have a placeholder
value. The extra calls/bindings happen when properties are set.

3,533.77

BUG=549289
==========

to

==========
[Media Router WebUI] Remove unnecessary default property values.

This change removes default property values if we don't need them at render
time. This cuts down initial load time because we avoid unnecessary function
(e.g. observer) calls or data binding in the DOM when we only have a placeholder
value. The extra calls/bindings happen when properties are set.

On a Linux machine using a debug build, the UI load time is:
- before change: 3533.77ms (average 20 runs)
- after charge: 3468.72ms (average 20 runs)

There is a 1.8% decrease in initial load time observed.

BUG=549289
==========

[apacible, 2016-04-29 17:26:41]

apacible@chromium.org changed reviewers:
+ imcheng@chromium.org

[apacible, 2016-04-29 17:28:05]

Patchset #1 (id:40001) has been deleted

[apacible, 2016-04-29 17:31:19]

PTAL, thanks!

This is the first of several patches using recommendations from the perf talk.

[imcheng, 2016-05-02 21:41:56]

Nice! lgtm. Please verify the UI doesn't regress with this change.

[apacible, 2016-05-03 01:31:40]

Description was changed from

==========
[Media Router WebUI] Remove unnecessary default property values.

This change removes default property values if we don't need them at render
time. This cuts down initial load time because we avoid unnecessary function
(e.g. observer) calls or data binding in the DOM when we only have a placeholder
value. The extra calls/bindings happen when properties are set.

On a Linux machine using a debug build, the UI load time is:
- before change: 3533.77ms (average 20 runs)
- after charge: 3468.72ms (average 20 runs)

There is a 1.8% decrease in initial load time observed.

BUG=549289
==========

to

==========
[Media Router WebUI] Remove unnecessary default property values.

This change removes default property values if we don't need them at render
time. This cuts down initial load time because we avoid unnecessary function
(e.g. observer) calls or data binding in the DOM when we only have a placeholder
value. The extra calls/bindings happen when properties are set.

On a Linux machine using a debug build, the UI load time is:
- before change: 3533.77ms (average 20 runs)
- after charge: 3468.72ms (average 20 runs)

There is a 1.8% decrease in initial load time observed.

BUG=549289
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[apacible, 2016-05-03 02:36:28]

On 2016/05/02 21:41:56, imcheng wrote:
> Nice! lgtm. Please verify the UI doesn't regress with this change.

Thanks, manually tested when I sent the patch out and after doing a rebase.

[apacible, 2016-05-03 02:36:35]

The CQ bit was checked by apacible@chromium.org

[apacible, 2016-05-03 02:36:36]

The patchset sent to the CQ was uploaded after l-g-t-m from imcheng@chromium.org
Link to the patchset: https://codereview.chromium.org/1933013002/#ps80001
(title: "Rebase.")

[commit-bot: I haz the power, 2016-05-03 02:36:53]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1933013002/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1933013002/80001

[commit-bot: I haz the power, 2016-05-03 02:42:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-05-03 02:42:43]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[apacible, 2016-05-03 05:35:02]

The CQ bit was checked by apacible@chromium.org

[apacible, 2016-05-03 05:35:02]

The patchset sent to the CQ was uploaded after l-g-t-m from imcheng@chromium.org
Link to the patchset: https://codereview.chromium.org/1933013002/#ps100001
(title: "Rebase.")

[commit-bot: I haz the power, 2016-05-03 05:35:11]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1933013002/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1933013002/100001

[commit-bot: I haz the power, 2016-05-03 05:48:42]

Committed patchset #3 (id:100001)

[commit-bot: I haz the power, 2016-05-03 05:50:04]

Description was changed from

==========
[Media Router WebUI] Remove unnecessary default property values.

This change removes default property values if we don't need them at render
time. This cuts down initial load time because we avoid unnecessary function
(e.g. observer) calls or data binding in the DOM when we only have a placeholder
value. The extra calls/bindings happen when properties are set.

On a Linux machine using a debug build, the UI load time is:
- before change: 3533.77ms (average 20 runs)
- after charge: 3468.72ms (average 20 runs)

There is a 1.8% decrease in initial load time observed.

BUG=549289
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
[Media Router WebUI] Remove unnecessary default property values.

This change removes default property values if we don't need them at render
time. This cuts down initial load time because we avoid unnecessary function
(e.g. observer) calls or data binding in the DOM when we only have a placeholder
value. The extra calls/bindings happen when properties are set.

On a Linux machine using a debug build, the UI load time is:
- before change: 3533.77ms (average 20 runs)
- after charge: 3468.72ms (average 20 runs)

There is a 1.8% decrease in initial load time observed.

BUG=549289
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/f02873962bcfd7a419e9d67e237aedfcee8e2262
Cr-Commit-Position: refs/heads/master@{#391173}
==========

[commit-bot: I haz the power, 2016-05-03 05:50:05]

Patchset 3 (id:??) landed as
https://crrev.com/f02873962bcfd7a419e9d67e237aedfcee8e2262
Cr-Commit-Position: refs/heads/master@{#391173}