New issue
Advanced search Search tips

Issue 643761 link

Starred by 2 users
Status: Fixed
Owner:
qyears...@chromium.org
Closed: Sep 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Feature



Sign in to add a comment

Make update-w3c-test-expectations easier to use.

Project Member Reported by qyears...@chromium.org, Sep 2 2016 
Specifically:
 - Provide a --help and --verbose option (using argparse).
 - Add more logging.

Relatedly, update-w3c-deps --auto-update could be made more user-friendly by adding more logging.
Project Member

Comment 1 by bugdroid1@chromium.org, Sep 15 2016 

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

commit 194e7e00f26329a3f0170e831c9c07b61ef20aeb
Author: qyearsley <qyearsley@chromium.org>
Date: Thu Sep 15 00:10:35 2016

Add --help and --verbose options to update-w3c-test-expectations.

This CL also adds more logging in update_w3c_test_expectations.py, and changes deps_updater.py to run this script with the --verbose flag.

Purpose of this CL:
 1. Make it easier to diagnose issues with w3c-test-autoroller in the future.
 2. Start to make update-w3c-test-expectations better to use directly.

BUG= 643761 

Review-Url: https://codereview.chromium.org/2337183002
Cr-Commit-Position: refs/heads/master@{#418718}

[modify] https://crrev.com/194e7e00f26329a3f0170e831c9c07b61ef20aeb/third_party/WebKit/Tools/Scripts/update-w3c-test-expectations
[modify] https://crrev.com/194e7e00f26329a3f0170e831c9c07b61ef20aeb/third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater.py
[modify] https://crrev.com/194e7e00f26329a3f0170e831c9c07b61ef20aeb/third_party/WebKit/Tools/Scripts/webkitpy/w3c/update_w3c_test_expectations.py
[modify] https://crrev.com/194e7e00f26329a3f0170e831c9c07b61ef20aeb/third_party/WebKit/Tools/Scripts/webkitpy/w3c/update_w3c_test_expectations_unittest.py

Comment 2 by qyears...@chromium.org, Sep 25 2016

Owner: qyears...@chromium.org
Status: Fixed (was: Unconfirmed)
►

Sign in to add a comment