Use 'pssh' data to determine key_id properly

media/cdm/cenc_utils.h

Index: media/cdm/cenc_utils.h
diff --git a/media/cdm/cenc_utils.h b/media/cdm/cenc_utils.h
index ec85fb34e8b2f36728ada7438708c2eaf741d8be..23d6c131408c86e77f85bee08437faaac93b7096 100644
--- a/media/cdm/cenc_utils.h
+++ b/media/cdm/cenc_utils.h
@@ -21,10 +21,6 @@ MEDIA_EXPORT bool ValidatePsshInput(const std::vector<uint8_t>& input);
 // more concatenated 'pssh' boxes. If |input| looks valid, then true is
 // returned and |key_ids| is updated to contain the values found. Otherwise
 // return false.
-// TODO(jrummell): This returns true if no Common SystemID 'pssh' boxes are
-// found, or are included but don't contain any key IDs. This should be
-// fixed once the test files are updated to include correct 'pssh' boxes.
-// http://crbug.com/460308
 MEDIA_EXPORT bool GetKeyIdsForCommonSystemId(const std::vector<uint8_t>& input,

[ddorwin, 2015/06/04 19:57:17]

Are there any call sites that were checking for the bool and the size that we
can now clean up?

[jrummell, 2015/06/15 22:22:09]

Nope. The only caller (other than tests) is aes_decryptor.cc, and it will simply
generate a license request with whatever keys it gets back (which could have
been none).

                                              KeyIdList* key_ids);