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:
@@ -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 \
|
||||||
|
|||||||
@@ -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 \
|
||||||
|
|||||||
@@ -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_);
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user