<?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>12016</bug_id>
          
          <creation_ts>2006-12-28 12:16:38 -0800</creation_ts>
          <short_desc>REGRESSION(r18342): fast/text/stroking-decorations.html test fails with bus error in no-svg build</short_desc>
          <delta_ts>2007-01-02 15:24:00 -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>420+</version>
          <rep_platform>Mac</rep_platform>
          <op_sys>OS X 10.4</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar, LayoutTestFailure, Regression</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="David Kilzer (:ddkilzer)">ddkilzer</reporter>
          <assigned_to name="David Kilzer (:ddkilzer)">ddkilzer</assigned_to>
          <cc>hyatt</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>39219</commentid>
    <comment_count>0</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2006-12-28 12:16:38 -0800</bug_when>
    <thetext>The fast/text/stroking-decorations.html test failes with a Bus Error in the no-svg build.  Stack trace:

Thread 0 Crashed:
0   libSystem.B.dylib           0x9000af30 __vfprintf + 479
1   libSystem.B.dylib           0x9000ad40 sprintf + 480
2   com.apple.WebCore           0x0100e850 WebCore::TextStream::operator&lt;&lt;(float) + 56 (TextStream.cpp:102)
3   com.apple.WebCore           0x01031921 WebCore::operator&lt;&lt;(WebCore::TextStream&amp;, WebCore::RenderObject const&amp;) + 211
7 (RenderTreeAsText.cpp:170)
4   com.apple.WebCore           0x010326dc WebCore::write(WebCore::TextStream&amp;, WebCore::RenderObject const&amp;, int) + 36 
(RenderTreeAsText.cpp:299)
5   com.apple.WebCore           0x010327c3 WebCore::write(WebCore::TextStream&amp;, WebCore::RenderObject const&amp;, int) + 267
 (RenderTreeAsText.cpp:309)
6   com.apple.WebCore           0x010327c3 WebCore::write(WebCore::TextStream&amp;, WebCore::RenderObject const&amp;, int) + 267
 (RenderTreeAsText.cpp:309)
7   com.apple.WebCore           0x01032bbc WebCore::write(WebCore::TextStream&amp;, WebCore::RenderLayer&amp;, WebCore::IntRect 
const&amp;, WebCore::IntRect const&amp;, WebCore::IntRect const&amp;, WebCore::IntRect const&amp;, int, int) + 694 (RenderTreeAsText.cpp
:367)
8   com.apple.WebCore           0x010325e1 WebCore::writeLayers(WebCore::TextStream&amp;, WebCore::RenderLayer const*, WebCo
re::RenderLayer*, WebCore::IntRect const&amp;, int) + 459 (RenderTreeAsText.cpp:393)
9   com.apple.WebCore           0x01032692 WebCore::writeLayers(WebCore::TextStream&amp;, WebCore::RenderLayer const*, WebCo
re::RenderLayer*, WebCore::IntRect const&amp;, int) + 636 (RenderTreeAsText.cpp:401)
10  com.apple.WebCore           0x01032caf WebCore::externalRepresentation(WebCore::RenderObject*) + 235 (RenderTreeAsTe
xt.cpp:463)
11  com.apple.WebCore           0x0107fcae -[WebCoreFrameBridge renderTreeAsExternalRepresentation] + 36 (WebCoreFrameBr
idge.mm:894)
12  com.apple.WebKit            0x0020be99 -[WebFrame(WebKitDebug) renderTreeAsExternalRepresentation] + 57
13  DumpRenderTree              0x00007903 dump + 828 (DumpRenderTree.m:546)
14  DumpRenderTree              0x0000891e -[WaitUntilDoneDelegate webView:locationChangeDone:forDataSource:] + 148 (Dum
pRenderTree.m:719)
15  DumpRenderTree              0x00008aa1 -[WaitUntilDoneDelegate webView:didFinishLoadForFrame:] + 67 (DumpRenderTree.
m:746)
16  libobjc.A.dylib             0x90a59d76 objc_msgSendv + 54
17  com.apple.Foundation        0x925ff43e -[NSInvocation invoke] + 932
18  com.apple.Foundation        0x92625433 -[NSInvocation invokeWithTarget:] + 67
19  com.apple.WebKit            0x00268732 -[_WebSafeForwarder forwardInvocation:] + 448 (WebView.mm:1660)
20  com.apple.Foundation        0x925fe4f4 -[NSObject(NSForwardInvocation) forward::] + 469
21  libobjc.A.dylib             0x90a59cc1 _objc_msgForward + 49
22  com.apple.WebKit            0x0029bc9d WebFrameLoaderClient::dispatchDidFinishLoad() + 153 (WebFrameLoaderClient.mm:
630)
23  com.apple.WebCore           0x012891e8 WebCore::FrameLoader::checkLoadCompleteForThisFrame() + 1080 (FrameLoaderMac.
mm:1043)
24  com.apple.WebCore           0x0129dd8c WebCore::FrameLoader::checkLoadComplete() + 144 (FrameLoader.cpp:2124)
25  com.apple.WebCore           0x0129e445 WebCore::FrameLoader::finishedLoading() + 227 (FrameLoader.cpp:1956)
26  com.apple.WebCore           0x0128f7c5 WebCore::MainResourceLoader::didFinishLoading() + 211 (MainResourceLoaderMac.
mm:324)
27  com.apple.WebCore           0x0128d858 WebCore::ResourceLoader::didFinishLoading(WebCore::ResourceHandle*) + 24 (Res
ourceLoaderMac.mm:447)
28  com.apple.WebCore           0x0129944d -[WebCoreResourceHandleAsDelegate connectionDidFinishLoading:] + 79 (Resource
HandleMac.mm:296)
29  com.apple.Foundation        0x9265ee00 -[NSURLConnection(NSURLConnectionInternal) _sendDidFinishLoadingCallback] + 1
76
30  com.apple.Foundation        0x9265cea5 -[NSURLConnection(NSURLConnectionInternal) _sendCallbacks] + 748
31  com.apple.Foundation        0x9265cb41 _sendCallbacks + 201
32  com.apple.CoreFoundation    0x9082afd2 CFRunLoopRunSpecific + 1213
33  com.apple.CoreFoundation    0x9082ab0e CFRunLoopRunInMode + 61
34  com.apple.Foundation        0x9262ddc6 -[NSRunLoop runMode:beforeDate:] + 182
35  DumpRenderTree              0x00009dfe runTest + 950 (DumpRenderTree.m:1086)
36  DumpRenderTree              0x00006da9 dumpRenderTree + 3355 (DumpRenderTree.m:400)
37  DumpRenderTree              0x00006fb0 main + 70 (DumpRenderTree.m:450)
38  DumpRenderTree              0x00002da2 _start + 216
39  DumpRenderTree              0x00002cc9 start + 41</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>39199</commentid>
    <comment_count>1</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2006-12-28 12:19:17 -0800</bug_when>
    <thetext>Considering this a regression since DRT didn&apos;t crash before r18342 when LayoutTests/fast/text/stroking-decorations.html was added.

