Bug 49151 – [Chromium] REGRESSION(71474?) failing webkit tests on chromium canaries

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Fumitoshi Ukai]

Since webkit r71474, webkit tests on chromium canaries started failing.

Win: http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Win/builds/872
Linux: http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Linux/builds/1409
Mac: http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Mac/builds/588

[Fumitoshi Ukai]

Reverted r71475 and r71474

[Dirk Pranke]

Looks like scm.find_checkout_root() isn't working correctly for an SVN-based checkout when the caller is outside of the root. Investigating further.

As a side note, it would be good if we could run at least a subset of the scm unittest on the bots ... I'll have to think about how to do that w/o running them all. Maybe as a short-term hack, move them out of the checkout directory?

[Dirk Pranke]

So, r71475 and r71474 aren't broken per se, but they started using  the code in scm to find the top of the webkit tree, instead of using its own logic, and it looks like that might be what broke.

[Dirk Pranke]

Created attachment 73290 [details]
Patch

[Eric Seidel (no email)]

Comment on attachment 73290 [details]
Patch

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

> WebKitTools/ChangeLog:31
> [phone]  Fumitoshi Ukai  <[e-mail]>
> +
> +        Unreviewed, rolling out r71475.

I don't think you meant to move Ukai's entry.

> WebKitTools/Scripts/webkitpy/common/system/filesystem_mock.py:55
> +    def isdir(self, path):
> +        # FIXME: Implement :)
> +        raise NotImplementedError

I'm still not a big fan of these stubs. :)

> WebKitTools/Scripts/webkitpy/common/system/filesystem_mock.py:77
> +                exception = IOError()
> +                exception.errno = errno.ENOENT
> +                exception.filename = path
> +                exception.strerror = "No such file or directory"

I bet there is a constructor...

> WebKitTools/Scripts/webkitpy/layout_tests/port/config.py:126
> +        # Note: this code somewhat duplicates the code in
> +        # scm.find_checkout_root(). However, that code only works if the top
> +        # of the SCM repository also matches the top of the WebKit tree. The
> +        # Chromium ports, for example, only check out subdirectories like
> +        # WebKitTools/Scripts, and so we still have to do additional work
> +        # to find the top of the tree.

I suspect lots of other pieces of webkitpy break for chromium too.

[Dirk Pranke]

(In reply to comment #5)
> (From update of attachment 73290 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=73290&action=review
> 
> > WebKitTools/ChangeLog:31
> > [phone]  Fumitoshi Ukai  <[e-mail]>
> > +
> > +        Unreviewed, rolling out r71475.
> 
> I don't think you meant to move Ukai's entry.
> 

No, I didn't. I caught the other one, but not this one. I'll add it back in. I'm not sure why I didn't notice it in the diff.

> > WebKitTools/Scripts/webkitpy/common/system/filesystem_mock.py:55
> > +    def isdir(self, path):
> > +        # FIXME: Implement :)
> > +        raise NotImplementedError
> 
> I'm still not a big fan of these stubs. :)
> 

Then, for you, I'll remove them :)

> > WebKitTools/Scripts/webkitpy/common/system/filesystem_mock.py:77
> > +                exception = IOError()
> > +                exception.errno = errno.ENOENT
> > +                exception.filename = path
> > +                exception.strerror = "No such file or directory"
> 
> I bet there is a constructor...
> 

Yeah, sort of. It appears to be relatively undocumented, but is obvious enough how to use it. Added.

[Dirk Pranke]

Committed r71580: <http://trac.webkit.org/changeset/71580>

[Mihai Parparita]

This appears to have broken NRWT's ability to pick up the currently set configuration (even though the debug config is used, it tries to look for release binaries):

$ set-webkit-configuration --debug
$ new-run-webkit-tests --chromium --use-drt
Defaulting to one child - see https://bugs.webkit.org/show_bug.cgi?id=38553
Unable to find test driver
    at /Users/mihaip/Developer/source/chromium1/src/third_party/WebKit/WebKit/chromium/xcodebuild/Release/DumpRenderTree.app/Contents/MacOS/DumpRenderTree

Unable to find layout test helper
    at /Users/mihaip/Developer/source/chromium1/src/third_party/WebKit/WebKit/chromium/xcodebuild/Release/LayoutTestHelper
Unable to find image diff exe
    at /Users/mihaip/Developer/source/chromium1/src/third_party/WebKit/WebKit/chromium/xcodebuild/Release/ImageDiff
    To override, invoke with --no-pixel-tests

wdiff not found. Install using MacPorts or some other means
For complete Mac build requirements, please see:

    http://code.google.com/p/chromium/wiki/MacBuildInstructions
Build check failed