<?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>38086</bug_id>
          
          <creation_ts>2010-04-24 18:20:17 -0700</creation_ts>
          <short_desc>focus handling needs improving</short_desc>
          <delta_ts>2010-04-25 15:04:20 -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 wx</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>OS X 10.5</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="Kevin Watters">kevinwatters</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>kevino</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>216769</commentid>
    <comment_count>0</comment_count>
      <attachid>54226</attachid>
    <who name="Kevin Watters">kevinwatters</who>
    <bug_when>2010-04-24 18:20:17 -0700</bug_when>
    <thetext>Created attachment 54226
improves focus handling

The wx port need to use page-&gt;focusController methods in a few places it is not currently.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>216853</commentid>
    <comment_count>1</comment_count>
    <who name="Kevin Ollivier">kevino</who>
    <bug_when>2010-04-25 15:04:20 -0700</bug_when>
    <thetext>Landed in r58231, thanks!</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>54226</attachid>
            <date>2010-04-24 18:20:17 -0700</date>
            <delta_ts>2010-04-25 14:21:36 -0700</delta_ts>
            <desc>improves focus handling</desc>
            <filename>focus_fixes.patch</filename>
            <type>text/plain</type>
            <size>5014</size>
            <attacher name="Kevin Watters">kevinwatters</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYktpdC93eC9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2ViS2l0L3d4L0NoYW5n
