media: Verify CDM Host files

media/cdm/content_decryption_module_ext.h

Index: media/cdm/content_decryption_module_ext.h
diff --git a/media/cdm/content_decryption_module_ext.h b/media/cdm/content_decryption_module_ext.h
new file mode 100644
index 0000000000000000000000000000000000000000..81d1dfec380546306cf7947c8dca050343c39a86
--- /dev/null
+++ b/media/cdm/content_decryption_module_ext.h
@@ -0,0 +1,55 @@
+// Copyright 2016 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.
+
+#ifndef CDM_CONTENT_DECRYPTION_MODULE_EXT_H_
+#define CDM_CONTENT_DECRYPTION_MODULE_EXT_H_

[xhwang, 2016/12/16 01:06:55]

Adding this new API in a different file since this API will be dynamically found
(using GetFunctionPointerFromNativeLibrary()). Other existing APIs in
media/cdm/api/content_decryption_module.h are statically called by the adapter.

In the future, when we switch to use mojo, all APIs will be dynamically found.

+
+#if defined(WIN32)

[Haoming Chen, 2017/01/04 22:23:39]

s/WIN32/OS_WIN?

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

This interface is intended to be used independent of Chromium, e.g. by other
non-Chromium based browsers. So we should not depend on Chromium defines like
OS_WIN.

+#include <windows.h>
+#endif
+
+#include "media/cdm/content_decryption_module_export.h"
+
+#if defined(_MSC_VER)
+typedef unsigned int uint32_t;
+#else
+#include <stdint.h>
+#endif
+
+#if defined(WIN32)

[Haoming Chen, 2017/01/04 22:23:39]

ditto

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

Acknowledged.

+typedef wchar_t FilePathCharType;
+typedef HANDLE CdmPlatformFile;
+const CdmPlatformFile kInvalidPlatformFile = INVALID_HANDLE_VALUE;
+#elif defined(OS_POSIX)

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

Why not just #else? That way it should compile everywhere.

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

I am not sure whether the file descriptor is an integer on all other platforms.
So even though it'll "compile", it might not work. I'd rather it to fail early
(at compile time).

Also, this is consistent with base::PlatformFile

https://cs.chromium.org/chromium/src/base/files/file.h?rcl=1483993878&l=35

+typedef char FilePathCharType;
+typedef int CdmPlatformFile;
+const CdmPlatformFile kInvalidPlatformFile = -1;
+#endif  // defined(WIN32)
+
+namespace cdm {
+
+struct CdmHostFile {
+  CdmHostFile(const FilePathCharType* file_path,
+              CdmPlatformFile file,
+              CdmPlatformFile sig_file)
+      : file_path(file_path), file(file), sig_file(sig_file) {}
+
+  const FilePathCharType* file_path = nullptr;
+  CdmPlatformFile file = kInvalidPlatformFile;
+  CdmPlatformFile sig_file = kInvalidPlatformFile;
+};
+
+}  // namespace cdm
+
+extern "C" {
+
+// Verifies CDM host files, which are opened in read-only mode and passed in
+// |cdm_host_files|. The CDM should return as soon as possible and process the
+// files asynchronously. All files should be closed by the CDM after the
+// processing is finished.
+CDM_API void VerifyHostFiles(const cdm::CdmHostFile* cdm_host_files,
+                             uint32_t num_files);
+}
+
+#endif  // CDM_CONTENT_DECRYPTION_MODULE_EXT_H_