Added memory limit for chrome.activityLogPrivate returned results

Added memory limit for chrome.activityLogPrivate returned results

The Activity Log will only send a max of 300 rows per lookup. This should be completely satisfactory for the Watchdog app -- we can't display more than 300 rows of data in any meaninful way anyway. This is also below the IPC memory boundary.

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

[felt, 2013-09-04 02:11:40]

Hi Matt,

Can you please review this?

By the way, thanks for telling me about the IPC limits. I tried and couldn't
make 300 rows go over 1mb.

Adrienne

[Matt Perry, 2013-09-04 19:34:25]

LGTM

I was just guessing about the 1MB limit - I don't remember exactly what it was.

[felt, 2013-09-04 19:42:18]

On 2013/09/04 19:34:25, Matt Perry wrote:
> LGTM
> 
> I was just guessing about the 1MB limit - I don't remember exactly what it
was.

Well, either way 300MB seems to be fine regardless -- I tried stress-testing it,
and no problems.

[commit-bot: I haz the power, 2013-09-04 23:19:57]

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

[commit-bot: I haz the power, 2013-09-05 00:31:17]

Retried try job too often on ios_dbg_simulator for step(s) ui_unittests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=ios_dbg_si...

[commit-bot: I haz the power, 2013-09-05 00:44:06]

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

[commit-bot: I haz the power, 2013-09-05 01:41:19]

Retried try job too often on ios_dbg_simulator for step(s) ui_unittests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=ios_dbg_si...

[commit-bot: I haz the power, 2013-09-05 12:36:21]

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

[commit-bot: I haz the power, 2013-09-05 13:26:58]

Change committed as 221406