<?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>115197</bug_id>
          
          <creation_ts>2013-04-25 12:39:08 -0700</creation_ts>
          <short_desc>[BlackBerry] Selection overlay on non-composited iframes are incorrectly positioned.</short_desc>
          <delta_ts>2013-04-25 14:38:34 -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 BlackBerry</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</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>1</everconfirmed>
          <reporter name="Andrew Lo">anlo</reporter>
          <assigned_to name="Andrew Lo">anlo</assigned_to>
          <cc>anilsson</cc>
    
    <cc>cgarcia</cc>
    
    <cc>commit-queue</cc>
    
    <cc>rwlbuis</cc>
    
    <cc>xan.lopez</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>881503</commentid>
    <comment_count>0</comment_count>
    <who name="Andrew Lo">anlo</who>
    <bug_when>2013-04-25 12:39:08 -0700</bug_when>
    <thetext>Selection overlay on non-composited iframes are incorrectly positioned when painted.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>881508</commentid>
    <comment_count>1</comment_count>
      <attachid>199740</attachid>
    <who name="Andrew Lo">anlo</who>
    <bug_when>2013-04-25 12:44:41 -0700</bug_when>
    <thetext>Created attachment 199740
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>881517</commentid>
    <comment_count>2</comment_count>
    <who name="Arvid Nilsson">anilsson</who>
    <bug_when>2013-04-25 12:55:33 -0700</bug_when>
    <thetext>LGTM, same strategy as used in SelectionHandler::regionForTextQuads which has been working well for a long time now =)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>881546</commentid>
    <comment_count>3</comment_count>
      <attachid>199740</attachid>
    <who name="Rob Buis">rwlbuis</who>
    <bug_when>2013-04-25 13:54:02 -0700</bug_when>
    <thetext>Comment on attachment 199740
Patch

LGTM.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>881565</commentid>
    <comment_count>4</comment_count>
      <attachid>199740</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2013-04-25 14:38:32 -0700</bug_when>
    <thetext>Comment on attachment 199740
Patch

Clearing flags on attachment: 199740

Committed r149145: &lt;http://trac.webkit.org/changeset/149145&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>881566</commentid>
    <comment_count>5</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2013-04-25 14:38:34 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>199740</attachid>
            <date>2013-04-25 12:44:41 -0700</date>
            <delta_ts>2013-04-25 14:38:32 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-115197-20130425154442.patch</filename>
            <type>text/plain</type>
            <size>2273</size>
            <attacher name="Andrew Lo">anlo</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTQ4OTkwCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L2Js
