Bug 68680 – Let TestWebKitAPI and webkit_unit_tests work for chromium-win

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Xianzhu Wang]

Bug 67756 and bug 68206 let TestWebKitAPI work on Chromium. However the run-api-tests script still doesn't work on chromium-win, because determineBaseProductDir() doesn't return correct value for chromium-win. I left a FIXME at http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitdirs.pm?rev=95647#L164.

[Xianzhu Wang]

Created attachment 108446 [details]
patch

I guessed the baseProductDir according to run-chromium-webkit-unit-tests.py, but I haven't tested it because I haven't a Windows build environment.

[Tony Chang]

Comment on attachment 108446 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=108446&action=review

> Tools/Scripts/webkitdirs.pm:164
> -            # FIXME: set the correct baseProductDir for chromium-win.
> +            $baseProductDir = "$sourceDir/Source/WebKit/chromium";

I think this is $sourceDir/Source/WebKit/chromium/build after chromium r102514.

[Xianzhu Wang]

Created attachment 108781 [details]
patch v2

The change in chromium also affected run-chromium-webkit-unit-tests. Fixed it by the way.

[Tony Chang]

Comment on attachment 108781 [details]
patch v2

View in context: https://bugs.webkit.org/attachment.cgi?id=108781&action=review

> Tools/Scripts/run-chromium-webkit-unit-tests:-38
> -# Allow running this script from any directory.
> -my $sourceRootDir = File::Spec->catfile($FindBin::Bin, "../..");
> -chdir($sourceRootDir);

Why was this removed?  Did it no longer work?

[Xianzhu Wang]

(In reply to comment #4)
> (From update of attachment 108781 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=108781&action=review
> 
> > Tools/Scripts/run-chromium-webkit-unit-tests:-38
> > -# Allow running this script from any directory.
> > -my $sourceRootDir = File::Spec->catfile($FindBin::Bin, "../..");
> > -chdir($sourceRootDir);
> 
> Why was this removed?  Did it no longer work?

This is no longer needed because we use productDir() to find the executable. I've verified that there is no test depending on cwd. webkit_unit_test can be run from any dir.

[WebKit Review Bot]

Comment on attachment 108781 [details]
patch v2

Clearing flags on attachment: 108781

Committed r96235: <http://trac.webkit.org/changeset/96235>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.