Add a few missing overrides found by a new clang warning.

ui: Add a few missing overrides found by a new clang warning.

Namely, -Winconsistent-missing-override. While here, also
remove "virtual" from overridden methods, as per new style.
No behavior change.

BUG=428099
Committed: https://crrev.com/4ec660f9904a722cfac0226b684cd6f4787ece07
Cr-Commit-Position: refs/heads/master@{#302035}

[Nico, 2014-10-30 00:44:04]

thakis@chromium.org changed reviewers:
+ sky@chromium.org

[Nico, 2014-10-30 00:44:05]

I present: The most boring cl you'll see today!

[sky, 2014-10-30 00:59:10]

https://codereview.chromium.org/692713002/diff/1/ui/base/dragdrop/os_exchange...
File ui/base/dragdrop/os_exchange_data_provider_win.h (right):

https://codereview.chromium.org/692713002/diff/1/ui/base/dragdrop/os_exchange...
ui/base/dragdrop/os_exchange_data_provider_win.h:150: //
OSExchangeData::Provider methods.
I thought we were getting rid of virtual too?

[Nico, 2014-10-30 01:00:01]

https://codereview.chromium.org/692713002/diff/1/ui/base/dragdrop/os_exchange...
File ui/base/dragdrop/os_exchange_data_provider_win.h (right):

https://codereview.chromium.org/692713002/diff/1/ui/base/dragdrop/os_exchange...
ui/base/dragdrop/os_exchange_data_provider_win.h:150: //
OSExchangeData::Provider methods.
On 2014/10/30 00:59:10, sky wrote:
> I thought we were getting rid of virtual too?

Yes, dcheng is doing this in a separate pass, using automated tooling. I think
today the plugin still insists on virtual.

[dcheng, 2014-10-30 01:04:25]

On 2014/10/30 01:00:01, Nico wrote:
>
https://codereview.chromium.org/692713002/diff/1/ui/base/dragdrop/os_exchange...
> File ui/base/dragdrop/os_exchange_data_provider_win.h (right):
> 
>
https://codereview.chromium.org/692713002/diff/1/ui/base/dragdrop/os_exchange...
> ui/base/dragdrop/os_exchange_data_provider_win.h:150: //
> OSExchangeData::Provider methods.
> On 2014/10/30 00:59:10, sky wrote:
> > I thought we were getting rid of virtual too?
> 
> Yes, dcheng is doing this in a separate pass, using automated tooling. I think
> today the plugin still insists on virtual.

Actually, it shouldn't require virtual anymore.

[Nico, 2014-10-30 02:56:05]

Removed the virtuals

[sky, 2014-10-30 03:16:57]

LGTM

[Nico, 2014-10-30 04:18:37]

The CQ bit was checked by thakis@chromium.org

[commit-bot: I haz the power, 2014-10-30 04:19:41]

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

[commit-bot: I haz the power, 2014-10-30 05:01:33]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-10-30 05:02:18]

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