Remove enterprise device user report size limit.

Remove enterprise device user report size limit.

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

[Joao da Silva, 2014-07-07 09:43:23]

lgtm, but why don't we remove the limit completely?

https://codereview.chromium.org/355203007/diff/1/chrome/browser/chromeos/poli...
File chrome/browser/chromeos/policy/device_status_collector.cc (right):

https://codereview.chromium.org/355203007/diff/1/chrome/browser/chromeos/poli...
chrome/browser/chromeos/policy/device_status_collector.cc:451: if
(request->user_size() >= kMaxUserCount)
500 is an extreme limit for this... why not just remove this logic? The bug
doesn't have enough context and I may be missing something here.

[Steve Condie, 2014-07-07 16:55:24]

On 2014/07/07 09:43:23, Joao da Silva wrote:
> lgtm, but why don't we remove the limit completely?
> 
>
https://codereview.chromium.org/355203007/diff/1/chrome/browser/chromeos/poli...
> File chrome/browser/chromeos/policy/device_status_collector.cc (right):
> 
>
https://codereview.chromium.org/355203007/diff/1/chrome/browser/chromeos/poli...
> chrome/browser/chromeos/policy/device_status_collector.cc:451: if
> (request->user_size() >= kMaxUserCount)
> 500 is an extreme limit for this... why not just remove this logic? The bug
> doesn't have enough context and I may be missing something here.

Ok, just removed the limit.

[Joao da Silva, 2014-07-07 17:46:41]

lgtm

[Steve Condie, 2014-07-07 20:54:05]

The CQ bit was checked by stepco@chromium.org

[commit-bot: I haz the power, 2014-07-07 20:55:09]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/stepco@chromium.org/355203007/20001

[commit-bot: I haz the power, 2014-07-08 00:03:49]

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

[commit-bot: I haz the power, 2014-07-08 00:35:30]

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

[commit-bot: I haz the power, 2014-07-08 00:35:31]

Try jobs failed on following builders:
  android_aosp on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_aosp/builds/9...)

[Steve Condie, 2014-07-08 18:08:36]

The CQ bit was checked by stepco@chromium.org

[commit-bot: I haz the power, 2014-07-08 18:11:28]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/stepco@chromium.org/355203007/20001

[commit-bot: I haz the power, 2014-07-08 19:20:14]

Change committed as 281789