commit 1d2e9cfbe3f2db16ecafd2adafb4c60d7235a2ce
author Chong Zhang <chz@google.com> Wed Oct 05 21:08:36 2016 -0700
committer Chong Zhang <chz@google.com> Thu Oct 06 21:19:37 2016 +0000
tree 9b1bb4de458850e661cd40ba1f4edc6fb3f42bf5
parent c380f611f779dd7dc29ccf80609893d67e78167a

IOMX: remove nodeBinder from allocateNode

This is no longer needed, the IOMXNode is already a binder,
we can just link to that.

bug: 31399200

Change-Id: Ic5dd5330dab448ae126aaec694f71f22857b3169

media/libmedia/IOMX.cpp

diff --git a/media/libmedia/IOMX.cpp b/media/libmedia/IOMX.cpp
index e25e56c..5063504 100644
--- a/media/libmedia/IOMX.cpp
+++ b/media/libmedia/IOMX.cpp
@@ -99,7 +99,7 @@
 
     virtual status_t allocateNode(
             const char *name, const sp<IOMXObserver> &observer,
-            sp<IBinder> *nodeBinder, sp<IOMXNode> *omxNode) {
+            sp<IOMXNode> *omxNode) {
         Parcel data, reply;
         data.writeInterfaceToken(IOMX::getInterfaceDescriptor());
         data.writeCString(name);
@@ -109,9 +109,6 @@
         status_t err = reply.readInt32();
         if (err == OK) {
             *omxNode = IOMXNode::asInterface(reply.readStrongBinder());
-            if (nodeBinder != NULL) {
-                *nodeBinder = remote();
-            }
         } else {
             omxNode->clear();
         }
@@ -671,8 +668,7 @@
 
             sp<IOMXNode> omxNode;
 
-            status_t err = allocateNode(
-                    name, observer, NULL /* nodeBinder */, &omxNode);
+            status_t err = allocateNode(name, observer, &omxNode);
 
             reply->writeInt32(err);
             if (err == OK) {