[webnfc] Validate NFCWatchOptions.url from nfc.watch method |
||
Issue descriptionAccording to https://w3c.github.io/web-nfc/#the-watch-method (Step 9), NFCWatchOptions.url must be validated.
,
Jun 8 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/0d88c752c2b25a6925713a573414e27fcae03995 commit 0d88c752c2b25a6925713a573414e27fcae03995 Author: Alexander Shalamov <alexander.shalamov@intel.com> Date: Thu Jun 08 10:42:21 2017 [webnfc] Validate NFCWatchOptions.url in watch operation This CL aligns implementation with specification [1] and validates NFCWatchOptions.url argument, so that developers would be notified immediately about invalid syntax of provided url. [1] https://github.com/w3c/web-nfc/issues/124 Bug: 729000 Change-Id: I1212bda6c7fbfb6da6fb77fd50359c1106638dd5 Reviewed-on: https://chromium-review.googlesource.com/522666 Commit-Queue: Alexander Shalamov <alexander.shalamov@intel.com> Reviewed-by: Reilly Grant <reillyg@chromium.org> Reviewed-by: Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#477937} [delete] https://crrev.com/f456b3c428ad5d71d37612e10336560a81541b56/third_party/WebKit/LayoutTests/nfc/watch-expected.txt [modify] https://crrev.com/0d88c752c2b25a6925713a573414e27fcae03995/third_party/WebKit/LayoutTests/nfc/watch.html [modify] https://crrev.com/0d88c752c2b25a6925713a573414e27fcae03995/third_party/WebKit/Source/modules/nfc/NFC.cpp
,
Jun 8 2017
|
||
►
Sign in to add a comment |
||
Comment 1 by alexande...@intel.com
, Jun 2 2017