Change kMaxDescriptorsPerMessage back to 7

Change kMaxDescriptorsPerMessage back to 7

This is a partial revert of crrev.com/320831 which increased
Chrome's memory usage by 106KB per IPC::Channel.

BUG=484154
TEST=git cl try

Committed: https://crrev.com/c986c543f807c5bca25872c929ca29a6ff982506
Cr-Commit-Position: refs/heads/master@{#328587}

[Yusuke Sato, 2015-05-06 01:35:06]

yusukes@chromium.org changed reviewers:
+ agl@chromium.org, mseaborn@chromium.org

[Yusuke Sato, 2015-05-06 01:35:26]

Adam, Mark, could you take a look?

[Mark Seaborn, 2015-05-06 17:22:22]

LGTM

https://codereview.chromium.org/1127913002/diff/20001/components/nacl/browser...
File components/nacl/browser/nacl_host_message_filter.cc (right):

https://codereview.chromium.org/1127913002/diff/20001/components/nacl/browser...
components/nacl/browser/nacl_host_message_filter.cc:31: //
TODO(nativeclient:3802): Remove |kMaxPreOpenResourceFiles|. Instead, send
Nit: the convention for TODO is to put the name of the person adding it in ()s.

If you're linking to the NaCl issue, can you use the full URL, please?  Not all
Chromium developers will know what "nativeclient:3802" links to.

https://codereview.chromium.org/1127913002/diff/20001/components/nacl/browser...
components/nacl/browser/nacl_host_message_filter.cc:32: // an arbitrary number
of FDs to the plugin with separate IPC messages.
Nit: "plugin" -> "NaCl loader process"

[Yusuke Sato, 2015-05-06 17:50:16]

https://codereview.chromium.org/1127913002/diff/20001/components/nacl/browser...
File components/nacl/browser/nacl_host_message_filter.cc (right):

https://codereview.chromium.org/1127913002/diff/20001/components/nacl/browser...
components/nacl/browser/nacl_host_message_filter.cc:31: //
TODO(nativeclient:3802): Remove |kMaxPreOpenResourceFiles|. Instead, send
On 2015/05/06 17:22:21, Mark Seaborn wrote:
> Nit: the convention for TODO is to put the name of the person adding it in
()s.
> 
> If you're linking to the NaCl issue, can you use the full URL, please?  Not
all
> Chromium developers will know what "nativeclient:3802" links to.

Done.

https://codereview.chromium.org/1127913002/diff/20001/components/nacl/browser...
components/nacl/browser/nacl_host_message_filter.cc:32: // an arbitrary number
of FDs to the plugin with separate IPC messages.
On 2015/05/06 17:22:21, Mark Seaborn wrote:
> Nit: "plugin" -> "NaCl loader process"

Done.

[Yusuke Sato, 2015-05-06 17:50:33]

yusukes@chromium.org changed reviewers:
+ bbudge@chromium.org

[Yusuke Sato, 2015-05-06 17:50:57]

+Bill
Please take a look at ppapi/.

[bbudge, 2015-05-06 17:59:14]

ppapi LGTM

[agl, 2015-05-06 19:17:57]

ipc/ LGTM.

[Yusuke Sato, 2015-05-06 19:40:11]

The CQ bit was checked by yusukes@chromium.org

[Yusuke Sato, 2015-05-06 19:40:11]

The patchset sent to the CQ was uploaded after l-g-t-m from
mseaborn@chromium.org
Link to the patchset: https://codereview.chromium.org/1127913002/#ps40001
(title: "address comments")

[commit-bot: I haz the power, 2015-05-06 19:40:40]

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

[commit-bot: I haz the power, 2015-05-06 19:45:55]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-05-06 19:46:53]

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