<?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>27512</bug_id>
          
          <creation_ts>2009-07-21 13:18:27 -0700</creation_ts>
          <short_desc>Test for QWebFrame::hasSetFocus() is trying to use a not defined resource.</short_desc>
          <delta_ts>2009-07-28 06:04:51 -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>WebKit Qt</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Andre Pedralho">apedralho</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>hausmann</cc>
    
    <cc>tonikitoo</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>133573</commentid>
    <comment_count>0</comment_count>
    <who name="Andre Pedralho">apedralho</who>
    <bug_when>2009-07-21 13:18:27 -0700</bug_when>
    <thetext>There is no frametest folder and qwebframe.qrc does not specify where to find it.

tst_qwebframe.cpp:
void tst_QWebFrame::hasSetFocus()
{
(...)
   QUrl url = QUrl(&quot;qrc:///frametest/iframe.html&quot;);
   _page-&gt;mainFrame()-&gt;load(url);
(...)
}

qwebframe.qrc:
&lt;!DOCTYPE RCC&gt;&lt;RCC version=&quot;1.0&quot;&gt;
&lt;qresource prefix=&quot;/&quot;&gt;
&lt;file&gt;image.png&lt;/file&gt;
&lt;file&gt;style.css&lt;/file&gt;
&lt;/qresource&gt;
&lt;/RCC&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>133581</commentid>
    <comment_count>1</comment_count>
      <attachid>33207</attachid>
    <who name="Andre Pedralho">apedralho</who>
    <bug_when>2009-07-21 13:38:14 -0700</bug_when>
    <thetext>Created attachment 33207
Copied resources from qwebpage tests and added the references to the qwebframe.qrc.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>133585</commentid>
    <comment_count>2</comment_count>
    <who name="Antonio Gomes">tonikitoo</who>
    <bug_when>2009-07-21 13:51:37 -0700</bug_when>
    <thetext>we are duping the resources used in qwebframe and qwebpage. should not we make them to share the resource ?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>133759</commentid>
    <comment_count>3</comment_count>
      <attachid>33207</attachid>
    <who name="Adam Treat">manyoso</who>
    <bug_when>2009-07-22 06:35:39 -0700</bug_when>
    <thetext>Comment on attachment 33207
Copied resources from qwebpage tests and added the references to the qwebframe.qrc.

As tonikitoo pointed out, we shouldn&apos;t dupe the resources, but rather share them.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>133809</commentid>
    <comment_count>4</comment_count>
    <who name="Andre Pedralho">apedralho</who>
    <bug_when>2009-07-22 11:06:42 -0700</bug_when>
    <thetext>(In reply to comment #3)
&gt; (From update of attachment 33207 [details])
&gt; As tonikitoo pointed out, we shouldn&apos;t dupe the resources, but rather share
&gt; them.

Agree with that, however &quot;the listed resource files must be located in the same directory as the .qrc file, or one of its subdirectories&quot; according to the Qt  resources docs.

As there are two different directories to QWebFrame and to QWebPage tests the only way to share the resources would be to have a pool of resources in WebKit/qt/tests/&apos;resources&apos;/ with the qrc files related to both tests.

I mean:

ls WebKit/qt/tests/:
benchmarks  qwebelement  qwebframe (...) qwebpage  qwebview  RESOURCES tests.pro

ls WebKit/qt/tests/resources:
FRAMETEST image.png qwebframe.qrc style.css tst_qwebpage.qrc

IMHO, the best solution here is to create separated resources to each tests, maybe duplicating it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>134086</commentid>
    <comment_count>5</comment_count>
    <who name="Andre Pedralho">apedralho</who>
    <bug_when>2009-07-23 06:06:21 -0700</bug_when>
    <thetext>Hausmann, we need your opinion to find the best solution here. Can you help us?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>134225</commentid>
    <comment_count>6</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2009-07-23 13:26:01 -0700</bug_when>
    <thetext>Hmmm, another option, potentially simpler, would be to replace the use of the resources for html with data urls. What do you think?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>134229</commentid>
    <comment_count>7</comment_count>
    <who name="Antonio Gomes">tonikitoo</who>
    <bug_when>2009-07-23 13:30:47 -0700</bug_when>
    <thetext>simon, he has done so for this renderContents removal patch (not uploaded yet) ... and it looks pretty nice. and it would work here too, imho

ps: we should &quot;invalid&quot; this one instead of mark as &quot;fixed&quot; ?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>134552</commentid>
    <comment_count>8</comment_count>
    <who name="Andre Pedralho">apedralho</who>
    <bug_when>2009-07-24 07:46:45 -0700</bug_when>
    <thetext>Reopening as the bug itself still there and might be fixed with the solution proposed in comment #6.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>134553</commentid>
    <comment_count>9</comment_count>
      <attachid>33445</attachid>
    <who name="Andre Pedralho">apedralho</who>
    <bug_when>2009-07-24 07:48:24 -0700</bug_when>
    <thetext>Created attachment 33445
Using setHtml methods to define the test web page instead of external resources.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>134968</commentid>
    <comment_count>10</comment_count>
      <attachid>33560</attachid>
    <who name="Andre Pedralho">apedralho</who>
    <bug_when>2009-07-27 11:39:18 -0700</bug_when>
    <thetext>Created attachment 33560
Using QSignalSpy to control whether a load finished signal has been emited instead of just wait for some miliseconds.

Marking old patch obsolete as per suggestions in IRC.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>135171</commentid>
    <comment_count>11</comment_count>
      <attachid>33560</attachid>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2009-07-28 06:01:54 -0700</bug_when>
    <thetext>Comment on attachment 33560
Using QSignalSpy to control whether a load finished signal has been emited instead of just wait for some miliseconds.

r=me. Thanks</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>135172</commentid>
    <comment_count>12</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2009-07-28 06:04:51 -0700</bug_when>
    <thetext>Landed in r46475</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>33207</attachid>
            <date>2009-07-21 13:38:14 -0700</date>
            <delta_ts>2009-07-24 07:48:24 -0700</delta_ts>
            <desc>Copied resources from qwebpage tests and added the references to the qwebframe.qrc.</desc>
            <filename>frametest.diff</filename>
            <type>text/plain</type>
            <size>2370</size>
            <attacher name="Andre Pedralho">apedralho</attacher>
            
              <data encoding="base64">Y29tbWl0IDI1ZTc3M2JkMzY3MWIxYTBjOTU2ZmFhMjE4NDQ2NmE5OWVhNjdkMmUKQXV0aG9yOiBB
bmRyZSBQZWRyYWxobyA8YW5kcmUucGVkcmFsaG9Ab3BlbmJvc3NhLm9yZz4KRGF0ZTogICBUdWUg
SnVsIDIxIDE2OjI2OjM5IDIwMDkgLTA0MDAKCiAgICAyMDA5LTA3LTIxICBBbmRyZSBQZWRyYWxo
byAgPGFuZHJlLnBlZHJhbGhvQG9wZW5ib3NzYS5vcmc+CiAgICAKICAgICAgICAgICAgIFJldmll
d2VkIGJ5IE5PQk9EWSAoT09QUyEpLgogICAgCiAgICAgICAgICAgICBBZGRlZCBtaXNzaW5nIHJl
c291cmNlcyB0byB0aGUgdGVzdHMgZm9yIFFXZWJGcmFtZTo6aGFzU2V0Rm9jdXMoKS4KICAgIAog
ICAgICAgICAgICAgSXQgc29sdmVzIGJ1ZyAjMjc1MTIuCiAgICAKICAgICAgICAgICAgICogdGVz
dHMvcXdlYmZyYW1lL3F3ZWJmcmFtZS5xcmM6CgpkaWZmIC0tZ2l0IGEvV2ViS2l0L3F0L0NoYW5n
ZUxvZyBiL1dlYktpdC9xdC9DaGFuZ2VMb2cKaW5kZXggZmUwYjQ4YS4uNjlmOWZkOSAxMDA2NDQK
LS0tIGEvV2ViS2l0L3F0L0NoYW5nZUxvZworKysgYi9XZWJLaXQvcXQvQ2hhbmdlTG9nCkBAIC0x
LDMgKzEsMTMgQEAKKzIwMDktMDctMjEgIEFuZHJlIFBlZHJhbGhvICA8YW5kcmUucGVkcmFsaG9A
b3BlbmJvc3NhLm9yZz4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKwor
ICAgICAgICBBZGRlZCBtaXNzaW5nIHJlc291cmNlcyB0byB0aGUgdGVzdHMgZm9yIFFXZWJGcmFt
ZTo6aGFzU2V0Rm9jdXMoKS4KKworICAgICAgICBJdCBzb2x2ZXMgYnVnICMyNzUxMi4KKworICAg
ICAgICAqIHRlc3RzL3F3ZWJmcmFtZS9xd2ViZnJhbWUucXJjOgorCiAyMDA5LTA3LTIxICBWb2xr
ZXIgSGlsc2hlaW1lciA8dm9sa2VyLmhpbHNoZWltZXJAbm9raWEuY29tPgogCiAgICAgICAgIFJl
dmlld2VkIGJ5IFNpbW9uIEhhdXNtYW5uLgpkaWZmIC0tZ2l0IGEvV2ViS2l0L3F0L3Rlc3RzL3F3
ZWJmcmFtZS9mcmFtZXRlc3QvaWZyYW1lLmh0bWwgYi9XZWJLaXQvcXQvdGVzdHMvcXdlYmZyYW1l
L2ZyYW1ldGVzdC9pZnJhbWUuaHRtbApuZXcgZmlsZSBtb2RlIDEwMDY0NAppbmRleCAwMDAwMDAw
Li5mMTcwMjdjCi0tLSAvZGV2L251bGwKKysrIGIvV2ViS2l0L3F0L3Rlc3RzL3F3ZWJmcmFtZS9m
cmFtZXRlc3QvaWZyYW1lLmh0bWwKQEAgLTAsMCArMSw2IEBACis8aHRtbD4KKzxib2R5PgorPHA+
dG9wPC9wPgorPGlmcmFtZSBzcmM9ImlmcmFtZTIuaHRtbCIgd2lkdGg9IjgwJSIgaGVpZ2h0PSIz
MCUiLz4KKzwvYm9keT4KKzwvaHRtbD4KZGlmZiAtLWdpdCBhL1dlYktpdC9xdC90ZXN0cy9xd2Vi
ZnJhbWUvZnJhbWV0ZXN0L2lmcmFtZTIuaHRtbCBiL1dlYktpdC9xdC90ZXN0cy9xd2ViZnJhbWUv
ZnJhbWV0ZXN0L2lmcmFtZTIuaHRtbApuZXcgZmlsZSBtb2RlIDEwMDY0NAppbmRleCAwMDAwMDAw
Li41MDE3NDM1Ci0tLSAvZGV2L251bGwKKysrIGIvV2ViS2l0L3F0L3Rlc3RzL3F3ZWJmcmFtZS9m
cmFtZXRlc3QvaWZyYW1lMi5odG1sCkBAIC0wLDAgKzEsNyBAQAorPGh0bWw+Cis8Ym9keT4KKzxw
PmFub3RoZXIgaWZyYW1lPC9wPgorPGlmcmFtZSBzcmM9ImlmcmFtZTMuaHRtbCIgd2lkdGg9Ijgw
JSIgaGVpZ2h0PSIzMCUiPjwvaWZyYW1lPgorPC9ib2R5PgorPC9odG1sPgorCmRpZmYgLS1naXQg
YS9XZWJLaXQvcXQvdGVzdHMvcXdlYmZyYW1lL2ZyYW1ldGVzdC9pZnJhbWUzLmh0bWwgYi9XZWJL
aXQvcXQvdGVzdHMvcXdlYmZyYW1lL2ZyYW1ldGVzdC9pZnJhbWUzLmh0bWwKbmV3IGZpbGUgbW9k
ZSAxMDA2NDQKaW5kZXggMDAwMDAwMC4uZWQ2YWM1YgotLS0gL2Rldi9udWxsCisrKyBiL1dlYktp
dC9xdC90ZXN0cy9xd2ViZnJhbWUvZnJhbWV0ZXN0L2lmcmFtZTMuaHRtbApAQCAtMCwwICsxLDUg
QEAKKzxodG1sPgorPGJvZHk+Cis8cD5pbm5lcjwvcD4KKzwvYm9keT4KKzwvaHRtbD4KZGlmZiAt
LWdpdCBhL1dlYktpdC9xdC90ZXN0cy9xd2ViZnJhbWUvcXdlYmZyYW1lLnFyYyBiL1dlYktpdC9x
dC90ZXN0cy9xd2ViZnJhbWUvcXdlYmZyYW1lLnFyYwppbmRleCAyNjZjZGNlLi4yODVmMDU2IDEw
MDY0NAotLS0gYS9XZWJLaXQvcXQvdGVzdHMvcXdlYmZyYW1lL3F3ZWJmcmFtZS5xcmMKKysrIGIv
V2ViS2l0L3F0L3Rlc3RzL3F3ZWJmcmFtZS9xd2ViZnJhbWUucXJjCkBAIC0yLDUgKzIsOCBAQAog
PHFyZXNvdXJjZSBwcmVmaXg9Ii8iPgogPGZpbGU+aW1hZ2UucG5nPC9maWxlPgogPGZpbGU+c3R5
bGUuY3NzPC9maWxlPgorPGZpbGU+ZnJhbWV0ZXN0L2lmcmFtZS5odG1sPC9maWxlPgorPGZpbGU+
ZnJhbWV0ZXN0L2lmcmFtZTIuaHRtbDwvZmlsZT4KKzxmaWxlPmZyYW1ldGVzdC9pZnJhbWUzLmh0
bWw8L2ZpbGU+CiA8L3FyZXNvdXJjZT4KIDwvUkNDPgoK
</data>
<flag name="review"
          id="17522"
          type_id="1"
          status="-"
          setter="manyoso"
    />
          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>33445</attachid>
            <date>2009-07-24 07:48:24 -0700</date>
            <delta_ts>2009-07-27 11:39:18 -0700</delta_ts>
            <desc>Using setHtml methods to define the test web page instead of external resources.</desc>
            <filename>fixed_hasSetFocus.diff</filename>
            <type>text/plain</type>
            <size>2762</size>
            <attacher name="Andre Pedralho">apedralho</attacher>
            
              <data encoding="base64">Y29tbWl0IDhjMDk5ZDUxZTAzMTc4Nzg5MmQ4MjNjNmRlY2YzNWQ4NzFlNDA1ZWQKQXV0aG9yOiBB
bmRyZSBQZWRyYWxobyA8YW5kcmUucGVkcmFsaG9Ab3BlbmJvc3NhLm9yZz4KRGF0ZTogICBGcmkg
SnVsIDI0IDEwOjQ0OjUzIDIwMDkgLTA0MDAKCiAgICAyMDA5LTA3LTI0ICBBbmRyZSBQZWRyYWxo
byAgPGFuZHJlLnBlZHJhbGhvQG9wZW5ib3NzYS5vcmc+CiAgICAKICAgICAgICAgICAgUmV2aWV3
ZWQgYnkgTk9CT0RZIChPT1BTISkuCiAgICAKICAgICAgICAgICAgRml4ZWQgdHN0X1FXZWJGcmFt
ZTo6aGFzU2V0Rm9jdXMgdGVzdCB3aGljaCB3YXMgdXNpbmcKICAgICAgICAgICAgYW4gdW5kZWZp
bmVkIHJlc291cmNlLgogICAgICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1
Zy5jZ2k/aWQ9Mjc1MTIKICAgIAogICAgICAgICAgICAqIHRlc3RzL3F3ZWJmcmFtZS90c3RfcXdl
YmZyYW1lLmNwcDoKCmRpZmYgLS1naXQgYS9XZWJLaXQvcXQvQ2hhbmdlTG9nIGIvV2ViS2l0L3F0
L0NoYW5nZUxvZwppbmRleCA0MmU0MzhkLi5mMWUxNWQwIDEwMDY0NAotLS0gYS9XZWJLaXQvcXQv
Q2hhbmdlTG9nCisrKyBiL1dlYktpdC9xdC9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxMyBAQAorMjAw
OS0wNy0yNCAgQW5kcmUgUGVkcmFsaG8gIDxhbmRyZS5wZWRyYWxob0BvcGVuYm9zc2Eub3JnPgor
CisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIEZpeGVkIHRz
dF9RV2ViRnJhbWU6Omhhc1NldEZvY3VzIHRlc3Qgd2hpY2ggd2FzIHVzaW5nCisgICAgICAgIGFu
IHVuZGVmaW5lZCByZXNvdXJjZS4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hv
d19idWcuY2dpP2lkPTI3NTEyCisKKyAgICAgICAgKiB0ZXN0cy9xd2ViZnJhbWUvdHN0X3F3ZWJm
cmFtZS5jcHA6CisKIDIwMDktMDctMjQgIEtlbm5ldGggUm9oZGUgQ2hyaXN0aWFuc2VuICA8a2Vu
bmV0aEB3ZWJraXQub3JnPgogCiAgICAgICAgIEJ1aWxkIGZpeCBmb3IgUXQuCmRpZmYgLS1naXQg
YS9XZWJLaXQvcXQvdGVzdHMvcXdlYmZyYW1lL3RzdF9xd2ViZnJhbWUuY3BwIGIvV2ViS2l0L3F0
L3Rlc3RzL3F3ZWJmcmFtZS90c3RfcXdlYmZyYW1lLmNwcAppbmRleCBjMGU3MmJlLi4zOGM4MTFm
IDEwMDY0NAotLS0gYS9XZWJLaXQvcXQvdGVzdHMvcXdlYmZyYW1lL3RzdF9xd2ViZnJhbWUuY3Bw
CisrKyBiL1dlYktpdC9xdC90ZXN0cy9xd2ViZnJhbWUvdHN0X3F3ZWJmcmFtZS5jcHAKQEAgLTI0
NTMsMjQgKzI0NTMsMzIgQEAgdm9pZCB0c3RfUVdlYkZyYW1lOjpiYXNlVXJsKCkKIAogdm9pZCB0
c3RfUVdlYkZyYW1lOjpoYXNTZXRGb2N1cygpCiB7Ci0gICAgUVNpZ25hbFNweSBsb2FkU3B5KG1f
cGFnZSwgU0lHTkFMKGxvYWRGaW5pc2hlZChib29sKSkpOwotICAgIFFVcmwgdXJsID0gUVVybCgi
cXJjOi8vL2ZyYW1ldGVzdC9pZnJhbWUuaHRtbCIpOwotICAgIG1fcGFnZS0+bWFpbkZyYW1lKCkt
PmxvYWQodXJsKTsKKyAgICBRU3RyaW5nIGh0bWwoIjxodG1sPjxib2R5PjxwPnRvcDwvcD4iIFwK
KyAgICAgICAgICAgICAgICAgICAgIjxpZnJhbWUgd2lkdGg9JzgwJScgaGVpZ2h0PSczMCUnLz4i
IFwKKyAgICAgICAgICAgICAgICAgIjwvYm9keT48L2h0bWw+Iik7CisgICAgUVdlYlBhZ2UgcGFn
ZTsKKyAgICBwYWdlLm1haW5GcmFtZSgpLT5zZXRIdG1sKGh0bWwpOworCisgICAgUUxpc3Q8UVdl
YkZyYW1lKj4gY2hpbGRyZW4gPSBwYWdlLm1haW5GcmFtZSgpLT5jaGlsZEZyYW1lcygpOworICAg
IFFXZWJGcmFtZSAqZnJhbWUgPSBjaGlsZHJlbi5hdCgwKTsKKyAgICBRU3RyaW5nIGlubmVySHRt
bCgiPGh0bWw+PGJvZHk+PHA+YW5vdGhlciBpZnJhbWU8L3A+IiBcCisgICAgICAgICAgICAgICAg
ICAgICAgICAiPGlmcmFtZSB3aWR0aD0nODAlJyBoZWlnaHQ9JzMwJScvPiIgXAorICAgICAgICAg
ICAgICAgICAgICAgICI8L2JvZHk+PC9odG1sPiIpOworICAgIGZyYW1lLT5zZXRIdG1sKGlubmVy
SHRtbCk7CiAKLSAgICA6OndhaXRGb3JTaWduYWwobV9wYWdlLCBTSUdOQUwobG9hZEZpbmlzaGVk
KGJvb2wpKSk7CisgICAgUVRlc3Q6OnFXYWl0KDIwMCk7CiAKLSAgICBtX3BhZ2UtPm1haW5GcmFt
ZSgpLT5zZXRGb2N1cygpOwotICAgIFFWRVJJRlkobV9wYWdlLT5tYWluRnJhbWUoKS0+aGFzRm9j
dXMoKSk7CisgICAgcGFnZS5tYWluRnJhbWUoKS0+c2V0Rm9jdXMoKTsKKyAgICBRVkVSSUZZKHBh
Z2UubWFpbkZyYW1lKCktPmhhc0ZvY3VzKCkpOwogCi0gICAgUUxpc3Q8UVdlYkZyYW1lKj4gY2hp
bGRyZW4gPSBtX3BhZ2UtPm1haW5GcmFtZSgpLT5jaGlsZEZyYW1lcygpOwogICAgIGZvciAoaW50
IGkgPSAwOyBpIDwgY2hpbGRyZW4uc2l6ZSgpOyArK2kpIHsKICAgICAgICAgY2hpbGRyZW4uYXQo
aSktPnNldEZvY3VzKCk7CiAgICAgICAgIFFWRVJJRlkoY2hpbGRyZW4uYXQoaSktPmhhc0ZvY3Vz
KCkpOwotICAgICAgICBRVkVSSUZZKCFtX3BhZ2UtPm1haW5GcmFtZSgpLT5oYXNGb2N1cygpKTsK
KyAgICAgICAgUVZFUklGWSghcGFnZS5tYWluRnJhbWUoKS0+aGFzRm9jdXMoKSk7CiAgICAgfQog
Ci0gICAgbV9wYWdlLT5tYWluRnJhbWUoKS0+c2V0Rm9jdXMoKTsKLSAgICBRVkVSSUZZKG1fcGFn
ZS0+bWFpbkZyYW1lKCktPmhhc0ZvY3VzKCkpOworICAgIHBhZ2UubWFpbkZyYW1lKCktPnNldEZv
Y3VzKCk7CisgICAgUVZFUklGWShwYWdlLm1haW5GcmFtZSgpLT5oYXNGb2N1cygpKTsKIH0KIAog
UVRFU1RfTUFJTih0c3RfUVdlYkZyYW1lKQo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>33560</attachid>
            <date>2009-07-27 11:39:18 -0700</date>
            <delta_ts>2009-07-28 06:01:53 -0700</delta_ts>
            <desc>Using QSignalSpy to control whether a load finished signal has been emited instead of just wait for some miliseconds.</desc>
            <filename>fixed_hasSetFocus.diff</filename>
            <type>text/plain</type>
            <size>2473</size>
            <attacher name="Andre Pedralho">apedralho</attacher>
            
              <data encoding="base64">Y29tbWl0IGM1YzUyMzU3M2Q3ZTdhZjk0NDY5MTdiZmYzMjhiM2Q0NTljOTUxZDQKQXV0aG9yOiBB
bmRyZSBQZWRyYWxobyA8YW5kcmUucGVkcmFsaG9Ab3BlbmJvc3NhLm9yZz4KRGF0ZTogICBGcmkg
SnVsIDI0IDEwOjQ0OjUzIDIwMDkgLTA0MDAKCiAgICAyMDA5LTA3LTI0ICBBbmRyZSBQZWRyYWxo
byAgPGFuZHJlLnBlZHJhbGhvQG9wZW5ib3NzYS5vcmc+CiAgICAKICAgICAgICAgICAgUmV2aWV3
ZWQgYnkgTk9CT0RZIChPT1BTISkuCiAgICAKICAgICAgICAgICAgRml4ZWQgdHN0X1FXZWJGcmFt
ZTo6aGFzU2V0Rm9jdXMgdGVzdCB3aGljaCB3YXMgdXNpbmcKICAgICAgICAgICAgYW4gdW5kZWZp
bmVkIHJlc291cmNlLgogICAgICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1
Zy5jZ2k/aWQ9Mjc1MTIKICAgIAogICAgICAgICAgICAqIHRlc3RzL3F3ZWJmcmFtZS90c3RfcXdl
YmZyYW1lLmNwcDoKCmRpZmYgLS1naXQgYS9XZWJLaXQvcXQvQ2hhbmdlTG9nIGIvV2ViS2l0L3F0
L0NoYW5nZUxvZwppbmRleCA4ZDZhOWQ4Li44NjZkNzk1IDEwMDY0NAotLS0gYS9XZWJLaXQvcXQv
Q2hhbmdlTG9nCisrKyBiL1dlYktpdC9xdC9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxMyBAQAorMjAw
OS0wNy0yNCAgQW5kcmUgUGVkcmFsaG8gIDxhbmRyZS5wZWRyYWxob0BvcGVuYm9zc2Eub3JnPgor
CisgICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAgRml4ZWQg
dHN0X1FXZWJGcmFtZTo6aGFzU2V0Rm9jdXMgdGVzdCB3aGljaCB3YXMgdXNpbmcKKyAgICAgICAg
IGFuIHVuZGVmaW5lZCByZXNvdXJjZS4KKyAgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3Jn
L3Nob3dfYnVnLmNnaT9pZD0yNzUxMgorCisgICAgICAgICAqIHRlc3RzL3F3ZWJmcmFtZS90c3Rf
cXdlYmZyYW1lLmNwcDoKKwogMjAwOS0wNy0yNyAgVm9sa2VyIEhpbHNoZWltZXIgPHZvbGtlci5o
aWxzaGVpbWVyQG5va2lhLmNvbT4KIAogICAgICAgICBSZXZpZXdlZCBieSBTaW1vbiBIYXVzbWFu
bi4KZGlmZiAtLWdpdCBhL1dlYktpdC9xdC90ZXN0cy9xd2ViZnJhbWUvdHN0X3F3ZWJmcmFtZS5j
cHAgYi9XZWJLaXQvcXQvdGVzdHMvcXdlYmZyYW1lL3RzdF9xd2ViZnJhbWUuY3BwCmluZGV4IDE2
MTZiNWYuLjEzNWIzZmIgMTAwNjQ0Ci0tLSBhL1dlYktpdC9xdC90ZXN0cy9xd2ViZnJhbWUvdHN0
X3F3ZWJmcmFtZS5jcHAKKysrIGIvV2ViS2l0L3F0L3Rlc3RzL3F3ZWJmcmFtZS90c3RfcXdlYmZy
YW1lLmNwcApAQCAtMjU0NCwxNiArMjU0NCwyOSBAQCB2b2lkIHRzdF9RV2ViRnJhbWU6OmJhc2VV
cmwoKQogCiB2b2lkIHRzdF9RV2ViRnJhbWU6Omhhc1NldEZvY3VzKCkKIHsKKyAgICBRU3RyaW5n
IGh0bWwoIjxodG1sPjxib2R5PjxwPnRvcDwvcD4iIFwKKyAgICAgICAgICAgICAgICAgICAgIjxp
ZnJhbWUgd2lkdGg9JzgwJScgaGVpZ2h0PSczMCUnLz4iIFwKKyAgICAgICAgICAgICAgICAgIjwv
Ym9keT48L2h0bWw+Iik7CisKICAgICBRU2lnbmFsU3B5IGxvYWRTcHkobV9wYWdlLCBTSUdOQUwo
bG9hZEZpbmlzaGVkKGJvb2wpKSk7Ci0gICAgUVVybCB1cmwgPSBRVXJsKCJxcmM6Ly8vZnJhbWV0
ZXN0L2lmcmFtZS5odG1sIik7Ci0gICAgbV9wYWdlLT5tYWluRnJhbWUoKS0+bG9hZCh1cmwpOwor
ICAgIG1fcGFnZS0+bWFpbkZyYW1lKCktPnNldEh0bWwoaHRtbCk7CisKKyAgICBRVGVzdDo6cVdh
aXQoMjAwKTsKKyAgICBRQ09NUEFSRShsb2FkU3B5LnNpemUoKSwgMSk7CisKKyAgICBRTGlzdDxR
V2ViRnJhbWUqPiBjaGlsZHJlbiA9IG1fcGFnZS0+bWFpbkZyYW1lKCktPmNoaWxkRnJhbWVzKCk7
CisgICAgUVdlYkZyYW1lICpmcmFtZSA9IGNoaWxkcmVuLmF0KDApOworICAgIFFTdHJpbmcgaW5u
ZXJIdG1sKCI8aHRtbD48Ym9keT48cD5hbm90aGVyIGlmcmFtZTwvcD4iIFwKKyAgICAgICAgICAg
ICAgICAgICAgICAgICI8aWZyYW1lIHdpZHRoPSc4MCUnIGhlaWdodD0nMzAlJy8+IiBcCisgICAg
ICAgICAgICAgICAgICAgICAgIjwvYm9keT48L2h0bWw+Iik7CisgICAgZnJhbWUtPnNldEh0bWwo
aW5uZXJIdG1sKTsKIAotICAgIDo6d2FpdEZvclNpZ25hbChtX3BhZ2UsIFNJR05BTChsb2FkRmlu
aXNoZWQoYm9vbCkpKTsKKyAgICBRVGVzdDo6cVdhaXQoMjAwKTsKKyAgICBRQ09NUEFSRShsb2Fk
U3B5LnNpemUoKSwgMik7CiAKICAgICBtX3BhZ2UtPm1haW5GcmFtZSgpLT5zZXRGb2N1cygpOwog
ICAgIFFWRVJJRlkobV9wYWdlLT5tYWluRnJhbWUoKS0+aGFzRm9jdXMoKSk7CiAKLSAgICBRTGlz
dDxRV2ViRnJhbWUqPiBjaGlsZHJlbiA9IG1fcGFnZS0+bWFpbkZyYW1lKCktPmNoaWxkRnJhbWVz
KCk7CiAgICAgZm9yIChpbnQgaSA9IDA7IGkgPCBjaGlsZHJlbi5zaXplKCk7ICsraSkgewogICAg
ICAgICBjaGlsZHJlbi5hdChpKS0+c2V0Rm9jdXMoKTsKICAgICAgICAgUVZFUklGWShjaGlsZHJl
bi5hdChpKS0+aGFzRm9jdXMoKSk7Cg==
</data>
<flag name="review"
          id="17823"
          type_id="1"
          status="+"
          setter="hausmann"
    />
          </attachment>
      

    </bug>

</bugzilla>