Sanitize data before providing it to the CDM

Sanitize data before providing it to the CDM

Sanitize |session_id| passed to load() and |response| passed to update().

BUG=456398
TEST=new blink layout tests

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

[jrummell, 2015-05-09 00:36:39]

jrummell@chromium.org changed reviewers:
+ ddorwin@chromium.org, sandersd@chromium.org

[jrummell, 2015-05-09 00:36:39]

PTAL.

[sandersd (OOO until July 31), 2015-05-09 01:06:57]

https://codereview.chromium.org/1132223002/diff/1/media/blink/webcontentdecry...
File media/blink/webcontentdecryptionmodulesession_impl.cc (right):

https://codereview.chromium.org/1132223002/diff/1/media/blink/webcontentdecry...
media/blink/webcontentdecryptionmodulesession_impl.cc:309: "Invalid session
Id.");
ID

https://codereview.chromium.org/1132223002/diff/1/media/cdm/json_web_key.cc
File media/cdm/json_web_key.cc (right):

https://codereview.chromium.org/1132223002/diff/1/media/cdm/json_web_key.cc#n...
media/cdm/json_web_key.cc:136: for (const auto key_pair : keys) {
Should this be key_pair&?

[jrummell, 2015-05-09 01:11:45]

Updated.

https://codereview.chromium.org/1132223002/diff/1/media/blink/webcontentdecry...
File media/blink/webcontentdecryptionmodulesession_impl.cc (right):

https://codereview.chromium.org/1132223002/diff/1/media/blink/webcontentdecry...
media/blink/webcontentdecryptionmodulesession_impl.cc:309: "Invalid session
Id.");
On 2015/05/09 01:06:57, sandersd wrote:
> ID

Done.

https://codereview.chromium.org/1132223002/diff/1/media/cdm/json_web_key.cc
File media/cdm/json_web_key.cc (right):

https://codereview.chromium.org/1132223002/diff/1/media/cdm/json_web_key.cc#n...
media/cdm/json_web_key.cc:136: for (const auto key_pair : keys) {
On 2015/05/09 01:06:57, sandersd wrote:
> Should this be key_pair&?

Done.

[sandersd (OOO until July 31), 2015-05-09 01:15:37]

lgtm

[jrummell, 2015-05-11 22:09:53]

The CQ bit was checked by jrummell@chromium.org

[commit-bot: I haz the power, 2015-05-11 22:12:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1132223002/20001

[commit-bot: I haz the power, 2015-05-11 23:20:04]

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

[commit-bot: I haz the power, 2015-05-11 23:20:05]

Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

[jrummell, 2015-05-12 22:37:18]

Updated to make Windows compile.

[sandersd (OOO until July 31), 2015-05-12 23:05:52]

Still lgtm.

[jrummell, 2015-05-13 00:03:54]

The CQ bit was checked by jrummell@chromium.org

[commit-bot: I haz the power, 2015-05-13 00:05:45]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1132223002/40001

[commit-bot: I haz the power, 2015-05-13 00:10:38]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-05-13 00:11:38]

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