• CJ DiMeglio's avatar
    Revert "[blinkpy] Find manifest items for tests by URL" · 266b8bce
    CJ DiMeglio authored
    This reverts commit 6f0b5d97.
    
    Reason for revert: Causing Webkit Linux Trusty Leak to fail
    https://ci.chromium.org/buildbot/chromium.webkit/WebKit%20Linux%20Trusty%20Leak/
    
    Original change's description:
    > [blinkpy] Find manifest items for tests by URL
    > 
    > When running WPT in web_tests, test names are more like URLs instead of
    > file paths (to be precise, the part of a test name after external/wpt is
    > a WPT URL), because WPT can generate multiple tests (variations) for a
    > single test file (e.g. .any.js, ?run_type).
    > 
    > Therefore, when asking questions like "is a test slow?", we need to find
    > the manifest item for that test by URL instead of by file path; whereas
    > if we want to know if a file in external/wpt is a test file, we find
    > the manifest item(s) for that file by file path.
    > 
    > This CL separates the two types of manifest queries, and chooses the
    > correct one for each call site. Besides, unit tests are improved to
    > cover the WPT variations (.any.js & ?run_type).
    > 
    > Bug: 831975, 800570
    > Change-Id: I230d5ec7df06b7df1387da1b93788d6cae55d153
    > Reviewed-on: https://chromium-review.googlesource.com/1043160
    > Commit-Queue: Robert Ma <robertma@chromium.org>
    > Reviewed-by: Quinten Yearsley <qyearsley@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#556043}
    
    TBR=qyearsley@chromium.org,robertma@chromium.org
    
    Change-Id: I5fb2c257967e32d1e4fce4277c10d77635e303d4
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: 831975, 800570
    Reviewed-on: https://chromium-review.googlesource.com/1044567Reviewed-by: default avatarCJ DiMeglio <lethalantidote@chromium.org>
    Commit-Queue: CJ DiMeglio <lethalantidote@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#556083}
    266b8bce
wpt_manifest.py 5.13 KB