<?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>91357</bug_id>
          
          <creation_ts>2012-07-15 22:10:48 -0700</creation_ts>
          <short_desc>[TestNetscapePlugin][X11] Additional key up event handler routine is required</short_desc>
          <delta_ts>2012-08-13 02:29:52 -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>Plug-ins</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Other</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>LayoutTestFailure</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="KwangYong Choi">ky0.choi</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>cdumez</cc>
    
    <cc>eric</cc>
    
    <cc>gyuyoung.kim</cc>
    
    <cc>kbalazs</cc>
    
    <cc>ky0.choi</cc>
    
    <cc>lucas.de.marchi</cc>
    
    <cc>marc.ordinasillopis</cc>
    
    <cc>rakuco</cc>
    
    <cc>rniwa</cc>
    
    <cc>ryuan.choi</cc>
    
    <cc>vestbo</cc>
    
    <cc>webkit.review.bot</cc>
    
    <cc>wez</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>670077</commentid>
    <comment_count>0</comment_count>
    <who name="KwangYong Choi">ky0.choi</who>
    <bug_when>2012-07-15 22:10:48 -0700</bug_when>
    <thetext>Fixed key up event handler routine to pass http/tests/plugins/plugin-document-has-focus.html on X11 architecture. The test can not be done without this patch. All other ports implemented it already.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>671116</commentid>
    <comment_count>1</comment_count>
      <attachid>152723</attachid>
    <who name="KwangYong Choi">ky0.choi</who>
    <bug_when>2012-07-17 03:07:38 -0700</bug_when>
    <thetext>Created attachment 152723
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>693206</commentid>
    <comment_count>2</comment_count>
      <attachid>152723</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2012-08-12 03:26:44 -0700</bug_when>
    <thetext>Comment on attachment 152723
Patch

Crazy.  This needs to be testRunner now iirc.  layoutTestController is dead.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>693434</commentid>
    <comment_count>3</comment_count>
      <attachid>157936</attachid>
    <who name="KwangYong Choi">ky0.choi</who>
    <bug_when>2012-08-13 01:02:45 -0700</bug_when>
    <thetext>Created attachment 157936
Patch

Applied the changes.

Don&apos;t expose window.layoutTestController since all layout tests use window.testRunner instead of window.layoutTestController as of r124685.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>693479</commentid>
    <comment_count>4</comment_count>
      <attachid>157936</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-08-13 02:29:45 -0700</bug_when>
    <thetext>Comment on attachment 157936
Patch

Clearing flags on attachment: 157936

Committed r125401: &lt;http://trac.webkit.org/changeset/125401&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>693480</commentid>
    <comment_count>5</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-08-13 02:29:52 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>152723</attachid>
            <date>2012-07-17 03:07:38 -0700</date>
            <delta_ts>2012-08-13 01:02:45 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>y0.choi@samsung.com]-[[TestNetscapePlugin][X11] Additional key up event handler routine is required]</filename>
            <type>text/plain</type>
            <size>1668</size>
            <attacher name="KwangYong Choi">ky0.choi</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCA1MmZm
