Relax DCHECK for |legacy_destination_url|

Relax DCHECK for |legacy_destination_url|

Only license requests should not specify |legacy_destination_url|.

BUG=448242
Committed: https://crrev.com/77f5922a4dabfd5682d25619e80e5104426b4192
Cr-Commit-Position: refs/heads/master@{#311550}

[jrummell, 2015-01-14 20:38:06]

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

[jrummell, 2015-01-14 20:38:06]

PTAL.

[ddorwin, 2015-01-14 21:11:52]

lgtm

https://codereview.chromium.org/804713003/diff/1/media/cdm/ppapi/cdm_adapter.cc
File media/cdm/ppapi/cdm_adapter.cc (right):

https://codereview.chromium.org/804713003/diff/1/media/cdm/ppapi/cdm_adapter....
media/cdm/ppapi/cdm_adapter.cc:725: message_type !=
cdm::MessageType::kLicenseRequest);
The inverted logic won't automatically catch new types. However, this is
probably consistent with how the CDM is implemented, so it's fine. (Plus, this
parameter will be removed sometime this year.)

[jrummell, 2015-01-14 21:42:37]

Thanks for the review.

https://codereview.chromium.org/804713003/diff/1/media/cdm/ppapi/cdm_adapter.cc
File media/cdm/ppapi/cdm_adapter.cc (right):

https://codereview.chromium.org/804713003/diff/1/media/cdm/ppapi/cdm_adapter....
media/cdm/ppapi/cdm_adapter.cc:725: message_type !=
cdm::MessageType::kLicenseRequest);
On 2015/01/14 21:11:52, ddorwin wrote:
> The inverted logic won't automatically catch new types. However, this is
> probably consistent with how the CDM is implemented, so it's fine. (Plus, this
> parameter will be removed sometime this year.)

Acknowledged.

[jrummell, 2015-01-14 21:42:42]

The CQ bit was checked by jrummell@chromium.org

[commit-bot: I haz the power, 2015-01-14 21:43:42]

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

[commit-bot: I haz the power, 2015-01-14 22:03:17]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-01-14 22:04:19]

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