Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[webnfc] NDEFScanOptions#mediaType being undefined means do not filter #21017

Merged
merged 1 commit into from Jan 8, 2020

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Jan 3, 2020

And the empty string should just match on empty.

The spec change:
w3c/web-nfc#496
w3c/web-nfc#498

BUG=520391

Change-Id: Idae55b976a6fbbdd252f6227cbe13ee50d16269c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1986632
Reviewed-by: Rijubrata Bhaumik <rijubrata.bhaumik@intel.com>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Leon Han <leon.han@intel.com>
Cr-Commit-Position: refs/heads/master@{#728813}

Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The review process for this patch is being conducted in the Chromium project.

And the empty string should just match on empty.

The spec change:
w3c/web-nfc#496
w3c/web-nfc#498

BUG=520391

Change-Id: Idae55b976a6fbbdd252f6227cbe13ee50d16269c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1986632
Reviewed-by: Rijubrata Bhaumik <rijubrata.bhaumik@intel.com>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Leon Han <leon.han@intel.com>
Cr-Commit-Position: refs/heads/master@{#728813}
@stephenmcgruer
Copy link
Contributor

Failure seems like an unrelated infrastructure/ tests failure:

Unexpected Results
------------------
/infrastructure/assumptions/allowed-to-play.html
  CRASH /infrastructure/assumptions/allowed-to-play.html <-- from Marionette

@stephenmcgruer stephenmcgruer merged commit 32cb1cf into master Jan 8, 2020
@stephenmcgruer stephenmcgruer deleted the chromium-export-cl-1986632 branch January 8, 2020 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants