media: Verify CDM Host files

media/cdm/ppapi/external_clear_key/clear_key_cdm.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/cdm/ppapi/external_clear_key/clear_key_cdm.h"
 
 #include <algorithm>
 #include <cstring>
 #include <sstream>
 #include <utility>
 
 #include "base/bind.h"
+#include "base/files/file.h"
 #include "base/logging.h"
 #include "base/macros.h"
 #include "base/time/time.h"
 #include "base/trace_event/trace_event.h"
 #include "media/base/cdm_callback_promise.h"
 #include "media/base/cdm_key_information.h"
 #include "media/base/decoder_buffer.h"
 #include "media/base/decrypt_config.h"
+#include "media/cdm/content_decryption_module_ext.h"
 #include "media/cdm/json_web_key.h"
 #include "media/cdm/ppapi/cdm_file_io_test.h"
 #include "media/cdm/ppapi/external_clear_key/cdm_video_decoder.h"
 #include "url/gurl.h"
 
 #if defined(CLEAR_KEY_CDM_USE_FAKE_AUDIO_DECODER)
 const int64_t kNoTimestamp = INT64_MIN;
 #endif  // CLEAR_KEY_CDM_USE_FAKE_AUDIO_DECODER
 
 #if defined(CLEAR_KEY_CDM_USE_FFMPEG_DECODER)
@@ skipping 219 matching lines @@
 
   // TODO(jrummell): Obtain the proper origin for this instance.
   GURL empty_origin;
   return new media::ClearKeyCdm(host, key_system_string, empty_origin);
 }
 
 const char* GetCdmVersion() {
   return kClearKeyCdmVersion;
 }
 
+// TODO(xhwang): Add a browser test to cover this path.
+void VerifyHostFiles(const cdm::CdmHostFile* cdm_host_files,
+                     uint32_t num_files) {
+  DVLOG(1) << __FUNCTION__;
+
+  LOG(ERROR) << "num_files: " << num_files;
+  for (uint32_t i = 0; i < num_files; ++i) {
+    LOG(ERROR) << "Path " << i << ": " << cdm_host_files[i].file_path;

[tinskip1, 2017/01/09 23:49:30]

Why is this necessary? Let the CDM figure out if there are any issues with the
files?

[xhwang, 2017/01/12 20:15:02]

This is only for my own logging. Will remove.

[xhwang, 2017/01/18 06:03:59]

Done.

+    std::vector<char> buffer(10);
+
+    base::File file(static_cast<base::PlatformFile>(cdm_host_files[i].file));
+    int bytes_read = file.Read(0, buffer.data(), buffer.size());
+    LOG(ERROR) << "File bytes read: " << bytes_read;
+    DCHECK_EQ(10, bytes_read) << "Read failed.";
+
+    base::File sig_file(
+        static_cast<base::PlatformFile>(cdm_host_files[i].sig_file));
+    bytes_read = sig_file.Read(0, buffer.data(), buffer.size());
+    LOG(ERROR) << "Sig file bytes read: " << bytes_read;
+    DCHECK_EQ(10, bytes_read) << "Read failed.";
+  }

[jrummell, 2016/12/16 20:21:30]

How about trying write() and checking that it fails?

[xhwang, 2017/01/12 20:15:02]

Good point. Will do.

[xhwang, 2017/01/18 06:03:59]

Done.

+
+  // TODO(xhwang): Close all files.
+}
+
 namespace media {
 
 ClearKeyCdm::ClearKeyCdm(ClearKeyCdmHost* host,
                          const std::string& key_system,
                          const GURL& origin)
     : decryptor_(new AesDecryptor(
           origin,
           base::Bind(&ClearKeyCdm::OnSessionMessage, base::Unretained(this)),
           base::Bind(&ClearKeyCdm::OnSessionClosed, base::Unretained(this)),
           base::Bind(&ClearKeyCdm::OnSessionKeysChange,
@@ skipping 707 matching lines @@
   is_running_platform_verification_test_ = true;
 
   std::string service_id = "test_service_id";
   std::string challenge = "test_challenge";
 
   host_->SendPlatformChallenge(service_id.data(), service_id.size(),
                                challenge.data(), challenge.size());
 }
 
 }  // namespace media