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

Commit 9e0bb045 authored by William Escande's avatar William Escande Committed by Android (Google) Code Review
Browse files

Merge "Fix missing CoD information when getting connection_request" into tm-mainline-prod

parents aa546de6 fe28f07c

system/gd/hci/acl_manager/classic_impl.h

+6 −0
Original line number Diff line number Diff line
@@ -252,6 +252,12 @@ struct classic_impl : public security::ISecurityManagerListener { 
        return;



      case ConnectionRequestLinkType::ACL:

        // Need to upstream Cod information when getting connection_request

        client_handler_->CallOn(

            client_callbacks_,

            &ConnectionCallbacks::OnConnectRequest,

            address,

            request.GetClassOfDevice());

        break;



      case ConnectionRequestLinkType::ESCO:

system/gd/hci/acl_manager/connection_callbacks.h

+2 −0
Original line number Diff line number Diff line
@@ -32,6 +32,8 @@ class ConnectionCallbacks { 
  virtual ~ConnectionCallbacks() = default;

  // Invoked when controller sends Connection Complete event with Success error code

  virtual void OnConnectSuccess(std::unique_ptr<ClassicAclConnection>) = 0;

  // Invoked when controller sends Connection Request

  virtual void OnConnectRequest(Address, ClassOfDevice) = 0;

  // Invoked when controller sends Connection Complete event with non-Success error code

  virtual void OnConnectFail(Address, ErrorCode reason) = 0;

system/gd/hci/acl_manager_test.cc

+1 −1
Original line number Diff line number Diff line
@@ -212,7 +212,7 @@ class AclManagerNoCallbacksTest : public ::testing::Test { 
    void Clear() {

      connections_.clear();

    }



    MOCK_METHOD(void, OnConnectRequest, (Address, ClassOfDevice), (override));

    MOCK_METHOD(void, OnConnectFail, (Address, ErrorCode reason), (override));



    MOCK_METHOD(void, HACK_OnEscoConnectRequest, (Address, ClassOfDevice), (override));

system/gd/hci/acl_manager_unittest.cc

+1 −0
Original line number Diff line number Diff line
@@ -350,6 +350,7 @@ class MockConnectionCallback : public ConnectionCallbacks { 
      connection_promise_.set_value(connections_.back());

    }

  }

  MOCK_METHOD(void, OnConnectRequest, (Address, ClassOfDevice), (override));

  MOCK_METHOD(void, OnConnectFail, (Address, ErrorCode reason), (override));



  MOCK_METHOD(void, HACK_OnEscoConnectRequest, (Address, ClassOfDevice), (override));

system/gd/hci/facade/acl_manager_facade.cc

+4 −0
Original line number Diff line number Diff line
@@ -374,6 +374,10 @@ class AclManagerFacadeService : public AclManagerFacade::Service, public Connect 
    current_connection_request_++;

  }



  void OnConnectRequest(Address address, ClassOfDevice cod) override {

    LOG_ERROR("Remote connect request unimplemented");

  }



  void OnConnectFail(Address address, ErrorCode reason) override {

    LOG_INFO("addr=%s, reason=%s", address.ToString().c_str(), ErrorCodeText(reason).c_str());

    std::unique_ptr<BasePacketBuilder> builder =