Bug 33271 – Number of repaint tests need new pixel results due to r52839

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

[James Robinson]

http://trac.webkit.org/changeset/52839 caused a number of fast/repaint tests to progress (by repainting a smaller amount of the screen).  They need new pixel results.

[James Robinson]

Created attachment 45997 [details]
Patch

[WebKit Review Bot]

style-queue ran check-webkit-style on attachment 45997 [details] without any errors.

Comment on attachment 45997 [details]
Patch

> Index: WebKitTools/Scripts/webkitpy/credentials.py
> ===================================================================
> --- WebKitTools/Scripts/webkitpy/credentials.py	(revision 52868)
> +++ WebKitTools/Scripts/webkitpy/credentials.py	(working copy)
> @@ -96,8 +96,8 @@ class Credentials(object):
>          if Git.in_working_directory(self.cwd):
>              (username, password) = self._credentials_from_git()
>  
> -        if not username or not password:
> -            (username, password) = self._credentials_from_keychain(username)
> +#        if not username or not password:
> +#            (username, password) = self._credentials_from_keychain(username)
>  
>          if not username:
>              username = raw_input("%s login: " % self.host)

This looks irrelevant.

[James Robinson]

Will fix the python noise.

For whoever reviews this:

fast/repaint/background-misaligned.html and
fast/repaint/float-in-new-block-with-layout-delta.html appear to be avoiding an
extra full screen repaint pass, but I'm not entirely sure I am interpreting the
results correctly.

fast/repaint/selected-replaced.html looks like r52839 actually introduced a
slight regression - a few pixels at the bottom of the white area to the right
of the apple appear to be painting an extra time.  It's very small.

The rest of the tests are clear progressions.

[James Robinson]

Created attachment 45998 [details]
Patch

[WebKit Review Bot]

style-queue ran check-webkit-style on attachment 45998 [details] without any errors.

[Eric Seidel (no email)]

Comment on attachment 45998 [details]
Patch

Please file a bug about any regressions.

[WebKit Commit Bot]

Comment on attachment 45998 [details]
Patch

Rejecting patch 45998 from commit-queue.

Failed to run "['/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/svn-apply', '--reviewer', 'Eric Seidel', '--force']" exit_code: 1
Last 500 characters of output:
cksum.rej
patching file LayoutTests/platform/mac/fast/repaint/selected-replaced-expected.checksum
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file LayoutTests/platform/mac/fast/repaint/selected-replaced-expected.checksum.rej
patching file LayoutTests/platform/mac/fast/repaint/transform-replaced-shadows-expected.checksum
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file LayoutTests/platform/mac/fast/repaint/transform-replaced-shadows-expected.checksum.rej

Full output: http://webkit-commit-queue.appspot.com/results/178462

[James Robinson]

Comment on attachment 45998 [details]
Patch

http://trac.webkit.org/changeset/53010

[Eric Seidel (no email)]

Can this bug be closed?

[James Robinson]

Fixed.  (still getting used to bugzilla)