Widevine HAL must not access mediametrics binder service.

Widevine HAL is not part of framework/system image and thus it is not
permitted to use Binder. Binder-like mechanisms which are permitted are
HwBinder  or VendorBinder (for communications only with vendor image
components).

Remove references to MediaAnalyticsItems used by ami_adapter.*.

Test: Play Movies & TV (streaming and pinned content)
Test: Netflix
Test: Widevine unit tests

bug: 36576915
Change-Id: I786f206c01d7c5ebfce719f3f51f5f7927f0359e
This commit is contained in:
Edwin Wong
2017-03-27 18:26:28 -07:00
parent bfe6046199
commit 54061a8bb8
3 changed files with 3 additions and 5 deletions

View File

@@ -149,7 +149,6 @@ LOCAL_SHARED_LIBRARIES := \
libcutils \ libcutils \
libdl \ libdl \
liblog \ liblog \
libmedia \
libprotobuf-cpp-lite \ libprotobuf-cpp-lite \
libstagefright_foundation \ libstagefright_foundation \
libutils \ libutils \
@@ -217,7 +216,6 @@ LOCAL_SHARED_LIBRARIES := \
libhidlmemory \ libhidlmemory \
libhwbinder \ libhwbinder \
liblog \ liblog \
libmedia \
libprotobuf-cpp-lite \ libprotobuf-cpp-lite \
libstagefright_foundation \ libstagefright_foundation \
libutils \ libutils \

View File

@@ -36,7 +36,6 @@ LOCAL_SRC_FILES := \
$(CORE_SRC_DIR)/privacy_crypto_openssl.cpp \ $(CORE_SRC_DIR)/privacy_crypto_openssl.cpp \
$(CORE_SRC_DIR)/service_certificate.cpp \ $(CORE_SRC_DIR)/service_certificate.cpp \
$(CORE_SRC_DIR)/usage_table_header.cpp \ $(CORE_SRC_DIR)/usage_table_header.cpp \
$(SRC_DIR)/ami_adapter.cpp \
$(SRC_DIR)/wv_content_decryption_module.cpp \ $(SRC_DIR)/wv_content_decryption_module.cpp \
$(METRICS_SRC_DIR)/distribution.cpp \ $(METRICS_SRC_DIR)/distribution.cpp \
$(METRICS_SRC_DIR)/event_metric.cpp \ $(METRICS_SRC_DIR)/event_metric.cpp \

View File

@@ -2,7 +2,6 @@
#include "wv_content_decryption_module.h" #include "wv_content_decryption_module.h"
#include "ami_adapter.h"
#include "cdm_client_property_set.h" #include "cdm_client_property_set.h"
#include "cdm_engine.h" #include "cdm_engine.h"
#include "initialization_data.h" #include "initialization_data.h"
@@ -23,7 +22,9 @@ namespace wvcdm {
Lock WvContentDecryptionModule::session_sharing_id_generation_lock_; Lock WvContentDecryptionModule::session_sharing_id_generation_lock_;
WvContentDecryptionModule::WvContentDecryptionModule() { WvContentDecryptionModule::WvContentDecryptionModule() {
report_root_ = new AmiAdapter(); // TODO (b/36497276)
// replace call to new AmiAdapter() and remove ami_apdater.*
report_root_ = NULL; // new AmiAdapter();
front_end_ = new metrics::MetricsFrontEnd(report_root_); front_end_ = new metrics::MetricsFrontEnd(report_root_);
metrics::MetricsFrontEnd::OverrideInstance(front_end_); metrics::MetricsFrontEnd::OverrideInstance(front_end_);
} }