Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(139)

Issue 2071853002: Revert of bluetooth: mac: Initial BluetoothRemoteGattCharacteristicMac implementation (Closed)

Created:
4 years, 6 months ago by Henrik Grunell
Modified:
4 years, 6 months ago
CC:
blink-reviews, chromium-reviews, haraken, ortuno+watch_chromium.org, scheib+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@servicescan_cleanup
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Revert of bluetooth: mac: Initial BluetoothRemoteGattCharacteristicMac implementation (patchset #14 id:260001 of https://codereview.chromium.org/1950033002/ ) Reason for revert: Breaks Mac compile: https://build.chromium.org/p/chromium/builders/Mac/builds/16785 Original issue's description: > bluetooth: mac: Initial BluetoothRemoteGattCharacteristicMac implementation > > Adding initial remote characteristic implementation on OS X, with only the basic methods (e.g. GetUUID) implemented. > > Characteristic discovery also implemented in BluetoothRemoteGattServiceMac. > > BUG=609064 > > Committed: https://crrev.com/6af3e124c3e3745f280d3b0c5c9491a5e603340f > Cr-Commit-Position: refs/heads/master@{#400127} TBR=ortuno@chromium.org,fbeaufort@chromium.org,msarda@chromium.org,scheib@chromium.org,jlebel@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=609064 Committed: https://crrev.com/4dfb0cd00934f1564b63f4510d1c8a7c6e4f0ac3 Cr-Commit-Position: refs/heads/master@{#400143}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+83 lines, -738 lines) Patch
M device/BUILD.gn View 1 chunk +0 lines, -2 lines 0 comments Download
M device/bluetooth/BUILD.gn View 1 chunk +0 lines, -2 lines 0 comments Download
M device/bluetooth/bluetooth.gyp View 1 chunk +0 lines, -2 lines 0 comments Download
M device/bluetooth/bluetooth_low_energy_device_mac.h View 2 chunks +0 lines, -2 lines 0 comments Download
M device/bluetooth/bluetooth_low_energy_device_mac.mm View 2 chunks +5 lines, -39 lines 0 comments Download
M device/bluetooth/bluetooth_low_energy_peripheral_delegate.mm View 2 chunks +0 lines, -10 lines 0 comments Download
D device/bluetooth/bluetooth_remote_gatt_characteristic_mac.h View 1 chunk +0 lines, -72 lines 0 comments Download
D device/bluetooth/bluetooth_remote_gatt_characteristic_mac.mm View 1 chunk +0 lines, -151 lines 0 comments Download
M device/bluetooth/bluetooth_remote_gatt_characteristic_unittest.cc View 4 chunks +11 lines, -27 lines 0 comments Download
M device/bluetooth/bluetooth_remote_gatt_service_mac.h View 2 chunks +1 line, -26 lines 0 comments Download
M device/bluetooth/bluetooth_remote_gatt_service_mac.mm View 4 chunks +5 lines, -89 lines 0 comments Download
M device/bluetooth/bluetooth_remote_gatt_service_unittest.cc View 6 chunks +15 lines, -30 lines 0 comments Download
M device/bluetooth/test/bluetooth_test_mac.h View 3 chunks +0 lines, -16 lines 0 comments Download
M device/bluetooth/test/bluetooth_test_mac.mm View 5 chunks +2 lines, -63 lines 0 comments Download
D device/bluetooth/test/mock_bluetooth_cbcharacteristic_mac.h View 1 chunk +0 lines, -23 lines 0 comments Download
D device/bluetooth/test/mock_bluetooth_cbcharacteristic_mac.mm View 1 chunk +0 lines, -118 lines 0 comments Download
M device/bluetooth/test/mock_bluetooth_cbperipheral_mac.h View 1 chunk +0 lines, -2 lines 0 comments Download
M device/bluetooth/test/mock_bluetooth_cbperipheral_mac.mm View 4 chunks +3 lines, -22 lines 0 comments Download
M device/bluetooth/test/mock_bluetooth_cbservice_mac.h View 2 chunks +1 line, -9 lines 0 comments Download
M device/bluetooth/test/mock_bluetooth_cbservice_mac.mm View 3 chunks +1 line, -28 lines 0 comments Download
M device/device_tests.gyp View 1 chunk +0 lines, -2 lines 0 comments Download
M third_party/WebKit/LayoutTests/TestExpectations View 1 chunk +25 lines, -3 lines 0 comments Download
M third_party/WebKit/Source/modules/bluetooth/BluetoothRemoteGATTService.cpp View 2 chunks +14 lines, -0 lines 0 comments Download

Messages

Total messages: 8 (3 generated)
Henrik Grunell
Created Revert of bluetooth: mac: Initial BluetoothRemoteGattCharacteristicMac implementation
4 years, 6 months ago (2016-06-16 13:47:16 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2071853002/1
4 years, 6 months ago (2016-06-16 13:47:27 UTC) #3
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 6 months ago (2016-06-16 13:48:26 UTC) #5
commit-bot: I haz the power
CQ bit was unchecked
4 years, 6 months ago (2016-06-16 13:48:28 UTC) #6
commit-bot: I haz the power
4 years, 6 months ago (2016-06-16 13:49:49 UTC) #8
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/4dfb0cd00934f1564b63f4510d1c8a7c6e4f0ac3
Cr-Commit-Position: refs/heads/master@{#400143}

Powered by Google App Engine
This is Rietveld 408576698