<?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>3825</bug_id>
          
          <creation_ts>2005-07-02 20:30:01 -0700</creation_ts>
          <short_desc>GW: need a KWQTimer with QTimer(parent) support</short_desc>
          <delta_ts>2005-07-03 16:30:49 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>New Bugs</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></keywords>
          <priority>P4</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>3250</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Eric Seidel (no email)">eric</reporter>
          <assigned_to name="Eric Seidel (no email)">eric</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>13722</commentid>
    <comment_count>0</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2005-07-02 20:30:02 -0700</bug_when>
    <thetext>This is a temporary solution to allow compiling.  Any code which actually depends on this method will log 
to the console.  Before shipping code depending on QTimer(parent) we would need to actually make these 
timers destruct when their parent objects do.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>13723</commentid>
    <comment_count>1</comment_count>
      <attachid>2764</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2005-07-02 20:31:57 -0700</bug_when>
    <thetext>Created attachment 2764
adds QTimer(parent) support</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>13781</commentid>
    <comment_count>2</comment_count>
      <attachid>2764</attachid>
    <who name="Maciej Stachowiak">mjs</who>
    <bug_when>2005-07-03 03:16:37 -0700</bug_when>
    <thetext>Comment on attachment 2764
adds QTimer(parent) support

r=me

*but*, the side effect could be a lot worse than a leak if this is really used
- it could lead to incorrect behavior or a crash if the timer fires after it&apos;s
expected to be gone.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>13834</commentid>
    <comment_count>3</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2005-07-03 16:30:49 -0700</bug_when>
    <thetext>landed with better log message:

if (parent) LOG(NotYetImplemented, &quot;Parent pointer ignored.  QTimer will be 
leaked and may fire after parent dealloc causing crash.&quot;);</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>2764</attachid>
            <date>2005-07-02 20:31:57 -0700</date>
            <delta_ts>2005-07-03 03:17:08 -0700</delta_ts>
            <desc>adds QTimer(parent) support</desc>
            <filename>qtimer.patch</filename>
            <type>text/plain</type>
            <size>1253</size>
            <attacher name="Eric Seidel (no email)">eric</attacher>
            
              <data encoding="base64">SW5kZXg6IC4va3dxL0tXUVRpbWVyLmgKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQpSQ1MgZmlsZTogL2N2cy9yb290L1dl
YkNvcmUva3dxL0tXUVRpbWVyLmgsdgpyZXRyaWV2aW5nIHJldmlzaW9uIDEuMTAKZGlmZiAtcCAt
dSAtcjEuMTAgLi9rd3EvS1dRVGltZXIuaAotLS0gLi9rd3EvS1dRVGltZXIuaAkyMDAzLzAxLzIy
IDAwOjEyOjM2CTEuMTAKKysrIC4va3dxL0tXUVRpbWVyLmgJMjAwNS8wNy8wMyAwMzowMDoxNwpA
QCAtMzcsNyArMzcsNyBAQCBjbGFzcyBOU1RpbWVyOwogCiBjbGFzcyBRVGltZXIgOiBwdWJsaWMg
UU9iamVjdCB7CiBwdWJsaWM6Ci0gICAgUVRpbWVyKCk7CisgICAgUVRpbWVyKFFPYmplY3QgKnBh
cmVudCA9IDApOwogICAgIH5RVGltZXIoKSB7IHN0b3AoKTsgfQogICAgIAogICAgIGJvb2wgaXNB
Y3RpdmUoKSBjb25zdDsKSW5kZXg6IC4va3dxL0tXUVRpbWVyLm1tCj09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KUkNTIGZp
bGU6IC9jdnMvcm9vdC9XZWJDb3JlL2t3cS9LV1FUaW1lci5tbSx2CnJldHJpZXZpbmcgcmV2aXNp
b24gMS4xNQpkaWZmIC1wIC11IC1yMS4xNSAuL2t3cS9LV1FUaW1lci5tbQotLS0gLi9rd3EvS1dR
VGltZXIubW0JMjAwNC8xMS8wOSAyMTo1MjoyMAkxLjE1CisrKyAuL2t3cS9LV1FUaW1lci5tbQky
MDA1LzA3LzAzIDAzOjAwOjE3CkBAIC0yNiw2ICsyNiw3IEBACiAjaW1wb3J0ICJLV1FUaW1lci5o
IgogCiAjaW1wb3J0ICJLV1FBc3NlcnRpb25zLmgiCisjaW1wb3J0ICJLV1FMb2dnaW5nLmgiCiAj
aW1wb3J0ICJLV1FGb3VuZGF0aW9uRXh0cmFzLmgiCiAKIC8vIFdlIGtub3cgdGhlIENvY29hIGNh
bGxzIGluIHRoaXMgZmlsZSBhcmUgc2FmZSBiZWNhdXNlIHRoZXkgYXJlIGFsbApAQCAtOTIsOSAr
OTMsMTAgQEAKIAogQGVuZAogCi1RVGltZXI6OlFUaW1lcigpCitRVGltZXI6OlFUaW1lcihRT2Jq
ZWN0ICpwYXJlbnQpCiAgICAgOiBtX3RpbWVyKG5pbCksIG1fbW9uaXRvckZ1bmN0aW9uKDApLCBt
X3RpbWVvdXRTaWduYWwodGhpcywgU0lHTkFMKHRpbWVvdXQoKSkpCiB7CisgICAgaWYgKHBhcmVu
dCkgTE9HKE5vdFlldEltcGxlbWVudGVkLCAiUVRpbWVyIHdpbGwgYmUgbGVha2VkLCBwYXJlbnQg
cG9pbnRlciBpZ25vcmVkLiIpOwogfQogCiBib29sIFFUaW1lcjo6aXNBY3RpdmUoKSBjb25zdAo=
</data>
<flag name="review"
          id="150"
          type_id="1"
          status="+"
          setter="mjs"
    />
          </attachment>
      

    </bug>

</bugzilla>