Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 84fa44d0 authored by Steven Moreland's avatar Steven Moreland Committed by Android (Google) Code Review
Browse files

Merge "mediatranscoding: avoid double-ownership in test" into sc-dev-plus-aosp

parents 0ccea651 58c938ee

services/mediatranscoding/tests/MediaTranscodingServiceTestHelper.h

+3 −5
Original line number Diff line number Diff line
@@ -259,9 +259,7 @@ std::ostream& operator<<(std::ostream& str, const EventTracker::Event& v) { 
static constexpr bool success = true;

static constexpr bool fail = false;



struct TestClientCallback : public BnTranscodingClientCallback,

                            public EventTracker,

                            public std::enable_shared_from_this<TestClientCallback> {

struct TestClientCallback : public BnTranscodingClientCallback, public EventTracker {

    TestClientCallback(const char* packageName, int32_t id)

          : mClientId(id), mClientPid(PID(id)), mClientUid(UID(id)), mPackageName(packageName) {

        ALOGI("TestClientCallback %d created: pid %d, uid %d", id, PID(id), UID(id));
@@ -348,8 +346,8 @@ struct TestClientCallback : public BnTranscodingClientCallback, 
        ALOGD("registering %s with uid %d", packageName, mClientUid);



        std::shared_ptr<ITranscodingClient> client;

        Status status =

                service->registerClient(shared_from_this(), kClientName, packageName, &client);

        Status status = service->registerClient(ref<TestClientCallback>(), kClientName, packageName,

                                                &client);



        mClient = status.isOk() ? client : nullptr;

        return status;