New issue
Advanced search Search tips

Issue 740176 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

[WPT Export] GitHub gives 400 status for request to remove provisional PR label

Project Member Reported by qyears...@chromium.org, Jul 7 2017

Issue description

I uploaded a CL for this but I want to verify that it actually works locally before landing:
https://chromium-review.googlesource.com/c/563468/

Verified that the exporter did not make a PR for my whitespace change, yay! crrev.com/c/563958/
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 7 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/3aa4657545a1a3e3d83e879dda67bb0966a6670b

commit 3aa4657545a1a3e3d83e879dda67bb0966a6670b
Author: Jeff Carpenter <jeffcarp@chromium.org>
Date: Fri Jul 07 23:01:45 2017

[WPT Export] Remove extra whitespace line from webusb WPT test

This is to verify recent changes to the exporter are functioning properly.

Bug:  740176 
Change-Id: I1fb68dafff0c3b760f3f50abe8300d3ead5124f4
Reviewed-on: https://chromium-review.googlesource.com/563958
Reviewed-by: Quinten Yearsley <qyearsley@chromium.org>
Commit-Queue: Jeff Carpenter <jeffcarp@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485084}
[modify] https://crrev.com/3aa4657545a1a3e3d83e879dda67bb0966a6670b/third_party/WebKit/LayoutTests/external/wpt/webusb/usbIsochronousInTransferResult.https.html

Verified the CL works locally, more context in the CL replies. Landing.
Status: Fixed (was: Assigned)

Sign in to add a comment