OTE4Li4xZjBhMzM4IDEwMDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hh
bmdlTG9nCkBAIC0xLDMgKzEsMTcgQEAKKzIwMTItMDctMTUgIEt3YW5nWW9uZyBDaG9pICA8a3kw
LmNob2lAc2Ftc3VuZy5jb20+CisKKyAgICAgICAgW1Rlc3ROZXRzY2FwZVBsdWdpbl1bWDExXSBB
ZGRpdGlvbmFsIGtleSB1cCBldmVudCBoYW5kbGVyIHJvdXRpbmUgaXMgcmVxdWlyZWQKKyAgICAg
ICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTkxMzU3CisKKyAgICAg
ICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgRml4ZWQga2V5IHVwIGV2
ZW50IGhhbmRsZXIgcm91dGluZSB0byBwYXNzIGh0dHAvdGVzdHMvcGx1Z2lucy9wbHVnaW4tZG9j
dW1lbnQtCisgICAgICAgIGhhcy1mb2N1cy5odG1sIG9uIFgxMSBhcmNoaXRlY3R1cmUuIFRoZSB0
ZXN0IGNhbiBub3QgYmUgZG9uZSB3aXRob3V0IHRoaXMgcGF0Y2guCisgICAgICAgIEFsbCBvdGhl
ciBwb3J0cyBpbXBsZW1lbnRlZCBpdCBhbHJlYWR5LgorCisgICAgICAgICogRHVtcFJlbmRlclRy
ZWUvVGVzdE5ldHNjYXBlUGx1Z0luL21haW4uY3BwOgorICAgICAgICAoaGFuZGxlRXZlbnRYMTEp
OiBNb2RpZmllZCBrZXkgdXAgZXZlbnQgaGFuZGxlciBmb3IgWDExCisKIDIwMTItMDctMTUgIEd5
dXlvdW5nIEtpbSAgPGd5dXlvdW5nLmtpbUBzYW1zdW5nLmNvbT4KIAogICAgICAgICBVbnJldmll
d2VkLiBBZGQgRUZMV2ViS2l0MlB1YmxpY0FQSSwgRUZMV2ViS2l0MlBsYXRmb3JtU3BlY2lmaWMg
dG8gd2F0Y2ggbGlzdC4KZGlmZiAtLWdpdCBhL1Rvb2xzL0R1bXBSZW5kZXJUcmVlL1Rlc3ROZXRz
Y2FwZVBsdWdJbi9tYWluLmNwcCBiL1Rvb2xzL0R1bXBSZW5kZXJUcmVlL1Rlc3ROZXRzY2FwZVBs
dWdJbi9tYWluLmNwcAppbmRleCBhMGQ5ODk1Li4xZDZiNWU4IDEwMDY0NAotLS0gYS9Ub29scy9E
dW1wUmVuZGVyVHJlZS9UZXN0TmV0c2NhcGVQbHVnSW4vbWFpbi5jcHAKKysrIGIvVG9vbHMvRHVt
cFJlbmRlclRyZWUvVGVzdE5ldHNjYXBlUGx1Z0luL21haW4uY3BwCkBAIC02NjAsNiArNjYwLDEx
IEBAIHN0YXRpYyBpbnQxNl90IGhhbmRsZUV2ZW50WDExKE5QUCBpbnN0YW5jZSwgUGx1Z2luT2Jq
ZWN0KiBvYmosIFhFdmVudCogZXZlbnQpCiAgICAgICAgIC8vIEZJWE1FOiBleHRyYWN0IGtleSBj
b2RlCiAgICAgICAgIGlmIChvYmotPmV2ZW50TG9nZ2luZykKICAgICAgICAgICAgIHBsdWdpbkxv
ZyhpbnN0YW5jZSwgImtleVVwICclYyciLCBrZXlFdmVudFRvQ2hhcigmZXZlbnQtPnhrZXkpKTsK
KyAgICAgICAgaWYgKG9iai0+dGVzdEtleWJvYXJkRm9jdXNGb3JQbHVnaW5zKSB7CisgICAgICAg
ICAgICBvYmotPmV2ZW50TG9nZ2luZyA9IGZhbHNlOworICAgICAgICAgICAgb2JqLT50ZXN0S2V5
Ym9hcmRGb2N1c0ZvclBsdWdpbnMgPSBGQUxTRTsKKyAgICAgICAgICAgIGV4ZWN1dGVTY3JpcHQo
b2JqLCAibGF5b3V0VGVzdENvbnRyb2xsZXIubm90aWZ5RG9uZSgpOyIpOworICAgICAgICB9CiAg
ICAgICAgIGJyZWFrOwogICAgIGNhc2UgR3JhcGhpY3NFeHBvc2U6CiAgICAgICAgIGlmIChvYmot
PmV2ZW50TG9nZ2luZykK
</data>
<flag name="review"
          id="161465"
          type_id="1"
          status="+"
          setter="eric"
    />
    <flag name="commit-queue"
          id="161466"
          type_id="3"
          status="-"
          setter="eric"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>157936</attachid>
            <date>2012-08-13 01:02:45 -0700</date>
            <delta_ts>2012-08-13 02:29:45 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>diff</filename>
            <type>text/plain</type>
            <size>1645</size>
            <attacher name="KwangYong Choi">ky0.choi</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCA2ZTli
YzhhLi4yYmMwMDExIDEwMDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hh
bmdlTG9nCkBAIC0xLDMgKzEsMTcgQEAKKzIwMTItMDgtMTMgIEt3YW5nWW9uZyBDaG9pICA8a3kw
LmNob2lAc2Ftc3VuZy5jb20+CisKKyAgICAgICAgW1Rlc3ROZXRzY2FwZVBsdWdpbl1bWDExXSBB
ZGRpdGlvbmFsIGtleSB1cCBldmVudCBoYW5kbGVyIHJvdXRpbmUgaXMgcmVxdWlyZWQKKyAgICAg
ICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTkxMzU3CisKKyAgICAg
ICAgUmV2aWV3ZWQgYnkgRXJpYyBTZWlkZWwuCisKKyAgICAgICAgRml4ZWQga2V5IHVwIGV2ZW50
IGhhbmRsZXIgcm91dGluZSB0byBwYXNzIGh0dHAvdGVzdHMvcGx1Z2lucy9wbHVnaW4tZG9jdW1l
bnQtCisgICAgICAgIGhhcy1mb2N1cy5odG1sIG9uIFgxMSBhcmNoaXRlY3R1cmUuIFRoZSB0ZXN0
IGNhbiBub3QgYmUgZG9uZSB3aXRob3V0IHRoaXMgcGF0Y2guCisgICAgICAgIEFsbCBvdGhlciBw
b3J0cyBpbXBsZW1lbnRlZCBpdCBhbHJlYWR5LgorCisgICAgICAgICogRHVtcFJlbmRlclRyZWUv
VGVzdE5ldHNjYXBlUGx1Z0luL21haW4uY3BwOgorICAgICAgICAoaGFuZGxlRXZlbnRYMTEpOiBN
b2RpZmllZCBrZXkgdXAgZXZlbnQgaGFuZGxlciBmb3IgWDExCisKIDIwMTItMDgtMTIgIE1PUklU
QSBIYWppbWUgIDxtb3JyaXRhQGdvb2dsZS5jb20+CiAKICAgICAgICAgVW5yZXZpZXdlZCwgYWRk
ZWQgYSBwcm94eSBhZGRyZXNzIHRvIGNvbnRyaWJ1dG9yc193aG9fYXJlX25vdF9jb21taXR0ZXJz
LgpkaWZmIC0tZ2l0IGEvVG9vbHMvRHVtcFJlbmRlclRyZWUvVGVzdE5ldHNjYXBlUGx1Z0luL21h
aW4uY3BwIGIvVG9vbHMvRHVtcFJlbmRlclRyZWUvVGVzdE5ldHNjYXBlUGx1Z0luL21haW4uY3Bw
CmluZGV4IGZkOTJiZmQuLjk3YTRjZjMgMTAwNjQ0Ci0tLSBhL1Rvb2xzL0R1bXBSZW5kZXJUcmVl
L1Rlc3ROZXRzY2FwZVBsdWdJbi9tYWluLmNwcAorKysgYi9Ub29scy9EdW1wUmVuZGVyVHJlZS9U
ZXN0TmV0c2NhcGVQbHVnSW4vbWFpbi5jcHAKQEAgLTY3MSw2ICs2NzEsMTEgQEAgc3RhdGljIGlu
dDE2X3QgaGFuZGxlRXZlbnRYMTEoTlBQIGluc3RhbmNlLCBQbHVnaW5PYmplY3QqIG9iaiwgWEV2
ZW50KiBldmVudCkKICAgICAgICAgLy8gRklYTUU6IGV4dHJhY3Qga2V5IGNvZGUKICAgICAgICAg
aWYgKG9iai0+ZXZlbnRMb2dnaW5nKQogICAgICAgICAgICAgcGx1Z2luTG9nKGluc3RhbmNlLCAi
a2V5VXAgJyVjJyIsIGtleUV2ZW50VG9DaGFyKCZldmVudC0+eGtleSkpOworICAgICAgICBpZiAo
b2JqLT50ZXN0S2V5Ym9hcmRGb2N1c0ZvclBsdWdpbnMpIHsKKyAgICAgICAgICAgIG9iai0+ZXZl
bnRMb2dnaW5nID0gZmFsc2U7CisgICAgICAgICAgICBvYmotPnRlc3RLZXlib2FyZEZvY3VzRm9y
UGx1Z2lucyA9IEZBTFNFOworICAgICAgICAgICAgZXhlY3V0ZVNjcmlwdChvYmosICJ0ZXN0UnVu
bmVyLm5vdGlmeURvbmUoKTsiKTsKKyAgICAgICAgfQogICAgICAgICBicmVhazsKICAgICBjYXNl
IEdyYXBoaWNzRXhwb3NlOgogICAgICAgICBpZiAob2JqLT5ldmVudExvZ2dpbmcpCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>