Bug 34006

Summary: svg/custom/text-zoom.xhtml fails on Windows
Product: WebKit Reporter: Adam Roben (:aroben) <aroben>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: mitz
Priority: P2 Keywords: InRadar, LayoutTestFailure, PlatformOnly
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Windows XP   
URL: http://build.webkit.org/results/Windows%20Release%20(Tests)/r53703%20(8266)/svg/custom/text-zoom-pretty-diff.html
Attachments:
Description Flags
Set the default value for WebKitZoomsTextOnlyPreferenceKey to true mjs: review+

Description Adam Roben (:aroben) 2010-01-22 09:31:24 PST
svg/custom/text-zoom.xhtml fails on Windows. See <http://build.webkit.org/results/Windows%20Release%20(Tests)/r53703%20(8266)/svg/custom/text-zoom-pretty-diff.html> for the diff produced by the test.

It looks like the green rectangle is getting zoomed when it shouldn't.
Comment 1 Adam Roben (:aroben) 2010-01-22 09:31:52 PST
<rdar://problem/7569820>
Comment 2 Adam Roben (:aroben) 2010-01-22 09:33:01 PST
The test fails in Safari as well:

If I check "Zoom Text Only" in the View menu, then zoom, the rectangle zooms and I do see red.

The same thing happens if I uncheck "Zoom Text Only".
Comment 3 Adam Roben (:aroben) 2010-01-22 09:43:51 PST
(In reply to comment #2)
> The test fails in Safari as well:
> 
> If I check "Zoom Text Only" in the View menu, then zoom, the rectangle zooms
> and I do see red.
> 
> The same thing happens if I uncheck "Zoom Text Only".

I was testing with r53683. With r53686 the test passes in Safari, but still fails in DRT.
Comment 4 Adam Roben (:aroben) 2010-01-22 10:33:27 PST
Expected failure results for this test were checked in in r53704, so this test now appears to pass on Windows, but the results are incorrect.
Comment 5 mitz 2010-01-22 12:53:49 PST
The problem is that WebPreferences::initializeDefaultSettings() fails to set the default value of WebKitZoomsTextOnlyPreferenceKey to true.
Comment 6 mitz 2010-01-23 17:32:16 PST
Created attachment 47284 [details]
Set the default value for WebKitZoomsTextOnlyPreferenceKey to true
Comment 7 Maciej Stachowiak 2010-01-23 17:45:28 PST
Comment on attachment 47284 [details]
Set the default value for WebKitZoomsTextOnlyPreferenceKey to true

r=me
Comment 8 mitz 2010-01-23 17:51:44 PST
Fixed in <http://trac.webkit.org/projects/webkit/changeset/53773>.