Bug 50359 - [chromium] force NRWT to use DRT
Summary: [chromium] force NRWT to use DRT
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other OS X 10.5
: P2 Normal
Assignee: Tony Chang
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-12-01 16:44 PST by Tony Chang
Modified: 2010-12-08 11:00 PST (History)
3 users (show)

See Also:


Attachments
Patch (1.42 KB, patch)
2010-12-01 16:44 PST, Tony Chang
ojan: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Tony Chang 2010-12-01 16:44:16 PST
[chromium] force NRWT to use DRT
Comment 1 Tony Chang 2010-12-01 16:44:41 PST
Created attachment 75333 [details]
Patch
Comment 2 Tony Chang 2010-12-01 16:45:51 PST
This is intentionally small so when the canaries break, I can roll back.  If it sticks, I'll make a follow up patch to remove the flag (and to remove the flag from the bots).
Comment 3 Dirk Pranke 2010-12-01 16:47:08 PST
Heh. LGTM!
Comment 4 Tony Chang 2010-12-02 10:59:25 PST
Just an fyi for victorw, the current gardener.
Comment 5 Tony Chang 2010-12-02 11:00:23 PST
Committed r73164: <http://trac.webkit.org/changeset/73164>
Comment 6 Tony Chang 2010-12-02 15:20:44 PST
Reverted r73164 for reason:

broke debugger tests on win

Committed r73207: <http://trac.webkit.org/changeset/73207>
Comment 7 Tony Chang 2010-12-07 13:06:05 PST
Committed r73456: <http://trac.webkit.org/changeset/73456>
Comment 8 Dirk Pranke 2010-12-07 20:20:01 PST
Okay, I should have caught this before, but realizing this now ... changing the default this way doesn't give us a way to run w/ TestShell anymore. We need to add a --no-use-drt flag (or a --use-test-shell or something). It's not hard to hack the RWT code to work around this, obviously, and we don't really want to encourage people to use this code, but as long as TestShell is being supported, I'd prefer we have a way to run this without local mods.

Thoughts?

Also, any thought as to when we might actually stop supporting TestShell and remove the code from the Tree?
Comment 9 Ojan Vafai 2010-12-08 08:39:58 PST
This was the most expedient way we could think of making this change. The plan, if this sticks for the next day or two, is to fully remove all TestShell support.
Comment 10 Tony Chang 2010-12-08 09:57:20 PST
I'm going to write a patch today for --use-test-shell (jamesr's recommendation for testing for DRT specific failures).

As mentioned in comment #2, this patch was intentionally small.
Comment 11 Dirk Pranke 2010-12-08 11:00:53 PST
(In reply to comment #10)
> I'm going to write a patch today for --use-test-shell (jamesr's recommendation for testing for DRT specific failures).
> 
> As mentioned in comment #2, this patch was intentionally small.

Sure, I think keeping the patch small was a good thing to. Just a bit too small ;)