<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>34006</bug_id>
          
          <creation_ts>2010-01-22 09:31:24 -0800</creation_ts>
          <short_desc>svg/custom/text-zoom.xhtml fails on Windows</short_desc>
          <delta_ts>2010-01-23 17:51:44 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Tools / Tests</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Windows XP</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc>http://build.webkit.org/results/Windows%20Release%20(Tests)/r53703%20(8266)/svg/custom/text-zoom-pretty-diff.html</bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar, LayoutTestFailure, PlatformOnly</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Adam Roben (:aroben)">aroben</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>mitz</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>183675</commentid>
    <comment_count>0</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2010-01-22 09:31:24 -0800</bug_when>
    <thetext>svg/custom/text-zoom.xhtml fails on Windows. See &lt;http://build.webkit.org/results/Windows%20Release%20(Tests)/r53703%20(8266)/svg/custom/text-zoom-pretty-diff.html&gt; for the diff produced by the test.

It looks like the green rectangle is getting zoomed when it shouldn&apos;t.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>183676</commentid>
    <comment_count>1</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2010-01-22 09:31:52 -0800</bug_when>
    <thetext>&lt;rdar://problem/7569820&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>183677</commentid>
    <comment_count>2</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2010-01-22 09:33:01 -0800</bug_when>
    <thetext>The test fails in Safari as well:

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

