Bug 42824 – Rebaseline fast/dom/prototype-inheritance.html for chromium

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Hans Wennborg]

Rebaseline fast/dom/prototype-inheritance.html for chromium

[Hans Wennborg]

Created attachment 62297 [details]
Patch

[David Levin]

Comment on attachment 62297 [details]
Patch

Please explain why (in the ChangeLog).

Also I see a new baseline for linux but none for Windows. (A windows version of this files exists: http://trac.webkit.org/browser/trunk/LayoutTests/platform/chromium-win/fast/dom/prototype-inheritance-expected.txt) Why?

[Ojan Vafai]

Comment on attachment 62297 [details]
Patch

> +++ b/LayoutTests/ChangeLog
> @@ -1,3 +1,13 @@
> [phone]  Hans Wennborg  <[e-mail]>
> +
> +        Reviewed by NOBODY (OOPS!).
> +
> +        Rebaseline fast/dom/prototype-inheritance.html for chromium
> +        https://bugs.webkit.org/show_bug.cgi?id=42824
> +

There should be more description here. For example, what revision did it start failing at? That way, the next person who looks at this patch will have enough information to understand why rebaselining is correct.

This looks like it's failing on Windows as well. Also, the Windows and Linux expectations for this test should be the same. You shouldn't need to add the linux expected results at all (Linux falls back to the Windows expectations).

[Ojan Vafai]

Weird. I don't know why bugzilla didn't warn me about an update conflict. Sorry for the noise.

[Hans Wennborg]

(In reply to comment #2)
> (From update of attachment 62297 [details])
> Please explain why (in the ChangeLog).
Done.

> 
> Also I see a new baseline for linux but none for Windows. (A windows version of this files exists: http://trac.webkit.org/browser/trunk/LayoutTests/platform/chromium-win/fast/dom/prototype-inheritance-expected.txt) Why?
Hmm, I simply ran the script, and it must have failed to rebaseline for Windows. When I ran it in the same way today it worked. Thanks for spotting this.

[Hans Wennborg]

Created attachment 62397 [details]
Patch

[Ojan Vafai]

Comment on attachment 62397 [details]
Patch

If you want this to go into the commit-queue instead of committing it yourself, you'll need to set commit-queue+. In the future if you set commit-queue?, then that lets the reviewer know you want it in the commit-queue once it's approved.

[WebKit Commit Bot]

Comment on attachment 62397 [details]
Patch

Clearing flags on attachment: 62397

Committed r63986: <http://trac.webkit.org/changeset/63986>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.