blimp: Increase max payload size limit from 1MB to 5MB.

blimp: Increase max payload size limit.

BUG=597026
Committed: https://crrev.com/fe446e5fe8e9b4989f6341b8fe7ec7614fb18a16
Cr-Commit-Position: refs/heads/master@{#382906}

[Khushal, 2016-03-23 04:35:32]

khushalsagar@chromium.org changed reviewers:
+ kmarshall@chromium.org

[Kevin M, 2016-03-23 17:01:08]

lgtm

[Kevin M, 2016-03-23 17:01:52]

Can you add "from 1MB to 3MB" to the end of the CL subject line?

[Khushal, 2016-03-23 17:56:42]

Description was changed from

==========
blimp: Increase max payload size limit.

BUG=597026
==========

to

==========
blimp: Increase max payload size limit.

BUG=597026
==========

[Khushal, 2016-03-23 17:57:54]

On 2016/03/23 17:01:52, Kevin M wrote:
> Can you add "from 1MB to 3MB" to the end of the CL subject line?

Done. Removed the DCHECK for output buffer size as well. Could you take another
look.

[Kevin M, 2016-03-23 18:37:35]

lgtm

https://codereview.chromium.org/1823383002/diff/20001/blimp/net/blimp_message...
File blimp/net/blimp_message_output_buffer.cc (right):

https://codereview.chromium.org/1823383002/diff/20001/blimp/net/blimp_message...
blimp/net/blimp_message_output_buffer.cc:69: DLOG(WARNING) << "Buffer size Limit
Exceeded. Maximum Bytes Limit: "
Nit: this isn't really a limit anymore, it's more like an advisory threshold.

Maybe something like:
"Warning: output buffer size exceeds X bytes (current: Y)".

[Khushal, 2016-03-23 19:06:49]

https://codereview.chromium.org/1823383002/diff/20001/blimp/net/blimp_message...
File blimp/net/blimp_message_output_buffer.cc (right):

https://codereview.chromium.org/1823383002/diff/20001/blimp/net/blimp_message...
blimp/net/blimp_message_output_buffer.cc:69: DLOG(WARNING) << "Buffer size Limit
Exceeded. Maximum Bytes Limit: "
On 2016/03/23 18:37:34, Kevin M wrote:
> Nit: this isn't really a limit anymore, it's more like an advisory threshold.
> 
> Maybe something like:
> "Warning: output buffer size exceeds X bytes (current: Y)".

Done.

[Khushal, 2016-03-23 19:06:54]

The CQ bit was checked by khushalsagar@chromium.org

[Khushal, 2016-03-23 19:06:55]

The patchset sent to the CQ was uploaded after l-g-t-m from
kmarshall@chromium.org
Link to the patchset: https://codereview.chromium.org/1823383002/#ps40001
(title: "Update comment.")

[commit-bot: I haz the power, 2016-03-23 19:07:20]

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

[commit-bot: I haz the power, 2016-03-23 19:55:27]

Description was changed from

==========
blimp: Increase max payload size limit.

BUG=597026
==========

to

==========
blimp: Increase max payload size limit.

BUG=597026
==========

[commit-bot: I haz the power, 2016-03-23 19:55:28]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-23 19:58:41]

Description was changed from

==========
blimp: Increase max payload size limit.

BUG=597026
==========

to

==========
blimp: Increase max payload size limit.

BUG=597026

Committed: https://crrev.com/fe446e5fe8e9b4989f6341b8fe7ec7614fb18a16
Cr-Commit-Position: refs/heads/master@{#382906}
==========

[commit-bot: I haz the power, 2016-03-23 19:58:42]

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