[webnfc]The function nfc.watch does not work when NFCWatchOptions recordType is set to 'url'.
Reported by
canx....@intel.com,
May 8 2017
|
|||||
Issue descriptionSteps to reproduce the problem: Test case: https://thewebera.github.io/web-test-suite/web-nfc/nfc/nfc_watch_recordtype_url-manual.https.html Test name: Test that nfc.watch succeeds when NFCWatchOptions recordType is set to 'url'. 1. Push test data with “url” recordType and watch with below watchOptions: let watchOptions = { recordType : "url", mediaType : "text/plain", url : "https://w3c.github.io/*" } 2.Tap an NFC tag to the test device with NFC support. What is the expected behavior? nfc.watch succeeds when NFCWatchOptions recordType is set to 'url'. What went wrong? No response ,then test case get TIMEOUT. Did this work before? N/A Does this work in other browsers? N/A Chrome version: 60.0.3089.0 Channel: canary OS Version: Nexus 7(Android 5.1) Flash Version: Shockwave Flash 25.0 r0
,
May 8 2017
,
May 19 2017
,
May 22 2017
Created 2 spec issues / fixes. https://github.com/w3c/web-nfc/pull/121 https://github.com/w3c/web-nfc/pull/119 Started working on implementation.
,
Jun 9 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/eab288dcba2b859deca28b66fafbd5fef67d4ce8 commit eab288dcba2b859deca28b66fafbd5fef67d4ce8 Author: Alexander Shalamov <alexander.shalamov@intel.com> Date: Fri Jun 09 03:12:54 2017 [webnfc] Implement URL pattern matching algorithm This CL implements WebNFC URL watch pattern matching algorithm [1]. New test cases added to unit tests. [1] https://w3c.github.io/web-nfc/#url-patterns BUG= 719293 Change-Id: Ia864dfa3838140c35eec9f7fd1a572977bce0ec2 Reviewed-on: https://chromium-review.googlesource.com/512622 Reviewed-by: Reilly Grant <reillyg@chromium.org> Commit-Queue: Alexander Shalamov <alexander.shalamov@intel.com> Cr-Commit-Position: refs/heads/master@{#478178} [modify] https://crrev.com/eab288dcba2b859deca28b66fafbd5fef67d4ce8/services/device/nfc/android/java/src/org/chromium/device/nfc/NfcImpl.java [modify] https://crrev.com/eab288dcba2b859deca28b66fafbd5fef67d4ce8/services/device/nfc/android/junit/src/org/chromium/device/nfc/NFCTest.java
,
Jun 9 2017
,
Jun 15 2017
I verified this issue with chrome 61.0.3128.0, it has been fixed. Close. Thanks!
,
Jun 15 2017
Thanks canx.cao@ |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by alexande...@intel.com
, May 8 2017Owner: alexande...@intel.com