Only log the job controller count when it's hitting boundaries for the first time

Only log the job controller count when it's hitting boundaries for the first time so that when 
normal job controllers are getting created, and deleted, which introduces a zigzag pattern in job 
controller count, we will not log the same user multiple times.

BUG=704988
Review-Url: https://codereview.chromium.org/2815153002
Cr-Commit-Position: refs/heads/master@{#464193}
Committed: https://chromium.googlesource.com/chromium/src/+/a95a63804d48bfb9acfcaabc124504419dcd3668

[Zhongyi Shi, 2017-04-12 20:48:52]

Description was changed from

==========
Only log the job controller count when it's hitting boundaries first time
so that when normal job controllers are getting created, and deleted, which
introduces a zigzag pattern of job controller count, we will not log the
same user multiple times.

BUG=704988
==========

to

==========
Only log the job controller count when it's hitting boundaries for the first
time so that when normal job controllers are getting created, 
and deleted, which introduces a zigzag pattern of job controller count,
we will not log the same user multiple times.

BUG=704988
==========

[Zhongyi Shi, 2017-04-12 20:49:14]

Description was changed from

==========
Only log the job controller count when it's hitting boundaries for the first
time so that when normal job controllers are getting created, 
and deleted, which introduces a zigzag pattern of job controller count,
we will not log the same user multiple times.

BUG=704988
==========

to

==========
Only log the job controller count when it's hitting boundaries for the first
time so that when 
normal job controllers are getting created, and deleted, which introduces a
zigzag pattern of job 
controller count, we will not log the same user multiple times.

BUG=704988
==========

[Zhongyi Shi, 2017-04-12 20:49:37]

Description was changed from

==========
Only log the job controller count when it's hitting boundaries for the first
time so that when 
normal job controllers are getting created, and deleted, which introduces a
zigzag pattern of job 
controller count, we will not log the same user multiple times.

BUG=704988
==========

to

==========
Only log the job controller count when it's hitting boundaries for the first
time so that when 
normal job controllers are getting created, and deleted, which introduces a
zigzag pattern in job 
controller count, we will not log the same user multiple times.

BUG=704988
==========

[Zhongyi Shi, 2017-04-12 20:50:19]

zhongyi@chromium.org changed reviewers:
+ rch@chromium.org

[Zhongyi Shi, 2017-04-12 20:50:19]

Thanks for the great suggestion! PTAL!

[Ryan Hamilton, 2017-04-12 21:05:53]

lgtm

https://codereview.chromium.org/2815153002/diff/1/net/http/http_stream_factor...
File net/http/http_stream_factory_impl.cc (right):

https://codereview.chromium.org/2815153002/diff/1/net/http/http_stream_factor...
net/http/http_stream_factory_impl.cc:366: return;
nit: now that the predicate is more than one line, we need {}s here.

[Zhongyi Shi, 2017-04-12 21:18:04]

Landing now, thanks!

https://codereview.chromium.org/2815153002/diff/1/net/http/http_stream_factor...
File net/http/http_stream_factory_impl.cc (right):

https://codereview.chromium.org/2815153002/diff/1/net/http/http_stream_factor...
net/http/http_stream_factory_impl.cc:366: return;
On 2017/04/12 21:05:53, Ryan Hamilton wrote:
> nit: now that the predicate is more than one line, we need {}s here.

Done.

[Zhongyi Shi, 2017-04-12 21:18:07]

The CQ bit was checked by zhongyi@chromium.org

[Zhongyi Shi, 2017-04-12 21:18:07]

The patchset sent to the CQ was uploaded after l-g-t-m from rch@chromium.org
Link to the patchset: https://codereview.chromium.org/2815153002/#ps20001
(title: "address nits")

[commit-bot: I haz the power, 2017-04-12 21:18:57]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[xunjieli, 2017-04-12 21:22:49]

On 2017/04/12 21:18:57, commit-bot: I haz the power wrote:
> CQ is trying da patch. Follow status at
>  
>
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

Great catch!

[commit-bot: I haz the power, 2017-04-12 23:05:39]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1492031887353270,
"parent_rev": "a44349c1a2af8659b1d326778e7209b1a94876aa", "commit_rev":
"a95a63804d48bfb9acfcaabc124504419dcd3668"}

[commit-bot: I haz the power, 2017-04-12 23:06:30]

Description was changed from

==========
Only log the job controller count when it's hitting boundaries for the first
time so that when 
normal job controllers are getting created, and deleted, which introduces a
zigzag pattern in job 
controller count, we will not log the same user multiple times.

BUG=704988
==========

to

==========
Only log the job controller count when it's hitting boundaries for the first
time so that when 
normal job controllers are getting created, and deleted, which introduces a
zigzag pattern in job 
controller count, we will not log the same user multiple times.

BUG=704988

Review-Url: https://codereview.chromium.org/2815153002
Cr-Commit-Position: refs/heads/master@{#464193}
Committed:
https://chromium.googlesource.com/chromium/src/+/a95a63804d48bfb9acfcaabc1245...
==========

[commit-bot: I haz the power, 2017-04-12 23:06:35]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/a95a63804d48bfb9acfcaabc1245...