Update host verification API

content_decryption_module_ext.h

Index: content_decryption_module_ext.h
diff --git a/content_decryption_module_ext.h b/content_decryption_module_ext.h
index 4a3c10ec56523c187d50a1cdefa7f9cade76e485..59c0835cd0619b6d5dac2af464dbe337611068d0 100644
--- a/content_decryption_module_ext.h
+++ b/content_decryption_module_ext.h
@@ -49,12 +49,16 @@ struct HostFile {
 
 extern "C" {
 
-// Verifies CDM host files, which are opened in read-only mode and passed in
-// |host_files|. The CDM MUST return as soon as possible and process the files
+// Functions in this file are dynamically retrieved by their versioned fucntion

[jrummell, 2017/01/19 18:44:19]

s/fucntion/function/ (last word on line).

[xhwang, 2017/01/19 19:42:18]

Done.

+// name. Increment the version number for any backward incompatible API changes.
+
+// Verifies CDM host. All files in |host_files| are opened in read-only mode.
+//
+// Returns false and close all files if verification failed immediately.
+// Otherwise the CDM MUST return true as soon as possible and process the files
 // asynchronously. All files MUST be closed by the CDM after this one-time
 // processing is finished.
-CDM_API void VerifyHostFiles(const cdm::HostFile* host_files,
-                             uint32_t num_files);
+CDM_API bool VerifyHost_0(const cdm::HostFile* host_files, uint32_t num_files);

[jrummell, 2017/01/19 18:44:19]

Should we also have the typedef for the calling signature here? I noticed in
your chrome change it simply referred to this file, but I wondered if it would
be better to have the typedef here. (And possibly the function name, although I
think #define VerifyHostFunctionName VerifyHost_0 is discouraged.)

[xhwang, 2017/01/19 19:42:18]

Summary of offline discussion.

Typically we don't do them in the header file to keep the interface clean. Also
looking at all examples in Chromium and we don't normally do this. So I'll keep
this as is for now. We can revisit this topic later.

 }
 
 #endif  // CDM_CONTENT_DECRYPTION_MODULE_EXT_H_