Move override/final to the approved C++11 features section.

Move override/final to the approved C++11 features section.

Discussion thread:
https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/VTNZzizN0zo

NOTRY=true

Committed: https://crrev.com/5f98d33d5566474db7b31e0960fb09cef0e7c990
Cr-Commit-Position: refs/heads/master@{#296541}

[dcheng, 2014-09-24 21:43:42]

dcheng@chromium.org changed reviewers:
+ ajwong@chromium.org, jamesr@chromium.org, thakis@chromium.org

[dcheng, 2014-09-24 21:43:42]

Here's an initial draft.

[Nico, 2014-09-24 21:46:14]

lgtm with comments, thanks!

https://codereview.chromium.org/596723003/diff/1/styleguide/c++/c++11.html
File styleguide/c++/c++11.html (right):

https://codereview.chromium.org/596723003/diff/1/styleguide/c++/c++11.html#ne...
styleguide/c++/c++11.html:73: <td>All uses of the old <code>FINAL</code> macro
will be replaced throughout the codebase. Uses should conform to the <a
href="http://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Inheritance">C++
style guide</a>.
Add "Recommended for new code." at the beginning, to make that clear.

Add a link to the discussion thread somewhere, like in the nullptr row.

[dcheng, 2014-09-24 21:50:50]

https://codereview.chromium.org/596723003/diff/1/styleguide/c++/c++11.html
File styleguide/c++/c++11.html (right):

https://codereview.chromium.org/596723003/diff/1/styleguide/c++/c++11.html#ne...
styleguide/c++/c++11.html:73: <td>All uses of the old <code>FINAL</code> macro
will be replaced throughout the codebase. Uses should conform to the <a
href="http://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Inheritance">C++
style guide</a>.
On 2014/09/24 21:46:14, Nico (hiding) wrote:
> Add "Recommended for new code." at the beginning, to make that clear.
> 

I've replaced this text to indicate that we prefer this in new code, but I've
removed the style guide reference, since our chrome plugin doesn't actually let
us follow the style guide atm.

> Add a link to the discussion thread somewhere, like in the nullptr row.

Done.

[Nico, 2014-09-24 21:52:28]

lgtm; maybe file a bug for the conversion effort.

(I liked the "existing OVERRIDES will be replaced" sentence you had previously,
but it's fine without it too)

[jamesr, 2014-09-24 22:04:54]

lgtm2

[dcheng, 2014-09-24 22:07:18]

On 2014/09/24 at 22:04:54, jamesr wrote:
> lgtm2

Added the conversion text back and filed https://crbug.com/417463

[dcheng, 2014-09-24 22:07:22]

The CQ bit was checked by dcheng@chromium.org

[commit-bot: I haz the power, 2014-09-24 22:07:44]

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

[Nico, 2014-09-24 22:09:18]

The CQ bit was unchecked by thakis@chromium.org

[Nico, 2014-09-24 22:09:27]

The CQ bit was checked by thakis@chromium.org

[commit-bot: I haz the power, 2014-09-24 22:11:16]

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

[commit-bot: I haz the power, 2014-09-24 22:12:18]

Committed patchset #4 (id:60001) as 561ce2b08a061593fac8be44439ff3a1e1e82444

[commit-bot: I haz the power, 2014-09-24 22:12:51]

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