ZUxvZwkocmV2aXNpb24gNTgyMDEpCisrKyBXZWJLaXQvd3gvQ2hhbmdlTG9nCSh3b3JraW5nIGNv
cHkpCkBAIC0xLDMgKzEsMTkgQEAKKzIwMTAtMDQtMjMgIEtldmluIFdhdHRlcnMgIDxrZXZpbndh
dHRlcnNAZ21haWwuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgor
CisgICAgICAgIFVwZGF0ZSBmb2N1cyBoYW5kbGluZyBjb2RlIHRvIG1hdGNoIGN1cnJlbnQgYXBw
cm9hY2hlcyB1c2VkIGJ5IG90aGVyIHBsYXRmb3JtcywKKyAgICAgICAgYW5kIGZpeCBmb2N1cyBo
YW5kbGluZyBmb3IgY29ybmVyIGNhc2VzIHN1Y2ggYXMgd2hlbiBhIG1vdXNlIGRvd24gcG9wcyB1
cCBhIGRpYWxvZy4gCisKKyAgICAgICAgKiBXZWJWaWV3LmNwcDoKKyAgICAgICAgKHd4V2ViVmll
dzo6Q3JlYXRlKToKKyAgICAgICAgKHd4V2ViVmlldzo6T25UTFdBY3RpdmF0ZWQpOgorICAgICAg
ICAod3hXZWJWaWV3OjpPblBhaW50KToKKyAgICAgICAgKHd4V2ViVmlldzo6T25Nb3VzZUV2ZW50
cyk6CisgICAgICAgICh3eFdlYlZpZXc6Ok9uU2V0Rm9jdXMpOgorICAgICAgICAod3hXZWJWaWV3
OjpPbktpbGxGb2N1cyk6CisgICAgICAgICogV2ViVmlldy5oOgorCiAyMDEwLTA0LTIwICBBZGFt
IEJhcnRoICA8YWJhcnRoQHdlYmtpdC5vcmc+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgRXJpYyBT
ZWlkZWwuCkluZGV4OiBXZWJLaXQvd3gvV2ViVmlldy5jcHAKPT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2ViS2l0
L3d4L1dlYlZpZXcuY3BwCShyZXZpc2lvbiA1ODE5OSkKKysrIFdlYktpdC93eC9XZWJWaWV3LmNw
cAkod29ya2luZyBjb3B5KQpAQCAtMzQwLDYgKzM0MCw5IEBACiAgICAgU2V0RGF0YWJhc2VzRW5h
YmxlZCh0cnVlKTsKICNlbmRpZgogCisgICAgd3hXaW5kb3cqIHRsdyA9IHd4R2V0VG9wTGV2ZWxQ
YXJlbnQodGhpcyk7CisgICAgdGx3LT5Db25uZWN0KC0xLCB3eEVWVF9BQ1RJVkFURSwgd3hBY3Rp
dmF0ZUV2ZW50SGFuZGxlcih3eFdlYlZpZXc6Ok9uVExXQWN0aXZhdGVkKSk7CisKICAgICBtX2lz
SW5pdGlhbGl6ZWQgPSB0cnVlOwogCiAgICAgcmV0dXJuIHRydWU7CkBAIC0zNTksNiArMzYyLDE1
IEBACiAgICAgbV9pbXBsLT5wYWdlID0gMDsgICAKIH0KIAordm9pZCB3eFdlYlZpZXc6Ok9uVExX
QWN0aXZhdGVkKHd4QWN0aXZhdGVFdmVudCYgZXZlbnQpCit7ICAgICAgICAKKyAgICBpZiAobV9p
bXBsICYmIG1faW1wbC0+cGFnZSAmJiBtX2ltcGwtPnBhZ2UtPmZvY3VzQ29udHJvbGxlcigpKQor
ICAgICAgICBtX2ltcGwtPnBhZ2UtPmZvY3VzQ29udHJvbGxlcigpLT5zZXRBY3RpdmUoZXZlbnQu
R2V0QWN0aXZlKCkpOworICAgIAorICAgIGV2ZW50LlNraXAoKTsKKyAgICAKK30KKwogdm9pZCB3
eFdlYlZpZXc6OlN0b3AoKQogewogICAgIGlmIChtX21haW5GcmFtZSkKQEAgLTUyOCwxMyArNTQw
LDYgQEAKICAgICBpZiAobV9iZWluZ0Rlc3Ryb3llZCB8fCAhbV9tYWluRnJhbWUpCiAgICAgICAg
IHJldHVybjsKIAotICAgIC8vIFdlYlZpZXcgYWN0aXZlIHN0YXRlIGlzIGJhc2VkIG9uIFRMVyBh
Y3RpdmUgc3RhdGUuCi0gICAgd3hUb3BMZXZlbFdpbmRvdyogdGx3ID0gZHluYW1pY19jYXN0PHd4
VG9wTGV2ZWxXaW5kb3cqPih3eEdldFRvcExldmVsUGFyZW50KHRoaXMpKTsKLSAgICBpZiAodGx3
ICYmIHRsdy0+SXNBY3RpdmUoKSkKLSAgICAgICAgbV9pbXBsLT5wYWdlLT5mb2N1c0NvbnRyb2xs
ZXIoKS0+c2V0QWN0aXZlKHRydWUpOwotICAgIGVsc2UgewotICAgICAgICBtX2ltcGwtPnBhZ2Ut
PmZvY3VzQ29udHJvbGxlcigpLT5zZXRBY3RpdmUoZmFsc2UpOwotICAgIH0KICAgICBXZWJDb3Jl
OjpGcmFtZSogZnJhbWUgPSBtX21haW5GcmFtZS0+R2V0RnJhbWUoKTsKICAgICBpZiAoIWZyYW1l
IHx8ICFmcmFtZS0+dmlldygpKQogICAgICAgICByZXR1cm47CkBAIC02MjAsNiArNjI1LDIxIEBA
CiAgICAgICAgIHJldHVybjsKICAgICB9CiAgICAgCisgICAgLy8gSWYgYW4gZXZlbnQsIHN1Y2gg
YXMgYSByaWdodC1jbGljayBldmVudCwgbGVhZHMgdG8gYSBmb2N1cyBjaGFuZ2UgKGUuZy4gaXQg
CisgICAgLy8gcmFpc2VzIGEgZGlhbG9nKSwgV2ViS2l0IG5ldmVyIGdldHMgdGhlIG1vdXNlIHVw
IGV2ZW50IGFuZCBuZXZlciByZWxpbnF1aXNoZXMgCisgICAgLy8gbW91c2UgY2FwdHVyZS4gVGhp
cyBsZWFkcyB0byBXZWJLaXQgaGFuZGxpbmcgbW91c2UgZXZlbnRzLCBzdWNoIGFzIG1vZGlmeWlu
ZworICAgIC8vIHRoZSBzZWxlY3Rpb24sIHdoaWxlIG90aGVyIGNvbnRyb2xzIG9yIHRvcCBsZXZl
bCB3aW5kb3dzIGhhdmUgdGhlIGZvY3VzLgorICAgIC8vIEknbSBub3Qgc3VyZSBpZiB0aGlzIGlz
IHRoZSByaWdodCBwbGFjZSB0byBoYW5kbGUgdGhpcywgYnV0IEkgY2FuJ3Qgc2VlbSB0bworICAg
IC8vIGZpbmQgYSBwcmVjZWRlbnQgb24gaG93IHRvIGhhbmRsZSB0aGlzIGluIG90aGVyIHBvcnRz
LgorICAgIGlmICh3eFdpbmRvdzo6RmluZEZvY3VzKCkgIT0gdGhpcykgeworICAgICAgICB3aGls
ZSAoSGFzQ2FwdHVyZSgpKQorICAgICAgICAgICAgUmVsZWFzZU1vdXNlKCk7CisKKyAgICAgICAg
ZnJhbWUtPmV2ZW50SGFuZGxlcigpLT5zZXRNb3VzZVByZXNzZWQoZmFsc2UpOworCisgICAgICAg
IHJldHVybjsKKyAgICB9CisgICAgICAgIAogICAgIGludCBjbGlja0NvdW50ID0gZXZlbnQuQnV0
dG9uRENsaWNrKCkgPyAyIDogMTsKIAogICAgIGlmIChjbGlja0NvdW50ID09IDEgJiYgbV9pbXBs
LT50cmlwbGVDbGlja1RpbWVyLklzUnVubmluZygpKSB7CkBAIC04ODIsMjYgKzkwMiwzNSBAQAog
CiB2b2lkIHd4V2ViVmlldzo6T25TZXRGb2N1cyh3eEZvY3VzRXZlbnQmIGV2ZW50KQogewotICAg
IFdlYkNvcmU6OkZyYW1lKiBmcmFtZSA9IDA7Ci0gICAgaWYgKG1fbWFpbkZyYW1lKQotICAgICAg
ICBmcmFtZSA9IG1fbWFpbkZyYW1lLT5HZXRGcmFtZSgpOwotICAgICAgICAKLSAgICBpZiAoZnJh
bWUpIHsKLSAgICAgICAgZnJhbWUtPnNlbGVjdGlvbigpLT5zZXRGb2N1c2VkKHRydWUpOworICAg
IGlmIChtX2ltcGwgJiYgbV9pbXBsLT5wYWdlICYmIG1faW1wbC0+cGFnZS0+Zm9jdXNDb250cm9s
bGVyKCkpIHsKKyAgICAgICAgbV9pbXBsLT5wYWdlLT5mb2N1c0NvbnRyb2xsZXIoKS0+c2V0Rm9j
dXNlZCh0cnVlKTsKKyAgICAgICAgbV9pbXBsLT5wYWdlLT5mb2N1c0NvbnRyb2xsZXIoKS0+c2V0
QWN0aXZlKHRydWUpOworCisgICAgICAgIGlmICghbV9pbXBsLT5wYWdlLT5mb2N1c0NvbnRyb2xs
ZXIoKS0+Zm9jdXNlZEZyYW1lKCkgJiYgbV9tYWluRnJhbWUpCisgICAgICAgICAgICBtX2ltcGwt
PnBhZ2UtPmZvY3VzQ29udHJvbGxlcigpLT5zZXRGb2N1c2VkRnJhbWUobV9tYWluRnJhbWUtPkdl
dEZyYW1lKCkpOwogICAgIH0KLQorICAgIAogICAgIGV2ZW50LlNraXAoKTsKIH0KIAogdm9pZCB3
eFdlYlZpZXc6Ok9uS2lsbEZvY3VzKHd4Rm9jdXNFdmVudCYgZXZlbnQpCiB7Ci0gICAgV2ViQ29y
ZTo6RnJhbWUqIGZyYW1lID0gMDsKLSAgICBpZiAobV9tYWluRnJhbWUpCi0gICAgICAgIGZyYW1l
ID0gbV9tYWluRnJhbWUtPkdldEZyYW1lKCk7Ci0gICAgICAgIAotICAgIGlmIChmcmFtZSkgewot
ICAgICAgICBmcmFtZS0+c2VsZWN0aW9uKCktPnNldEZvY3VzZWQoZmFsc2UpOworICAgIGlmICht
X2ltcGwgJiYgbV9pbXBsLT5wYWdlICYmIG1faW1wbC0+cGFnZS0+Zm9jdXNDb250cm9sbGVyKCkp
IHsKKyAgICAgICAgbV9pbXBsLT5wYWdlLT5mb2N1c0NvbnRyb2xsZXIoKS0+c2V0Rm9jdXNlZChm
YWxzZSk7CisKKyAgICAgICAgLy8gV2UgYWxzbyBoYW5kbGUgYWN0aXZlIHN0YXRlIGluIE9uVExX
QWN0aXZhdGVkLCBidXQgaWYgYSB1c2VyIGRvZXMgbm90CisgICAgICAgIC8vIGNhbGwgZXZlbnQu
U2tpcCgpIGluIHRoZWlyIG93biBFVlRfQUNUSVZBVEUgaGFuZGxlciwgd2Ugd29uJ3QgZ2V0IHRo
b3NlCisgICAgICAgIC8vIGNhbGxiYWNrcy4gU28gd2UgaGFuZGxlIGFjdGl2ZSBzdGF0ZSBoZXJl
IGFzIHdlbGwgYXMgYSBmYWxsYmFjay4KKyAgICAgICAgd3hUb3BMZXZlbFdpbmRvdyogdGx3ID0g
ZHluYW1pY19jYXN0PHd4VG9wTGV2ZWxXaW5kb3cqPih3eEdldFRvcExldmVsUGFyZW50KHRoaXMp
KTsKKyAgICAgICAgaWYgKHRsdyAmJiB0bHctPklzQWN0aXZlKCkpCisgICAgICAgICAgICBtX2lt
cGwtPnBhZ2UtPmZvY3VzQ29udHJvbGxlcigpLT5zZXRBY3RpdmUodHJ1ZSk7CisgICAgICAgIGVs
c2UKKyAgICAgICAgICAgIG1faW1wbC0+cGFnZS0+Zm9jdXNDb250cm9sbGVyKCktPnNldEFjdGl2
ZShmYWxzZSk7CiAgICAgfQorICAgIAorICAgIHdoaWxlIChIYXNDYXB0dXJlKCkpCisgICAgICAg
IFJlbGVhc2VNb3VzZSgpOworICAgIAogICAgIGV2ZW50LlNraXAoKTsKIH0KIApJbmRleDogV2Vi
S2l0L3d4L1dlYlZpZXcuaAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJLaXQvd3gvV2ViVmlldy5oCShyZXZp
c2lvbiA1ODE5OSkKKysrIFdlYktpdC93eC9XZWJWaWV3LmgJKHdvcmtpbmcgY29weSkKQEAgLTIy
Niw2ICsyMjYsNyBAQAogICAgIHZvaWQgT25LZXlFdmVudHMod3hLZXlFdmVudCYgZXZlbnQpOwog
ICAgIHZvaWQgT25TZXRGb2N1cyh3eEZvY3VzRXZlbnQmIGV2ZW50KTsKICAgICB2b2lkIE9uS2ls
bEZvY3VzKHd4Rm9jdXNFdmVudCYgZXZlbnQpOworICAgIHZvaWQgT25UTFdBY3RpdmF0ZWQod3hB
Y3RpdmF0ZUV2ZW50JiBldmVudCk7CiAgICAgCiBwcml2YXRlOgogICAgIC8vIGFueSBjbGFzcyB3
aXNoaW5nIHRvIHByb2Nlc3Mgd3hXaW5kb3dzIGV2ZW50cyBtdXN0IHVzZSB0aGlzIG1hY3JvCg==
</data>
<flag name="review"
          id="38139"
          type_id="1"
          status="+"
          setter="kevino"
    />
    <flag name="commit-queue"
          id="38160"
          type_id="3"
          status="-"
          setter="kevino"
    />
          </attachment>
      

    </bug>

</bugzilla>