Changed for loop to c++11 style in chrome/browser/prerender/prerender_local_predictor.cc:573.

Changed for loop to c++11 style in chrome/browser/prerender/prerender_local_predictor.cc:573.

Also added myself to AUTHORS

Committed: https://crrev.com/61f0538ddac87c3c9ca0a233befc3193387efc2c
Cr-Commit-Position: refs/heads/master@{#324021}

[olli.syrjala, 2015-03-27 14:39:26]

could you review?

[olli.syrjala, 2015-03-27 14:39:50]

olli.syrjala@intel.com changed reviewers:
+ cbentzel@chromium.org

[cbentzel, 2015-03-30 11:43:40]

Before looking at the change, I need to make sure that you are listed as a
contributor.

Intel has signed a corporate CLA, but you are not explicitly listed as a
contributor. See
https://www.chromium.org/developers/contributing-code/external-contributor-ch...
for the rules around this.

Once you get added to the company roster, I will be happy to take a look at this
change.

[olli.syrjala, 2015-04-02 06:30:16]

Hi, now I should be there.

[cbentzel, 2015-04-02 12:54:20]

cbentzel@chromium.org changed reviewers:
+ mmenke@chromium.org

[cbentzel, 2015-04-02 12:54:20]

I may be removed from prerender/OWNERS soon, but this looks fine modulo the tiny
correction below.

Adding a future OWNER.

https://codereview.chromium.org/1033283002/diff/1/chrome/browser/prerender/pr...
File chrome/browser/prerender/prerender_local_predictor.cc (right):

https://codereview.chromium.org/1033283002/diff/1/chrome/browser/prerender/pr...
chrome/browser/prerender/prerender_local_predictor.cc:573: for
(PrerenderProperties* p : issued_prerenders_){
Nit: space between ) and {

[mmenke, 2015-04-02 13:24:27]

LGTM as well.

[olli.syrjala, 2015-04-07 06:24:00]

The CQ bit was checked by olli.syrjala@intel.com

[commit-bot: I haz the power, 2015-04-07 06:24:09]

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

[commit-bot: I haz the power, 2015-04-07 06:27:59]

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

[commit-bot: I haz the power, 2015-04-07 06:28:00]

Try jobs failed on following builders:
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[olli.syrjala, 2015-04-07 07:02:04]

The CQ bit was checked by olli.syrjala@intel.com

[olli.syrjala, 2015-04-07 07:02:05]

The patchset sent to the CQ was uploaded after l-g-t-m from mmenke@chromium.org
Link to the patchset: https://codereview.chromium.org/1033283002/#ps40001
(title: "rebase to ToT")

[commit-bot: I haz the power, 2015-04-07 07:02:14]

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

[commit-bot: I haz the power, 2015-04-07 07:59:47]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-04-07 08:00:37]

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