Add missing 'override' to BluetoothRemoteGattDescriptorWin destructor.

Add missing 'override' to BluetoothRemoteGattDescriptorWin destructor.

BUG=82385
R=scheib@chromium.org, thakis@chromium.org

Committed: https://chromium.googlesource.com/chromium/src/+/6a30b38f3e54e25db749a72d03d7a2e0acdd6119

[dcheng, 2016-03-01 22:59:55]

dcheng@chromium.org changed reviewers:
+ hans@chromium.org, keybuk@chromium.org, thakis@chromium.org

[Nico, 2016-03-01 23:06:52]

lgtm if the "patch failure" is not interesting

(i kind of feel like i wrote this patch before…)

[dcheng, 2016-03-01 23:07:25]

(not sure why this patch isn't applying on the bots, the original patch
https://codereview.chromium.org/1728163006 doesn't seem to have been reverted)

[Nico, 2016-03-01 23:10:04]

whoa we only have 3 clang slaves please don't schedule 5 try jobs

(we'll have more slaves eventually, but that day is not today)

[Nico, 2016-03-01 23:10:10]

The CQ bit was checked by thakis@chromium.org

[Nico, 2016-03-01 23:10:10]

The patchset sent to the CQ was uploaded after l-g-t-m from thakis@chromium.org
Link to the patchset: https://codereview.chromium.org/1745273005/#ps20001
(title: "rebase?")

[commit-bot: I haz the power, 2016-03-01 23:11:07]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1745273005/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1745273005/20001

[dcheng, 2016-03-01 23:12:23]

The CQ bit was unchecked by dcheng@chromium.org

[dcheng, 2016-03-01 23:13:05]

On 2016/03/01 at 23:10:04, thakis wrote:
> whoa we only have 3 clang slaves please don't schedule 5 try jobs
> 
> (we'll have more slaves eventually, but that day is not today)

I'm not sure what happened, I checked one but it checked a bunch of them? Either
way if the patch doesn't apply, I don't feel comfortable blindly landing this (I
found this on a local build where I'm trying to fix something else)

[scheib, 2016-03-01 23:16:02]

scheib@chromium.org changed reviewers:
+ scheib@chromium.org

[scheib, 2016-03-01 23:16:02]

LGTM

[dcheng, 2016-03-01 23:16:05]

On 2016/03/01 at 23:13:05, dcheng wrote:
> On 2016/03/01 at 23:10:04, thakis wrote:
> > whoa we only have 3 clang slaves please don't schedule 5 try jobs
> > 
> > (we'll have more slaves eventually, but that day is not today)
> 
> I'm not sure what happened, I checked one but it checked a bunch of them?
Either way if the patch doesn't apply, I don't feel comfortable blindly landing
this (I found this on a local build where I'm trying to fix something else)

(Also, I don't think win_compile_* are clang bots too, are they?)

[Nico, 2016-03-01 23:27:47]

The CQ bit was checked by thakis@chromium.org

[Nico, 2016-03-01 23:28:17]

https://build.chromium.org/p/chromium.fyi/console?category=win%20clang is
ablaze, can't get worse :-P hitting cq again.

[commit-bot: I haz the power, 2016-03-01 23:28:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1745273005/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1745273005/20001

[commit-bot: I haz the power, 2016-03-02 00:40:34]

Description was changed from

==========
Add missing 'override' to BluetoothRemoteGattDescriptorWin destructor.

BUG=82385
==========

to

==========
Add missing 'override' to BluetoothRemoteGattDescriptorWin destructor.

BUG=82385
R=scheib@chromium.org, thakis@chromium.org

Committed: https://crrev.com/6a30b38f3e54e25db749a72d03d7a2e0acdd6119
Cr-Commit-Position: refs/heads/master@{#378627}
==========

[commit-bot: I haz the power, 2016-03-02 00:40:35]

Patchset 2 (id:??) landed as
https://crrev.com/6a30b38f3e54e25db749a72d03d7a2e0acdd6119
Cr-Commit-Position: refs/heads/master@{#378627}

[dcheng, 2016-03-02 00:41:14]

Description was changed from

==========
Add missing 'override' to BluetoothRemoteGattDescriptorWin destructor.

BUG=82385
R=scheib@chromium.org, thakis@chromium.org

Committed: https://crrev.com/6a30b38f3e54e25db749a72d03d7a2e0acdd6119
Cr-Commit-Position: refs/heads/master@{#378627}
==========

to

==========
Add missing 'override' to BluetoothRemoteGattDescriptorWin destructor.

BUG=82385
R=scheib@chromium.org, thakis@chromium.org

Committed:
https://chromium.googlesource.com/chromium/src/+/6a30b38f3e54e25db749a72d03d7...
==========

[dcheng, 2016-03-02 00:41:16]

Committed patchset #2 (id:20001) manually as
6a30b38f3e54e25db749a72d03d7a2e0acdd6119 (presubmit successful).