Replace OVERRIDE with its C++11 counterparts in src/skia

Replace OVERRIDE with its C++11 counterparts in src/skia

This step is a giant search for OVERRIDE to
replace it with its lowercase versions.

BUG=417463
Committed: https://crrev.com/3c200fad33da10ca7b7bda1bba471d80ac570649
Cr-Commit-Position: refs/heads/master@{#298861}

[RAGHURAM, 2014-10-08 06:34:12]

r.nagaraj@samsung.com changed reviewers:
+ fmalita@chromium.org, robertphillips@google.com

[RAGHURAM, 2014-10-08 06:34:12]

Please review.

Thanks & Regards

[f(malita), 2014-10-08 11:55:08]

Looks good, but while at it let's also replace the SK_OVERRIDEs in
skia/ext/pixel_ref_utils.cc.

https://codereview.chromium.org/641523002/diff/1/skia/ext/vector_platform_dev...
File skia/ext/vector_platform_device_emf_win.h (right):

https://codereview.chromium.org/641523002/diff/1/skia/ext/vector_platform_dev...
skia/ext/vector_platform_device_emf_win.h:53: SkCanvas::DrawBitmapRectFlags
flags) SK_OVERRIDE;
SK_OVERRIDE -> override

[robertphillips, 2014-10-08 12:01:40]

lgtm modulo the change Florin suggested

[RAGHURAM, 2014-10-08 12:05:26]

On 2014/10/08 11:55:08, Florin Malita wrote:
> Looks good, but while at it let's also replace the SK_OVERRIDEs in
> skia/ext/pixel_ref_utils.cc.
> 
>
https://codereview.chromium.org/641523002/diff/1/skia/ext/vector_platform_dev...
> File skia/ext/vector_platform_device_emf_win.h (right):
> 
>
https://codereview.chromium.org/641523002/diff/1/skia/ext/vector_platform_dev...
> skia/ext/vector_platform_device_emf_win.h:53: SkCanvas::DrawBitmapRectFlags
> flags) SK_OVERRIDE;
> SK_OVERRIDE -> override

I have uploaded the patch with the changes suggested. Please take a look.

Thanks & Regards

[robertphillips, 2014-10-08 12:05:51]

lgtm

[f(malita), 2014-10-08 12:06:16]

LGTM, thanks.

[f(malita), 2014-10-08 12:06:19]

The CQ bit was checked by fmalita@chromium.org

[commit-bot: I haz the power, 2014-10-08 12:07:12]

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

[RAGHURAM, 2014-10-08 12:13:58]

On 2014/10/08 12:06:16, Florin Malita wrote:
> LGTM, thanks.

The chromium presubmit is failing. Could you please guide me. 
I ll do the needful.

Thanks & Regards

[commit-bot: I haz the power, 2014-10-08 12:14:12]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-10-08 12:14:13]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[robertphillips, 2014-10-08 12:28:40]

Presumably your company has already filled out a Corporate Contributor License
Agreement but I don't see you in the AUTHORS file
(http://src.chromium.org/svn/trunk/src/AUTHORS).

[f(malita), 2014-10-08 12:29:11]

On 2014/10/08 12:13:58, r.nagaraj wrote:
> On 2014/10/08 12:06:16, Florin Malita wrote:
> > LGTM, thanks.
> 
> The chromium presubmit is failing. Could you please guide me. 
> I ll do the needful.
> 
> Thanks & Regards

OK, I checked and you don't appear on the Chromium CLA signers list. Not lgtm
for now.

You need to

1) go through the corporate CLA signing process:
http://www.chromium.org/developers/contributing-code/external-contributor-che...

2) add yourself to the AUTHORS file (can be done with this patch, see
codereview.chromium.org/617723003/ for an example)

Thanks!

[f(malita), 2014-10-08 12:31:29]

On 2014/10/08 12:29:11, Florin Malita wrote:
> see codereview.chromium.org/617723003/ for an example)

Ugh, wrong URL, try: https://codereview.chromium.org/583633002/

[robertphillips, 2014-10-08 12:34:14]

I just checked and you aren't listed as an Authorized contributor in your
company's Corporate CLA Signer's list.

[RAGHURAM, 2014-10-08 12:36:07]

On 2014/10/08 12:34:14, robertphillips wrote:
> I just checked and you aren't listed as an Authorized contributor in your
> company's Corporate CLA Signer's list.

Yes I have asked the concern people in our company to mail a request for the
same.

Thanks & Regards

[RAGHURAM, 2014-10-09 05:37:49]

On 2014/10/08 12:29:11, Florin Malita wrote:
> On 2014/10/08 12:13:58, r.nagaraj wrote:
> > On 2014/10/08 12:06:16, Florin Malita wrote:
> > > LGTM, thanks.
> > 
> > The chromium presubmit is failing. Could you please guide me. 
> > I ll do the needful.
> > 
> > Thanks & Regards
> 
> OK, I checked and you don't appear on the Chromium CLA signers list. Not lgtm
> for now.
> 
> You need to
> 
> 1) go through the corporate CLA signing process:
>
http://www.chromium.org/developers/contributing-code/external-contributor-che...

I have been added to the CLA list.

> 2) add yourself to the AUTHORS file (can be done with this patch, see
> codereview.chromium.org/617723003/ for an example)
>

I have added myself in the author list and updated patch set.
PTAL.

Thanks & Regards

[f(malita), 2014-10-09 12:18:16]

Thanks. LGTM.

[f(malita), 2014-10-09 12:18:29]

The CQ bit was checked by fmalita@chromium.org

[commit-bot: I haz the power, 2014-10-09 12:20:02]

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

[robertphillips, 2014-10-09 12:20:21]

lgtm

[commit-bot: I haz the power, 2014-10-09 13:22:33]

Committed patchset #3 (id:40001) as 60d943a787c49d4e605fe7df1229087cd71a815f

[commit-bot: I haz the power, 2014-10-09 13:23:15]

Patchset 3 (id:??) landed as
https://crrev.com/3c200fad33da10ca7b7bda1bba471d80ac570649
Cr-Commit-Position: refs/heads/master@{#298861}