The same thing happens if I uncheck &quot;Zoom Text Only&quot;.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>183679</commentid>
    <comment_count>3</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2010-01-22 09:43:51 -0800</bug_when>
    <thetext>(In reply to comment #2)
&gt; The test fails in Safari as well:
&gt; 
&gt; If I check &quot;Zoom Text Only&quot; in the View menu, then zoom, the rectangle zooms
&gt; and I do see red.
&gt; 
&gt; The same thing happens if I uncheck &quot;Zoom Text Only&quot;.

I was testing with r53683. With r53686 the test passes in Safari, but still fails in DRT.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>183694</commentid>
    <comment_count>4</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2010-01-22 10:33:27 -0800</bug_when>
    <thetext>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.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>183761</commentid>
    <comment_count>5</comment_count>
    <who name="">mitz</who>
    <bug_when>2010-01-22 12:53:49 -0800</bug_when>
    <thetext>The problem is that WebPreferences::initializeDefaultSettings() fails to set the default value of WebKitZoomsTextOnlyPreferenceKey to true.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>184035</commentid>
    <comment_count>6</comment_count>
      <attachid>47284</attachid>
    <who name="">mitz</who>
    <bug_when>2010-01-23 17:32:16 -0800</bug_when>
    <thetext>Created attachment 47284
Set the default value for WebKitZoomsTextOnlyPreferenceKey to true</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>184036</commentid>
    <comment_count>7</comment_count>
      <attachid>47284</attachid>
    <who name="Maciej Stachowiak">mjs</who>
    <bug_when>2010-01-23 17:45:28 -0800</bug_when>
    <thetext>Comment on attachment 47284
Set the default value for WebKitZoomsTextOnlyPreferenceKey to true

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>184038</commentid>
    <comment_count>8</comment_count>
    <who name="">mitz</who>
    <bug_when>2010-01-23 17:51:44 -0800</bug_when>
    <thetext>Fixed in &lt;http://trac.webkit.org/projects/webkit/changeset/53773&gt;.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>47284</attachid>
            <date>2010-01-23 17:32:16 -0800</date>
            <delta_ts>2010-01-23 17:45:27 -0800</delta_ts>
            <desc>Set the default value for WebKitZoomsTextOnlyPreferenceKey to true</desc>
            <filename>34006_r1.diff</filename>
            <type>text/plain</type>
            <size>3453</size>
            <attacher>mitz</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYktpdC93aW4vQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYktpdC93aW4vQ2hh
bmdlTG9nCShyZXZpc2lvbiA1Mzc3MikKKysrIFdlYktpdC93aW4vQ2hhbmdlTG9nCSh3b3JraW5n
IGNvcHkpCkBAIC0xLDMgKzEsMTQgQEAKKzIwMTAtMDEtMjMgIERhbiBCZXJuc3RlaW4gIDxtaXR6
QGFwcGxlLmNvbT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAg
ICAgICA8cmRhcjovL3Byb2JsZW0vNzU2OTgyMD4gc3ZnL2N1c3RvbS90ZXh0LXpvb20ueGh0bWwg
ZmFpbHMgb24gV2luZG93cworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1
Zy5jZ2k/aWQ9MzQwMDYKKworICAgICAgICAqIFdlYlByZWZlcmVuY2VzLmNwcDoKKyAgICAgICAg
KFdlYlByZWZlcmVuY2VzOjppbml0aWFsaXplRGVmYXVsdFNldHRpbmdzKTogQWRkZWQgV2ViS2l0
Wm9vbXNUZXh0T25seVByZWZlcmVuY2VLZXkKKyAgICAgICAgd2l0aCBhIGRlZmF1bHQgdmFsdWUg
b2YgdHJ1ZS4KKwogMjAxMC0wMS0yMCAgU3RldmUgRmFsa2VuYnVyZyAgPHNmYWxrZW5AYXBwbGUu
Y29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5IFNhbSBXZWluaWcuCkluZGV4OiBXZWJLaXQvd2lu
L1dlYlByZWZlcmVuY2VzLmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJLaXQvd2luL1dlYlByZWZlcmVu
Y2VzLmNwcAkocmV2aXNpb24gNTM3NTIpCisrKyBXZWJLaXQvd2luL1dlYlByZWZlcmVuY2VzLmNw
cAkod29ya2luZyBjb3B5KQpAQCAtMSw1ICsxLDUgQEAKIC8qCi0gKiBDb3B5cmlnaHQgKEMpIDIw
MDYsIDIwMDcsIDIwMDgsIDIwMDkgQXBwbGUgSW5jLiAgQWxsIHJpZ2h0cyByZXNlcnZlZC4KKyAq
IENvcHlyaWdodCAoQykgMjAwNiwgMjAwNywgMjAwOCwgMjAwOSwgMjAxMCBBcHBsZSBJbmMuICBB
bGwgcmlnaHRzIHJlc2VydmVkLgogICoKICAqIFJlZGlzdHJpYnV0aW9uIGFuZCB1c2UgaW4gc291
cmNlIGFuZCBiaW5hcnkgZm9ybXMsIHdpdGggb3Igd2l0aG91dAogICogbW9kaWZpY2F0aW9uLCBh
cmUgcGVybWl0dGVkIHByb3ZpZGVkIHRoYXQgdGhlIGZvbGxvd2luZyBjb25kaXRpb25zCkBAIC0y
MTIsNiArMjEyLDcgQEAgdm9pZCBXZWJQcmVmZXJlbmNlczo6aW5pdGlhbGl6ZURlZmF1bHRTZQog
ICAgIENGRGljdGlvbmFyeUFkZFZhbHVlKGRlZmF1bHRzLCBDRlNUUihXZWJLaXREYXRhYmFzZXNF
bmFibGVkUHJlZmVyZW5jZUtleSksIGtDRkJvb2xlYW5UcnVlKTsKICAgICBDRkRpY3Rpb25hcnlB
ZGRWYWx1ZShkZWZhdWx0cywgQ0ZTVFIoV2ViS2l0TG9jYWxTdG9yYWdlRW5hYmxlZFByZWZlcmVu
Y2VLZXkpLCBrQ0ZCb29sZWFuVHJ1ZSk7CiAgICAgQ0ZEaWN0aW9uYXJ5QWRkVmFsdWUoZGVmYXVs
dHMsIENGU1RSKFdlYktpdEV4cGVyaW1lbnRhbE5vdGlmaWNhdGlvbnNFbmFibGVkUHJlZmVyZW5j
ZUtleSksIGtDRkJvb2xlYW5GYWxzZSk7CisgICAgQ0ZEaWN0aW9uYXJ5QWRkVmFsdWUoZGVmYXVs
dHMsIENGU1RSKFdlYktpdFpvb21zVGV4dE9ubHlQcmVmZXJlbmNlS2V5KSwga0NGQm9vbGVhblRy
dWUpOwogICAgIENGRGljdGlvbmFyeUFkZFZhbHVlKGRlZmF1bHRzLCBDRlNUUihXZWJLaXRBbGxv
d0FuaW1hdGVkSW1hZ2VzUHJlZmVyZW5jZUtleSksIGtDRkJvb2xlYW5UcnVlKTsKICAgICBDRkRp
Y3Rpb25hcnlBZGRWYWx1ZShkZWZhdWx0cywgQ0ZTVFIoV2ViS2l0QWxsb3dBbmltYXRlZEltYWdl
TG9vcGluZ1ByZWZlcmVuY2VLZXkpLCBrQ0ZCb29sZWFuVHJ1ZSk7CiAgICAgQ0ZEaWN0aW9uYXJ5
QWRkVmFsdWUoZGVmYXVsdHMsIENGU1RSKFdlYktpdERpc3BsYXlJbWFnZXNLZXkpLCBrQ0ZCb29s
ZWFuVHJ1ZSk7CkluZGV4OiBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gTGF5
b3V0VGVzdHMvQ2hhbmdlTG9nCShyZXZpc2lvbiA1Mzc3MikKKysrIExheW91dFRlc3RzL0NoYW5n
ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDEyIEBACisyMDEwLTAxLTIzICBEYW4gQmVy
bnN0ZWluICA8bWl0ekBhcHBsZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChP
T1BTISkuCisKKyAgICAgICAgPHJkYXI6Ly9wcm9ibGVtLzc1Njk4MjA+IHN2Zy9jdXN0b20vdGV4
dC16b29tLnhodG1sIGZhaWxzIG9uIFdpbmRvd3MKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtp
dC5vcmcvc2hvd19idWcuY2dpP2lkPTM0MDA2CisKKyAgICAgICAgKiBwbGF0Zm9ybS93aW4vc3Zn
L2N1c3RvbS90ZXh0LXpvb20tZXhwZWN0ZWQudHh0OiBSZW1vdmVkLgorCiAyMDEwLTAxLTIzICBS
b2JlcnQgSG9nYW4gIDxyb2JlcnRAcm9iZXJ0aG9nYW4ubmV0PgogCiAgICAgICAgIFJldmlld2Vk
IGJ5IEtlbm5ldGggUm9oZGUgQ2hyaXN0aWFuc2VuLgpJbmRleDogTGF5b3V0VGVzdHMvcGxhdGZv
cm0vd2luL3N2Zy9jdXN0b20vdGV4dC16b29tLWV4cGVjdGVkLnR4dAo9PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBM
YXlvdXRUZXN0cy9wbGF0Zm9ybS93aW4vc3ZnL2N1c3RvbS90ZXh0LXpvb20tZXhwZWN0ZWQudHh0
CShyZXZpc2lvbiA1Mzc1MikKKysrIExheW91dFRlc3RzL3BsYXRmb3JtL3dpbi9zdmcvY3VzdG9t
L3RleHQtem9vbS1leHBlY3RlZC50eHQJKHdvcmtpbmcgY29weSkKQEAgLTEsMTIgKzAsMCBAQAot
bGF5ZXIgYXQgKDAsMCkgc2l6ZSA4MDB4NjAwCi0gIFJlbmRlclZpZXcgYXQgKDAsMCkgc2l6ZSA4
MDB4NjAwCi1sYXllciBhdCAoMCwwKSBzaXplIDgwMHg5Ci0gIFJlbmRlckJsb2NrIHtodG1sfSBh
dCAoMCwwKSBzaXplIDgwMHg5Ci0gICAgUmVuZGVyQm9keSB7Ym9keX0gYXQgKDksOSkgc2l6ZSA3
ODJ4MAotbGF5ZXIgYXQgKDAsMzYpIHNpemUgMTIweDEyMAotICBSZW5kZXJCbG9jayAocG9zaXRp
b25lZCkge2Rpdn0gYXQgKDAsMzYpIHNpemUgMTIweDEyMCBbYmdjb2xvcj0jRkYwMDAwXQotbGF5
ZXIgYXQgKDAsMzYpIHNpemUgMjQweDI0NQotICBSZW5kZXJCbG9jayAocG9zaXRpb25lZCkge2Rp
dn0gYXQgKDAsMzYpIHNpemUgMjQweDI0MAotICAgIFJlbmRlclNWR1Jvb3Qge3N2Z30gYXQgKDks
NDUpIHNpemUgMTIweDEyMAotICAgICAgUmVuZGVyUGF0aCB7cmVjdH0gYXQgKDksNDUpIHNpemUg
MTIweDEyMCBbZmlsbD17W3R5cGU9U09MSURdIFtjb2xvcj0jMDA4MDAwXX1dIFtkYXRhPSJNMC4w
MCwwLjAwIEw1MC4wMCwwLjAwIEw1MC4wMCw1MC4wMCBMMC4wMCw1MC4wMCBaIl0KLSAgICBSZW5k
ZXJUZXh0IHsjdGV4dH0gYXQgKDAsMCkgc2l6ZSAweDAK
</data>
<flag name="review"
          id="29693"
          type_id="1"
          status="+"
          setter="mjs"
    />
          </attachment>
      

    </bug>

</bugzilla>