[webnfc] Restrict NFC message size to 32KB

[webnfc] Restrict NFC message size to 32KB

This patch limits maximum NFC message size for nfc.push() operation to
32KB. The 32KB limit is selected based on several factors: capacity of
consumer NFC tags, data transfer rate and IPC mechanism that is used by
mojo service. Layout test is modified to test that promise is rejected
for nfc.push() operation when message size exceeds 32KB.

BUG=520391
Committed: https://crrev.com/b706a428c0ca98907c6ece6aa80f5ce215381acf
Cr-Commit-Position: refs/heads/master@{#434131}

[shalamov, 2016-11-22 11:26:58]

The CQ bit was checked by alexander.shalamov@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2016-11-22 11:27:06]

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

[commit-bot: I haz the power, 2016-11-22 12:49:04]

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

[commit-bot: I haz the power, 2016-11-22 12:49:05]

Dry run: This issue passed the CQ dry run.

[shalamov, 2016-11-22 12:49:25]

alexander.shalamov@intel.com changed reviewers:
+ haraken@chromium.org, kenneth.r.christiansen@intel.com, tsepez@chromium.org

[shalamov, 2016-11-22 12:49:25]

Based on feedback from @haraken, I've moved max NFC message size restriction
from codereview.chromium.org/1759373003/ to this CL. On the browser side, when
NFC message is received, check is done in this CL
https://codereview.chromium.org/1765533004

Please take a look.

[kenneth.r.christiansen, 2016-11-22 13:11:23]

lgtm

https://codereview.chromium.org/2524743002/diff/1/third_party/WebKit/Source/m...
File third_party/WebKit/Source/modules/nfc/NFC.cpp (right):

https://codereview.chromium.org/2524743002/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/nfc/NFC.cpp:429: size_t getNFCMessageSize(
measure instead of get?

[haraken, 2016-11-22 14:25:07]

LGTM

[Tom Sepez, 2016-11-22 16:57:12]

lgtm

[shalamov, 2016-11-23 08:09:03]

The CQ bit was checked by alexander.shalamov@intel.com

[commit-bot: I haz the power, 2016-11-23 08:09:08]

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

[commit-bot: I haz the power, 2016-11-23 08:13:52]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1479888543860840, "parent_rev":
"d18e8c6b68a8f43289946a6b34890114bcea918f", "commit_rev":
"e53dcb6e9e930106426481a3b049b38db3d75e19"}

[commit-bot: I haz the power, 2016-11-23 08:14:16]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-23 08:16:02]

Description was changed from

==========
[webnfc] Restrict NFC message size to 32KB

This patch limits maximum NFC message size for nfc.push() operation to
32KB. The 32KB limit is selected based on several factors: capacity of
consumer NFC tags, data transfer rate and IPC mechanism that is used by
mojo service. Layout test is modified to test that promise is rejected
for nfc.push() operation when message size exceeds 32KB.

BUG=520391
==========

to

==========
[webnfc] Restrict NFC message size to 32KB

This patch limits maximum NFC message size for nfc.push() operation to
32KB. The 32KB limit is selected based on several factors: capacity of
consumer NFC tags, data transfer rate and IPC mechanism that is used by
mojo service. Layout test is modified to test that promise is rejected
for nfc.push() operation when message size exceeds 32KB.

BUG=520391

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

[commit-bot: I haz the power, 2016-11-23 08:16:03]

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