Bug 55009

Summary: nrwt: --platform chromium-gpu doesn't work with --worker-model=processes
Product: WebKit Reporter: Dirk Pranke <dpranke>
Component: New BugsAssignee: Dirk Pranke <dpranke>
Status: RESOLVED FIXED    
Severity: Normal CC: jamesr, mihaip, ojan, tony
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: OS X 10.5   
Attachments:
Description Flags
Patch
none
fix baseline_path() to point to the unversioned directory, add tests jamesr: review+

Description Dirk Pranke 2011-02-22 17:07:57 PST
nrwt: --platform chromium-gpu doesn't work with --worker-model=processes
Comment 1 Dirk Pranke 2011-02-22 17:10:08 PST
Created attachment 83417 [details]
Patch
Comment 2 Dirk Pranke 2011-02-22 17:11:54 PST
Note that there's no unit tests for this ... I have a bunch of patches pending that will revamp how we do versioning and clean up this code, so writing unit tests specifically for this patch will be wasted effort. I tested it by hand on the Mac and it should be fine.
Comment 3 James Robinson 2011-02-22 17:12:20 PST
Comment on attachment 83417 [details]
Patch

R=me
Comment 4 Dirk Pranke 2011-02-22 17:37:06 PST
Created attachment 83420 [details]
fix baseline_path() to point to the unversioned directory, add tests
Comment 5 Dirk Pranke 2011-02-22 17:37:36 PST
so much for thinking testing wasn't worth it ... the first patch didn't work right. The second one adds tests :).
Comment 6 James Robinson 2011-02-22 17:38:15 PST
Comment on attachment 83420 [details]
fix baseline_path() to point to the unversioned directory, add tests

Ahoy!
Comment 7 Dirk Pranke 2011-02-22 18:09:30 PST
Committed r79380: <http://trac.webkit.org/changeset/79380>