MediaRecorder: content browsertests fix a few lint warnings in mediarecorder_test.html

MediaRecorder: content browsertests fix a few lint warnings in mediarecorder_test.html

This CL fixes a few lint warnings scattered around
mediarecorder_test.html. No new functionality added.

BUG=575323
Test=On linux host and trybots.

Committed: https://crrev.com/7831a89af115138ba54b3e9b64f5aa040a566853
Cr-Commit-Position: refs/heads/master@{#368643}

[cpaulin (no longer in chrome), 2016-01-07 22:30:42]

cpaulin@chromium.org changed reviewers:
+ mcasas@chromium.org

[cpaulin (no longer in chrome), 2016-01-07 22:30:42]

This CL fixes the lint warnings in mediarecorder_test.html, no new functionality
added. Please review at your leisure.

[cpaulin (no longer in chrome), 2016-01-07 22:48:33]

Description was changed from

==========
MediaRecorder: fix a few lint errors in mediarecorder_test.html

This CL fixes a few lint warnings scattered around
mediarecorder_test.html. No new functionality added.

BUG=575323
Test=On linux host and trybots.
==========

to

==========
MediaRecorder: fix a few lint warnings in mediarecorder_test.html

This CL fixes a few lint warnings scattered around
mediarecorder_test.html. No new functionality added.

BUG=575323
Test=On linux host and trybots.
==========

[mcasas, 2016-01-07 23:04:11]

https://codereview.chromium.org/1565343002/diff/1/content/test/data/media/med...
File content/test/data/media/mediarecorder_test.html (right):

https://codereview.chromium.org/1565343002/diff/1/content/test/data/media/med...
content/test/data/media/mediarecorder_test.html:360: var videoSize = 0;
Unused as well, as I wrote in the bug.

[cpaulin (no longer in chrome), 2016-01-07 23:09:11]

cpaulin@chromium.org changed reviewers:
+ phoglund@chromium.org

[cpaulin (no longer in chrome), 2016-01-07 23:09:12]

Small review to fix lint warnings in mediarecorder_test.html

[cpaulin (no longer in chrome), 2016-01-07 23:14:06]

Removed unused videoSize from testPausePreventsDataavailableFromBeingFired.
Ready for review

https://codereview.chromium.org/1565343002/diff/1/content/test/data/media/med...
File content/test/data/media/mediarecorder_test.html (right):

https://codereview.chromium.org/1565343002/diff/1/content/test/data/media/med...
content/test/data/media/mediarecorder_test.html:360: var videoSize = 0;
On 2016/01/07 23:04:11, mcasas wrote:
> Unused as well, as I wrote in the bug.


That one got away.

[phoglund_chromium, 2016-01-08 09:46:13]

lgtm

[cpaulin (no longer in chrome), 2016-01-08 19:57:08]

mcasas@ PTAL

[cpaulin (no longer in chrome), 2016-01-08 20:49:51]

New Patch set is out: removed the "== true" boolean expressions in 4 locations

[mcasas, 2016-01-08 23:19:28]

lgtm

[cpaulin (no longer in chrome), 2016-01-08 23:38:19]

Description was changed from

==========
MediaRecorder: fix a few lint warnings in mediarecorder_test.html

This CL fixes a few lint warnings scattered around
mediarecorder_test.html. No new functionality added.

BUG=575323
Test=On linux host and trybots.
==========

to

==========
MediaRecorder: content browsertests fix a few lint warning in
mediarecorder_test.html

This CL fixes a few lint warnings scattered around
mediarecorder_test.html. No new functionality added.

BUG=575323
Test=On linux host and trybots.
==========

[cpaulin (no longer in chrome), 2016-01-08 23:39:00]

Description was changed from

==========
MediaRecorder: content browsertests fix a few lint warning in
mediarecorder_test.html

This CL fixes a few lint warnings scattered around
mediarecorder_test.html. No new functionality added.

BUG=575323
Test=On linux host and trybots.
==========

to

==========
MediaRecorder: content browsertests fix a few lint warnings in
mediarecorder_test.html

This CL fixes a few lint warnings scattered around
mediarecorder_test.html. No new functionality added.

BUG=575323
Test=On linux host and trybots.
==========

[cpaulin (no longer in chrome), 2016-01-11 18:38:35]

The CQ bit was checked by cpaulin@chromium.org

[cpaulin (no longer in chrome), 2016-01-11 18:38:35]

The patchset sent to the CQ was uploaded after l-g-t-m from
phoglund@chromium.org
Link to the patchset: https://codereview.chromium.org/1565343002/#ps30001
(title: "Removed the "== true" checks in 4 locations")

[commit-bot: I haz the power, 2016-01-11 18:39:18]

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

[commit-bot: I haz the power, 2016-01-11 19:49:46]

Description was changed from

==========
MediaRecorder: content browsertests fix a few lint warnings in
mediarecorder_test.html

This CL fixes a few lint warnings scattered around
mediarecorder_test.html. No new functionality added.

BUG=575323
Test=On linux host and trybots.
==========

to

==========
MediaRecorder: content browsertests fix a few lint warnings in
mediarecorder_test.html

This CL fixes a few lint warnings scattered around
mediarecorder_test.html. No new functionality added.

BUG=575323
Test=On linux host and trybots.
==========

[commit-bot: I haz the power, 2016-01-11 19:49:46]

Committed patchset #3 (id:30001)

[commit-bot: I haz the power, 2016-01-11 19:50:41]

Description was changed from

==========
MediaRecorder: content browsertests fix a few lint warnings in
mediarecorder_test.html

This CL fixes a few lint warnings scattered around
mediarecorder_test.html. No new functionality added.

BUG=575323
Test=On linux host and trybots.
==========

to

==========
MediaRecorder: content browsertests fix a few lint warnings in
mediarecorder_test.html

This CL fixes a few lint warnings scattered around
mediarecorder_test.html. No new functionality added.

BUG=575323
Test=On linux host and trybots.

Committed: https://crrev.com/7831a89af115138ba54b3e9b64f5aa040a566853
Cr-Commit-Position: refs/heads/master@{#368643}
==========

[commit-bot: I haz the power, 2016-01-11 19:50:42]

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