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

Improve Code Coverage for WebUSB #12678

Merged
merged 1 commit into from Aug 29, 2018

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Aug 24, 2018

This change improves the code coverage for the sources files under
//third_party/blink/renderer/modules/webusb.

The affected files are:

  • usb.cc
  • usb_alternate_interface.cc
  • usb_configuration.cc
  • usb_device.cc
  • usb_endpoint.cc
  • usb_interface.cc

Bug: 854831
Change-Id: Ie67533ece27f1c944b0e9186d01a61219a30f805
Reviewed-on: https://chromium-review.googlesource.com/1183789
Commit-Queue: Ovidio Henriquez odejesush@chromium.org
Reviewed-by: Reilly Grant reillyg@chromium.org
Cr-Commit-Position: refs/heads/master@{#586930}

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.

Already reviewed downstream.

@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-1183789 branch 3 times, most recently from 5a339e7 to 796d27e Compare August 28, 2018 22:52
This change improves the code coverage for the sources files under
//third_party/blink/renderer/modules/webusb.

The affected files are:
* usb.cc
* usb_alternate_interface.cc
* usb_configuration.cc
* usb_device.cc
* usb_endpoint.cc
* usb_interface.cc

Bug: 854831
Change-Id: Ie67533ece27f1c944b0e9186d01a61219a30f805
Reviewed-on: https://chromium-review.googlesource.com/1183789
Commit-Queue: Ovidio Henriquez <odejesush@chromium.org>
Reviewed-by: Reilly Grant <reillyg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#586930}
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