YWNrYmVycnkvQ2hhbmdlTG9nIGIvU291cmNlL1dlYktpdC9ibGFja2JlcnJ5L0NoYW5nZUxvZwpp
bmRleCAzZmY3OTU2YjNkZDI1ZjkzMzI5YWYyN2UxZjU3MmMwNDEyNjMxMGVhLi5kY2IwMGZhNDJi
Y2QwODYwOTBiODQ1N2I3OGE4ZDdhMjhjZTBhM2JkIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0
L2JsYWNrYmVycnkvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9XZWJLaXQvYmxhY2tiZXJyeS9DaGFu
Z2VMb2cKQEAgLTEsMyArMSwxNyBAQAorMjAxMy0wNC0yNSAgQW5kcmV3IExvICA8YW5sb0ByaW0u
Y29tPgorCisgICAgICAgIFtCbGFja0JlcnJ5XSBTZWxlY3Rpb24gb3ZlcmxheSBvbiBub24tY29t
cG9zaXRlZCBpZnJhbWVzIGFyZSBpbmNvcnJlY3RseSBwb3NpdGlvbmVkLgorICAgICAgICBodHRw
czovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTE1MTk3CisKKyAgICAgICAgUmV2
aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgV2hlbiBkcmF3aW5nIHRoZSBzZWxl
Y3Rpb24gb3ZlcmxheSwgdGhlIHJlY3RzIHRvCisgICAgICAgIHBhaW50IHdoZW4gc2VsZWN0aW5n
IHRleHQgb24gbm9uLWNvbXBvc2l0ZWQgc3ViLWZyYW1lcworICAgICAgICBuZWVkIHRvIGJlIGFk
anVzdGVkIGJ5IHRoZSBmcmFtZSBwb3NpdGlvbi4KKworICAgICAgICAqIFdlYktpdFN1cHBvcnQv
U2VsZWN0aW9uT3ZlcmxheS5jcHA6CisgICAgICAgIChCbGFja0JlcnJ5OjpXZWJLaXQ6OlNlbGVj
dGlvbk92ZXJsYXk6OnBhaW50Q29udGVudHMpOgorCiAyMDEzLTA0LTIyICBYYW4gTG9wZXogIDx4
bG9wZXpAaWdhbGlhLmNvbT4KIAogICAgICAgICBbQmxhY2tCZXJyeV0gQVNTRVJUIGluIEZyYW1l
TG9hZGVyQ2xpZW50Ojpjb252ZXJ0TWFpblJlc291cmNlTG9hZFRvRG93bmxvYWQKZGlmZiAtLWdp
dCBhL1NvdXJjZS9XZWJLaXQvYmxhY2tiZXJyeS9XZWJLaXRTdXBwb3J0L1NlbGVjdGlvbk92ZXJs
YXkuY3BwIGIvU291cmNlL1dlYktpdC9ibGFja2JlcnJ5L1dlYktpdFN1cHBvcnQvU2VsZWN0aW9u
T3ZlcmxheS5jcHAKaW5kZXggMmI1NjFhODUwZjNkNWNmMWFmMjM0ODUyNDllZGRiODIyNTYzMzBl
OS4uNWY0YTEwYjY1ZDlkYTVkYTkwZWQyNGE5NWJlNzdjMDlmNmZlOWZkNCAxMDA2NDQKLS0tIGEv
U291cmNlL1dlYktpdC9ibGFja2JlcnJ5L1dlYktpdFN1cHBvcnQvU2VsZWN0aW9uT3ZlcmxheS5j
cHAKKysrIGIvU291cmNlL1dlYktpdC9ibGFja2JlcnJ5L1dlYktpdFN1cHBvcnQvU2VsZWN0aW9u
T3ZlcmxheS5jcHAKQEAgLTIyLDExICsyMiwxMyBAQAogCiAjaW5jbHVkZSAiU2VsZWN0aW9uT3Zl
cmxheS5oIgogCisjaW5jbHVkZSAiRnJhbWUuaCIKICNpbmNsdWRlICJHcmFwaGljc0NvbnRleHQu
aCIKICNpbmNsdWRlICJMYXllck1lc3NhZ2UuaCIKICNpbmNsdWRlICJMYXllcldlYktpdFRocmVh
ZC5oIgogI2luY2x1ZGUgIlBhdGguaCIKICNpbmNsdWRlICJSZW5kZXJUaGVtZS5oIgorI2luY2x1
ZGUgIlJlbmRlclZpZXcuaCIKICNpbmNsdWRlICJXZWJQYWdlX3AuaCIKIAogI2luY2x1ZGUgPEJs
YWNrQmVycnlQbGF0Zm9ybU1lc3NhZ2VDbGllbnQuaD4KQEAgLTExMSw2ICsxMTMsMTMgQEAgdm9p
ZCBTZWxlY3Rpb25PdmVybGF5OjpwYWludENvbnRlbnRzKGNvbnN0IEdyYXBoaWNzTGF5ZXIqIGxh
eWVyLCBHcmFwaGljc0NvbnRleHQKIAogICAgIGZvciAodW5zaWduZWQgaSA9IDA7IGkgPCBxdWFk
cy5zaXplKCk7ICsraSkgewogICAgICAgICBGbG9hdFJlY3QgcmVjdFRvUGFpbnQgPSBxdWFkc1tp
XS5ib3VuZGluZ0JveCgpOworCisgICAgICAgIC8vIFNlbGVjdGlvbiBvbiBub24tY29tcG9zaXRl
ZCBzdWItZnJhbWVzIG5lZWQgdG8gYmUgYWRqdXN0ZWQuCisgICAgICAgIGlmICghbV9wYWdlLT5m
b2N1c2VkT3JNYWluRnJhbWUoKS0+Y29udGVudFJlbmRlcmVyKCktPmlzQ29tcG9zaXRlZCgpKSB7
CisgICAgICAgICAgICBXZWJDb3JlOjpJbnRQb2ludCBmcmFtZVBvc2l0aW9uID0gbV9wYWdlLT5m
cmFtZU9mZnNldChtX3BhZ2UtPmZvY3VzZWRPck1haW5GcmFtZSgpKTsKKyAgICAgICAgICAgIHJl
Y3RUb1BhaW50Lm1vdmUoZnJhbWVQb3NpdGlvbi54KCksIGZyYW1lUG9zaXRpb24ueSgpKTsKKyAg
ICAgICAgfQorCiAgICAgICAgIHJlY3RUb1BhaW50LmludGVyc2VjdChjbGlwKTsKICAgICAgICAg
aWYgKHJlY3RUb1BhaW50LmlzRW1wdHkoKSkKICAgICAgICAgICAgIGNvbnRpbnVlOwo=
</data>

          </attachment>
      

    </bug>

</bugzilla>