Bug 140199 – Two tests, which include data uri images, need to be changed and rebaselined since the expected results are incorrect

• Component: Images
• Status: RESOLVED FIXED

[Said Abou-Hallawa]

fast/forms/basic-buttons.html
compositing/tiling/huge-layer-img.html

These tests have data uri images and both assume that the data of the images are loaded in memory immediately after setting the source of the <img> tag.

Both tests need to be changed such that the results can be dumped only after the image onload() event is fired. And of course the expected results need to be changed since they are incorrect.

[Said Abou-Hallawa]

Created attachment 244186 [details]
Patch

[Said Abou-Hallawa]

Created attachment 244189 [details]
Patch

[Build Bot]

Comment on attachment 244189 [details]
Patch

Attachment 244189 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/6083102665342976

New failing tests:
fast/forms/basic-buttons.html

[Build Bot]

Created attachment 244197 [details]
Archive of layout-test-results from ews105 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews105  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.5

[Build Bot]

Comment on attachment 244189 [details]
Patch

Attachment 244189 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/6701012496678912

New failing tests:
fast/forms/basic-buttons.html

[Build Bot]

Created attachment 244199 [details]
Archive of layout-test-results from ews103 for mac-mountainlion

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews103  Port: mac-mountainlion  Platform: Mac OS X 10.8.5

[Said Abou-Hallawa]

Created attachment 244204 [details]
Patch

[Simon Fraser (smfr)]

Comment on attachment 244204 [details]
Patch

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

> LayoutTests/ChangeLog:12
> +        set it as the source of an <img> tag. We need to rebase-line the expected results since

We normally write "rebaseline"

[Said Abou-Hallawa]

Created attachment 244207 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 244207 [details]
Patch

Rejecting attachment 244207 [details] from commit-queue.

[e-mail] does not have committer permissions according to http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/contributors.json.

- If you do not have committer rights please read http://webkit.org/coding/contributing.html for instructions on how to use bugzilla flags.

- If you have committer rights please correct the error in Tools/Scripts/webkitpy/common/config/contributors.json by adding yourself to the file (no review needed).  The commit-queue restarts itself every 2 hours.  After restart the commit-queue will correctly respect your committer rights.

[Said Abou-Hallawa]

Comment on attachment 244207 [details]
Patch

I changed my email on webkit.org before changing it in contributors.json. I changed my email back on webkit.org and I logged in with it to be able to cq+ this bug.

[WebKit Commit Bot]

Comment on attachment 244207 [details]
Patch

Rejecting attachment 244207 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.appspot.com', '--bot-id=webkit-cq-01', 'validate-changelog', '--check-oops', '--non-interactive', 244207, '--port=mac']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

ChangeLog entry in LayoutTests/ChangeLog contains OOPS!.

Full output: http://webkit-queues.appspot.com/results/6527590307201024

[Said Abou-Hallawa]

Created attachment 244214 [details]
Patch

[Simon Fraser (smfr)]

Comment on attachment 244214 [details]
Patch

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

> LayoutTests/fast/forms/basic-buttons.html:66
> +function onImageLoad()
> +{
> +    if (typeof counter == 'undefined')
> +        counter = 0;
> +    
> +    if (++counter == 7) {
> +        printSize('button');
> +        printSize('input');
> +        if (window.testRunner)
> +            testRunner.notifyDone();
> +    }
> +}

This change is actually misleading. Since the images are created at document load time, the image load events should fire before the document load event.

So it would be better to change the test to just wait for the window load event, rather than counting images.

[Said Abou-Hallawa]

Created attachment 244219 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 244219 [details]
Patch

Clearing flags on attachment: 244219

Committed r178078: <http://trac.webkit.org/changeset/178078>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Alexey Proskuryakov]

Comment on attachment 244219 [details]
Patch

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

> LayoutTests/ChangeLog:22
> +        * fast/forms/basic-buttons.html:
> +        * platform/mac-mountainlion/fast/forms/basic-buttons-expected.txt:
> +        * platform/mac/fast/forms/basic-buttons-expected.png:
> +        * platform/mac/fast/forms/basic-buttons-expected.txt:

This test also has results in platform/mac-mavericks, so Mavericks bots started to fail after landing.

> LayoutTests/fast/forms/basic-buttons.html:14
> +if (window.testRunner)
> +    testRunner.waitUntilDone();

This isn't needed. Test results are always dumped after the load event fires, so there is no need for waitUntilDone/notifyDone here.

[Alexey Proskuryakov]

Re-opening to fix the above. I'm going to tackle this now, as Mavericks bots are red.

[Alexey Proskuryakov]

Created attachment 244246 [details]
update the test and results

[WebKit Commit Bot]

Comment on attachment 244246 [details]
update the test and results

Rejecting attachment 244246 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.appspot.com', '--bot-id=webkit-cq-01', 'validate-changelog', '--check-oops', '--non-interactive', 244246, '--port=mac']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

/Volumes/Data/EWS/WebKit/LayoutTests/ChangeLog neither lists a valid reviewer nor contains the string "Unreviewed" or "Rubber stamp" (case insensitive).

Full output: http://webkit-queues.appspot.com/results/6531610396590080

[Alexey Proskuryakov]

Argh. Landed the follow-up manually in r178117.