</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>39190</commentid>
    <comment_count>2</comment_count>
      <attachid>12086</attachid>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2006-12-28 13:27:30 -0800</bug_when>
    <thetext>Created attachment 12086
Patch v1

Fixes incorrect initialization of precisionFormats array (missing comma caused pre-processor concatenation of last two strings) in TextStream.cpp.

Fixes RenderTreeAsText::externalRepresentation() to set the precision of the TextStream regardless of SVG support.  This allows fast/text/stroking-decorations.html and fast/text/stroking.html to pass on the no-svg build.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>39191</commentid>
    <comment_count>3</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2006-12-28 13:41:02 -0800</bug_when>
    <thetext>(In reply to comment #2)
&gt; Created an attachment (id=12086) [edit]
&gt; Patch v1

I ran all tests in both the svg and no-svg builds, and there are no regressions caused by these changes (only fixes to the stroking tests for the no-svg build).

</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>39149</commentid>
    <comment_count>4</comment_count>
      <attachid>12086</attachid>
    <who name="">mitz</who>
    <bug_when>2006-12-28 14:28:52 -0800</bug_when>
    <thetext>Comment on attachment 12086
Patch v1

Missing space:
+        - fixhttp://bugs.webkit.org/show_bug.cgi?id=12016

&quot;Make fast/text/stroking-decorations.html and fast/text/stroking.html pass on no-svg build&quot; just repeats the bug&apos;s summary, adding the name of another failing test. It&apos;s redundant. You could explain the change (HTML tests now print out the stroke width, which is a float, so the precision needs to be set) or just not write anything.

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>39157</commentid>
    <comment_count>5</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2006-12-28 15:08:41 -0800</bug_when>
    <thetext>Committed revision 18459.

</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>37613</commentid>
    <comment_count>6</comment_count>
    <who name="Darin Adler">darin</who>
    <bug_when>2007-01-02 15:24:00 -0800</bug_when>
    <thetext>&lt;rdar://problem/4897040&gt; no-svg Buildbot crash at WebCore::TextStream::operator&lt;&lt;(float) + 72</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>12086</attachid>
            <date>2006-12-28 13:27:30 -0800</date>
            <delta_ts>2006-12-28 14:28:52 -0800</delta_ts>
            <desc>Patch v1</desc>
            <filename>bug-12016-v1.diff</filename>
            <type>text/plain</type>
            <size>2134</size>
            <attacher name="David Kilzer (:ddkilzer)">ddkilzer</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiAxODQ1NykKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTcgQEAKKzIwMDYtMTItMjggIERhdmlkIEtpbHplciAgPHNldCBFTUFJTF9BRERS
RVNTIGVudmlyb25tZW50IHZhcmlhYmxlPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAo
T09QUyEpLgorCisgICAgICAgIC0gZml4aHR0cDovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5j
Z2k/aWQ9MTIwMTYKKyAgICAgICAgICBSRUdSRVNTSU9OOiBmYXN0L3RleHQvc3Ryb2tpbmctZGVj
b3JhdGlvbnMuaHRtbCB0ZXN0IGZhaWxzIHdpdGggYnVzIGVycm9yIGluIG5vLXN2ZyBidWlsZAor
CisgICAgICAgIE5vIHRlc3QgY2FzZXMgKG5vIGNoYW5nZSBpbiBmdW5jdGlvbmFsaXR5KS4KKwor
ICAgICAgICAqIHBsYXRmb3JtL1RleHRTdHJlYW0uY3BwOiBQcm9wZXJseSBpbml0aWFsaXplIHBy
ZXNpY2lvbkZvcm1hdHMuCisgICAgICAgICogcmVuZGVyaW5nL1JlbmRlclRyZWVBc1RleHQuY3Bw
OgorICAgICAgICAoV2ViQ29yZTo6ZXh0ZXJuYWxSZXByZXNlbnRhdGlvbik6IE1ha2UgZmFzdC90
ZXh0L3N0cm9raW5nLWRlY29yYXRpb25zLmh0bWwgYW5kCisgICAgICAgIGZhc3QvdGV4dC9zdHJv
a2luZy5odG1sIHBhc3Mgb24gbm8tc3ZnIGJ1aWxkLgorCiAyMDA2LTEyLTI4ICBNaXR6IFBldHRl
bCAgPG1pdHpAd2Via2l0Lm9yZz4KIAogICAgICAgICBSZXZpZXdlZCBieSBEYXJpbi4KSW5kZXg6
IFdlYkNvcmUvcGxhdGZvcm0vVGV4dFN0cmVhbS5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2ViQ29yZS9w
bGF0Zm9ybS9UZXh0U3RyZWFtLmNwcAkocmV2aXNpb24gMTg0NTYpCisrKyBXZWJDb3JlL3BsYXRm
b3JtL1RleHRTdHJlYW0uY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0zNCw3ICszNCw3IEBACiBuYW1l
c3BhY2UgV2ViQ29yZSB7CiAKIGNvbnN0IHNpemVfdCBpbnRlZ2VyT3JQb2ludGVyQXNTdHJpbmdC
dWZmZXJTaXplID0gMTAwOyAvLyBsYXJnZSBlbm91Z2ggZm9yIGFueSBpbnRlZ2VyIG9yIHBvaW50
ZXIgaW4gc3RyaW5nIGZvcm1hdCwgaW5jbHVkaW5nIHRyYWlsaW5nIG51bGwgY2hhcmFjdGVyCi1j
b25zdCBjaGFyKiBjb25zdCBwcmVjaXNpb25Gb3JtYXRzWzZdID0geyAiJS4wZiIsICIlLjFmIiwg
IiUuMmYiLCAiJS4zZiIsICIlLjRmIiwgIiUuNWYiICIlLjZmIn07IAorY29uc3QgY2hhciogY29u
c3QgcHJlY2lzaW9uRm9ybWF0c1s3XSA9IHsgIiUuMGYiLCAiJS4xZiIsICIlLjJmIiwgIiUuM2Yi
LCAiJS40ZiIsICIlLjVmIiwgIiUuNmYifTsgCiBjb25zdCBpbnQgbWF4UHJlY2lzaW9uID0gNjsg
Ly8gbXVzdCBtYXRjaCBzaXplIG9mIHByZWNpc2lvbkZvcm1hdHMKIGNvbnN0IGludCBkZWZhdWx0
UHJlY2lzaW9uID0gNjsgLy8gbWF0Y2hlcyBxdCBhbmQgc3ByaW50ZiguLiwgIiVmIiwgLi4uKSBi
ZWhhdmlvdXIKIApJbmRleDogV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyVHJlZUFzVGV4dC5jcHAK
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PQotLS0gV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyVHJlZUFzVGV4dC5jcHAJKHJl
dmlzaW9uIDE4NDU2KQorKysgV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyVHJlZUFzVGV4dC5jcHAJ
KHdvcmtpbmcgY29weSkKQEAgLTQ1Miw4ICs0NTIsOCBAQCBEZXByZWNhdGVkU3RyaW5nIGV4dGVy
bmFsUmVwcmVzZW50YXRpb24oCiAgICAgRGVwcmVjYXRlZFN0cmluZyBzOwogICAgIGlmIChvKSB7
CiAgICAgICAgIFRleHRTdHJlYW0gdHMoJnMpOwotI2lmZGVmIFNWR19TVVBQT1JUCiAgICAgICAg
IHRzLnByZWNpc2lvbigyKTsKKyNpZmRlZiBTVkdfU1VQUE9SVAogICAgICAgICB3cml0ZVJlbmRl
clJlc291cmNlcyh0cywgby0+ZG9jdW1lbnQoKSk7CiAjZW5kaWYKICAgICAgICAgby0+dmlldygp
LT5mcmFtZVZpZXcoKS0+bGF5b3V0KCk7Cg==
</data>
<flag name="review"
          id="4455"
          type_id="1"
          status="+"
          setter="mitz"
    />
          </attachment>
      

    </bug>

</bugzilla>