FileAPI: Remove TODOs around readAsBinaryString - it's back in the spec

FileAPI: Remove TODOs around readAsBinaryString - it's back in the spec

FileReader.readAsBinaryString() was removed from the spec after typed
arrays became a thing. But all browsers kept shipping it, and it looks
risky to remove, so it was added back to the spec. Remove the use
counters and TODOs in the IDL files.

R=foolip@chromium.org

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

[jsbell, 2016-07-21 22:17:41]

The CQ bit was checked by jsbell@chromium.org to run a CQ dry run

[jsbell, 2016-07-21 22:18:43]

Per https://github.com/w3c/FileAPI/issues/39 FileReader.readAsBinaryString() is
legit again.

foolip@ - can you review?

[commit-bot: I haz the power, 2016-07-21 22:18:43]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[foolip, 2016-07-21 22:23:37]

lgtm

Yay! Pushing back against spec removals like this is great, even if not *always*
the winning strategy.

[commit-bot: I haz the power, 2016-07-22 02:03:21]

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

[commit-bot: I haz the power, 2016-07-22 02:03:22]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[jsbell, 2016-07-22 17:14:26]

The CQ bit was checked by jsbell@chromium.org

[commit-bot: I haz the power, 2016-07-22 17:14:42]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-22 18:06:03]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-22 18:07:23]

Description was changed from

==========
FileAPI: Remove TODOs around readAsBinaryString - it's back in the spec

FileReader.readAsBinaryString() was removed from the spec after typed
arrays became a thing. But all browsers kept shipping it, and it looks
risky to remove, so it was added back to the spec. Remove the use
counters and TODOs in the IDL files.

R=foolip@chromium.org
==========

to

==========
FileAPI: Remove TODOs around readAsBinaryString - it's back in the spec

FileReader.readAsBinaryString() was removed from the spec after typed
arrays became a thing. But all browsers kept shipping it, and it looks
risky to remove, so it was added back to the spec. Remove the use
counters and TODOs in the IDL files.

R=foolip@chromium.org

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

[commit-bot: I haz the power, 2016-07-22 18:07:24]

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