Fix for uninitialized access.

Fix for uninitialized access.

BUG=394390
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=283760

[mmenke, 2014-07-16 17:37:02]

LGTM.  I did the exact same thing in another CL a week or two ago.  :(

https://codereview.chromium.org/391393002/diff/1/content/browser/loader/resou...
File content/browser/loader/resource_scheduler.cc (right):

https://codereview.chromium.org/391393002/diff/1/content/browser/loader/resou...
content/browser/loader/resource_scheduler.cc:637:
ResourceScheduler::ResourceScheduler(): active_clients_loading_(0),
nit:  Need another space before the colon.

https://codereview.chromium.org/391393002/diff/1/content/browser/loader/resou...
File content/browser/loader/resource_scheduler_unittest.cc (right):

https://codereview.chromium.org/391393002/diff/1/content/browser/loader/resou...
content/browser/loader/resource_scheduler_unittest.cc:546: // TODO(aiolos):
remove when throttling and coalescing have both landed
nit:  Capitalize + add period (For all of these)

[aiolos (Not reviewing), 2014-07-16 17:40:10]

The CQ bit was checked by aiolos@chromium.org

[commit-bot: I haz the power, 2014-07-16 17:42:51]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/aiolos@chromium.org/391393002/1

[earthdok, 2014-07-16 18:22:08]

On 2014/07/16 17:42:51, I haz the power (commit-bot) wrote:
> CQ is trying da patch. Follow status at
>  https://chromium-status.appspot.com/cq/aiolos@chromium.org/391393002/1

Looks like you forgot the terminating "_".

[earthdok, 2014-07-16 18:34:44]

On 2014/07/16 18:22:08, earthdok wrote:
> On 2014/07/16 17:42:51, I haz the power (commit-bot) wrote:
> > CQ is trying da patch. Follow status at
> >  https://chromium-status.appspot.com/cq/aiolos@chromium.org/391393002/1
> 
> Looks like you forgot the terminating "_".

Also, initialization order doesn't match declaration order.

[mmenke, 2014-07-16 18:40:19]

On 2014/07/16 18:34:44, earthdok wrote:
> On 2014/07/16 18:22:08, earthdok wrote:
> > On 2014/07/16 17:42:51, I haz the power (commit-bot) wrote:
> > > CQ is trying da patch. Follow status at
> > >  https://chromium-status.appspot.com/cq/aiolos@chromium.org/391393002/1
> > 
> > Looks like you forgot the terminating "_".
> 
> Also, initialization order doesn't match declaration order.

Well...I should have been more careful in my rather cursory review.

[aiolos (Not reviewing), 2014-07-16 19:18:00]

On 2014/07/16 18:40:19, mmenke wrote:
> On 2014/07/16 18:34:44, earthdok wrote:
> > On 2014/07/16 18:22:08, earthdok wrote:
> > > On 2014/07/16 17:42:51, I haz the power (commit-bot) wrote:
> > > > CQ is trying da patch. Follow status at
> > > >  https://chromium-status.appspot.com/cq/aiolos@chromium.org/391393002/1
> > > 
> > > Looks like you forgot the terminating "_".
> > 
> > Also, initialization order doesn't match declaration order.
> 
> Well...I should have been more careful in my rather cursory review.

Sorry, I thought I'd uploaded after I fixed that in the CL. Fixed.

[aiolos (Not reviewing), 2014-07-16 19:18:11]

The CQ bit was checked by aiolos@chromium.org

[commit-bot: I haz the power, 2014-07-16 19:19:44]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/aiolos@chromium.org/391393002/2

[commit-bot: I haz the power, 2014-07-16 22:06:28]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg_triggered_tests on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg_triggered...)
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)

[commit-bot: I haz the power, 2014-07-17 02:39:06]

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

[commit-bot: I haz the power, 2014-07-17 02:39:07]

Try jobs failed on following builders:
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)

[earthdok, 2014-07-17 10:14:38]

The CQ bit was checked by earthdok@chromium.org

[commit-bot: I haz the power, 2014-07-17 10:17:05]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/aiolos@chromium.org/391393002/2

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

Change committed as 283760