New issue
Advanced search Search tips

Issue 653056 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 3
Type: Feature

Blocking:
issue 626465



Sign in to add a comment

bluetooth: macOS: cleanup when Chrome is not compiled for 10.8

Project Member Reported by jlebel@chromium.org, Oct 5 2016

Issue description

Some methods are used defined in 10.9. To avoid compiler warnings, pragma instructions are being used:
#pragma clang diagnostic ignored "-Wpartial-availability"

-[CBCentralManager retrieveConnectedPeripheralsWithServices:]
used in device/bluetooth/bluetooth_adapter_mac.mm, BluetoothAdapter::RetrievedConnectedPeripherals()
defined in 10.9

-[<CBPeripheralDelegate> peripheral:didModifyServices:] 
used in device/bluetooth/test/mock_bluetooth_cbperipheral_mac.mm, [MockCBPeripheral didModifyServices:].
defined in 10.9
 
Could you mark this as blocked by the 10.8 issue if there is one?

Comment 2 by scheib@chromium.org, Oct 15 2016

Blocking: 626465
Labels: -Type-Bug Type-Feature
Project Member

Comment 3 by bugdroid1@chromium.org, Nov 7 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d9c107f9476d72220f6ab128578183ba9f1b18f6

commit d9c107f9476d72220f6ab128578183ba9f1b18f6
Author: jlebel <jlebel@chromium.org>
Date: Mon Nov 07 22:43:43 2016

bluetooth: mac: Removing #pragma clang diagnostic ignored "-Wpartial-availability"

Now the macOS version target is 10.9, therefore the definition of
-[<CBPeripheralDelegate> peripheral:didModifyServices:] is known by
the bot compilers.
Note that this method is optional for <CBPeripheralDelegate>.

BUG= 653056 

Review-Url: https://codereview.chromium.org/2485633002
Cr-Commit-Position: refs/heads/master@{#430405}

[modify] https://crrev.com/d9c107f9476d72220f6ab128578183ba9f1b18f6/device/bluetooth/test/mock_bluetooth_cbperipheral_mac.mm

Status: Fixed (was: Untriaged)
Owner: jlebel@chromium.org

Sign in to add a comment