<?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>89160</bug_id>
          
          <creation_ts>2012-06-14 19:53:07 -0700</creation_ts>
          <short_desc>[BlackBerry] Selection - Crash when manipulating selection by dragging handle</short_desc>
          <delta_ts>2012-07-05 17:24: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 BlackBerry</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Other</rep_platform>
          <op_sys>Other</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>WORKSFORME</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>
          <dependson>89193</dependson>
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Sean Wang">xuewen.ok</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>rwlbuis</cc>
    
    <cc>tonikitoo</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>649796</commentid>
    <comment_count>0</comment_count>
    <who name="Sean Wang">xuewen.ok</who>
    <bug_when>2012-06-14 19:53:07 -0700</bug_when>
    <thetext>RIM bug:164970
I got this:

Program received signal SIGSEGV, Segmentation fault.
[Switching to pid 63352986 tid 16]
WebCore::Node::shadowAncestorNode (this=0x0) at
/home/anilsson/dev/webkit/Source/WebCore/dom/Node.cpp:1496
1496        if (isSVGElement())
(gdb) bt
#0  WebCore::Node::shadowAncestorNode (this=0x0) at
/home/anilsson/dev/webkit/Source/WebCore/dom/Node.cpp:1496
#1  0x7bc85520 in WebCore::comparePositions (a=..., b=...)
    at /home/anilsson/dev/webkit/Source/WebCore/editing/htmlediting.cpp:78
#2  0x7bc82d00 in WebCore::VisibleSelection::setWithoutValidation
(this=0x16d74d8, base=..., extent=...)
    at
/home/anilsson/dev/webkit/Source/WebCore/editing/VisibleSelection.cpp:448
#3  0x7bbab3ce in BlackBerry::WebKit::SelectionHandler::setSelection
(this=0x14ffbb0, start=..., end=...)
    at
/home/anilsson/dev/webkit/Source/WebKit/blackberry/WebKitSupport/SelectionHandler.cpp:492
#4  0x7bb6b99e in BlackBerry::WebKit::WebPage::setSelection (this=0x147d8a8,
startPoint=..., endPoint=...)
    at /home/anilsson/dev/webkit/Source/WebKit/blackberry/Api/WebPage.cpp:4586
#5  0x797cb2ce in BlackBerry::Platform::MethodDelegate2&lt;void
(BlackBerry::WebKit::WebPage::*)(BlackBerry::Platform::IntPoint const&amp;,
BlackBerry::Platform::IntPoint const&amp;), BlackBerry::WebKit::WebPage,
BlackBerry::Platform::IntPoint, BlackBerry::Platform::IntPoint&gt;::execute
(this=0x14fd730)
    at
/home/anilsson/dev/staging/usr/include/browser/platform/BlackBerryPlatformMessageDelegate.h:934
#6  0x7b06cbcc in BlackBerry::Platform::ExecutableMessage::execute
(this=&lt;optimized out&gt;)
    at
/home/anilsson/dev/platform/blackberryplatform/BlackBerryPlatformExecutableMessage.cpp:38
#7  0x7b079416 in BlackBerry::Platform::MessageClient::executeMessage
(this=0x13aeb70, message=0x141bc58, returnCode=0)
    at
/home/anilsson/dev/platform/blackberryplatform/BlackBerryPlatformMessageClient.cpp:680
#8  0x7b079b16 in BlackBerry::Platform::MessageClient::coalesceMessage
(this=0x13aeb70, message=0x141bc58, returnCode=0)
    at
/home/anilsson/dev/platform/blackberryplatform/BlackBerryPlatformMessageClient.cpp:666
#9  0x7b079bb2 in BlackBerry::Platform::MessageClient::receivePendingMessage
(this=0x13aeb70, timeout=&lt;optimized out&gt;)
    at
/home/anilsson/dev/platform/blackberryplatform/BlackBerryPlatformMessageClient.cpp:641
#10 0x7b079be8 in BlackBerry::Platform::MessageClient::processNextMessage
(this=0x13aeb70, timeout=&lt;optimized out&gt;)
    at
/home/anilsson/dev/platform/blackberryplatform/BlackBerryPlatformMessageClient.cpp:562
#11 0x7b079dbe in BlackBerry::Platform::MessageClient::exec (this=0x13aeb70)
    at
/home/anilsson/dev/platform/blackberryplatform/BlackBerryPlatformMessageClient.cpp:595
#12 0x797c57f2 in WebKitThread::exec (this=0x13aeb58) at
/home/anilsson/dev/libwebview/WebKitThread.cpp:195
#13 0x7b0775ba in BlackBerry::Platform::MessageClient::run (context=0x13aeb70)
    at
/home/anilsson/dev/platform/blackberryplatform/BlackBerryPlatformMessageClient.cpp:581
#14 0x01bafed0 in getgroups (gidsetsize=0, grouplist=0x0) at
../../getgroups.c:32
Backtrace stopped: previous frame inner to this frame (corrupt stack?)

(gdb) disassemble 
Dump of assembler code for function WebCore::Node::shadowAncestorNode() const:
=&gt; 0x7bc259f8 &lt;+0&gt;:    ldr    r3, [r0, #20]
   0x7bc259fa &lt;+2&gt;:    lsls    r1, r3, #25
   0x7bc259fc &lt;+4&gt;:    bmi.n    0x7bc25a18 &lt;WebCore::Node::shadowAncestorNode()
const+32&gt;
   0x7bc259fe &lt;+6&gt;:    tst.w    r3, #65536    ; 0x10000
   0x7bc25a02 &lt;+10&gt;:    mov    r3, r0
   0x7bc25a04 &lt;+12&gt;:    beq.n    0x7bc25a0e
&lt;WebCore::Node::shadowAncestorNode() const+22&gt;
   0x7bc25a06 &lt;+14&gt;:    b.n    0x7bc25a16 &lt;WebCore::Node::shadowAncestorNode()
const+30&gt;
   0x7bc25a08 &lt;+16&gt;:    ldr    r2, [r3, #20]
   0x7bc25a0a &lt;+18&gt;:    lsls    r2, r2, #15
   0x7bc25a0c &lt;+20&gt;:    bmi.n    0x7bc25a16
&lt;WebCore::Node::shadowAncestorNode() const+30&gt;
   0x7bc25a0e &lt;+22&gt;:    ldr    r3, [r3, #12]
   0x7bc25a10 &lt;+24&gt;:    cmp    r3, #0
   0x7bc25a12 &lt;+26&gt;:    bne.n    0x7bc25a08
&lt;WebCore::Node::shadowAncestorNode() const+16&gt;
   0x7bc25a14 &lt;+28&gt;:    bx    lr
   0x7bc25a16 &lt;+30&gt;:    ldr    r0, [r3, #12]
   0x7bc25a18 &lt;+32&gt;:    bx    lr
End of assembler dump.
(gdb) info registers 
r0             0x0    0
r1             0x16d75e0    23950816
r2             0x1    1
r3             0x8    8
r4             0x0    0
r5             0x21    33
r6             0x1476360    21455712
r7             0x16d74d0    23950544
r8             0x0    0
r9             0x0    0
r10            0x16d7594    23950740
r11            0x18db988    26065288
r12            0x58    88
sp             0x16d7478    0x16d7478
lr             0x7bc85521    2076726561
pc             0x7bc259f8    0x7bc259f8 &lt;WebCore::Node::shadowAncestorNode()
const&gt;
cpsr           0x40000030    1073741872</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>649806</commentid>
    <comment_count>1</comment_count>
      <attachid>147717</attachid>
    <who name="Sean Wang">xuewen.ok</who>
    <bug_when>2012-06-14 20:19:50 -0700</bug_when>
    <thetext>Created attachment 147717
patch

It has been internal reviewed by Antonio Gomes</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>649910</commentid>
    <comment_count>2</comment_count>
      <attachid>147717</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-06-14 23:16:28 -0700</bug_when>
    <thetext>Comment on attachment 147717
patch

Clearing flags on attachment: 147717

Committed r120404: &lt;http://trac.webkit.org/changeset/120404&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>649912</commentid>
    <comment_count>3</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-06-14 23:16:32 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>650045</commentid>
    <comment_count>4</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-06-15 02:52:45 -0700</bug_when>
    <thetext>Re-opened since this is blocked by 89193</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>663753</commentid>
    <comment_count>5</comment_count>
    <who name="Rob Buis">rwlbuis</who>
    <bug_when>2012-07-05 15:09:07 -0700</bug_when>
    <thetext>(In reply to comment #4)
&gt; Re-opened since this is blocked by 89193

That one is reported as fixed, so this one can be closed again?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>663794</commentid>
    <comment_count>6</comment_count>
    <who name="Sean Wang">xuewen.ok</who>
    <bug_when>2012-07-05 17:24:51 -0700</bug_when>
    <thetext>This issue has been luckily fixed by unknown patches. So close it.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>147717</attachid>
            <date>2012-06-14 20:19:50 -0700</date>
            <delta_ts>2012-06-14 23:16:28 -0700</delta_ts>
            <desc>patch</desc>
            <filename>patch</filename>
            <type>text/plain</type>
            <size>2587</size>
            <attacher name="Sean Wang">xuewen.ok</attacher>
            
              <data encoding="base64">RnJvbSA5OGJjOTI2NzkzYzBhNDIzMWUxNjlkYTE2ODgxZTNmNzliZGZhMTc2IE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBTZWFuIFdhbmcgPFh1ZXdlbi5XYW5nQHRvcmNobW9iaWxlLmNv
bS5jbj4KRGF0ZTogRnJpLCAxNSBKdW4gMjAxMiAxMTowMTozNSArMDgwMApTdWJqZWN0OiBbUEFU
Q0hdIFtCbGFja0JlcnJ5XSBTZWxlY3Rpb24gLSBDcmFzaCB3aGVuIG1hbmlwdWxhdGluZyBzZWxl
Y3Rpb24KIGJ5IGRyYWdnaW5nIGhhbmRsZQogaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i
dWcuY2dpP2lkPTg5MTYwCgpSSU1CVUc6MTY0OTcwCkF2b2lkIHRvIHNldCBwb3NpdGlvbiBpbiBz
aGFkb3cgdHJlZSB0byB0aGUgbmV3IHNlbGVjdGlvbidzIGJhc2UuIEl0CmNyYXNoZXMgd2hlbiBz
ZXR0aW5nIGEgc2hhZG93IHBvc2l0aW9uIHRvIGEgc2VsZWN0aW9uJyBiYXNlLgoKUmV2aWV3ZWQg
YnkgQW50b25pbyBHb21lcy4KCiogV2ViS2l0U3VwcG9ydC9TZWxlY3Rpb25IYW5kbGVyLmNwcDoK
KEJsYWNrQmVycnk6OldlYktpdDo6dmlzaWJsZVBvc2l0aW9uRm9yUG9pbnRJZ25vcmluZ0NsaXBw
aW5nKToKLS0tCiBTb3VyY2UvV2ViS2l0L2JsYWNrYmVycnkvQ2hhbmdlTG9nICAgICAgICAgICAg
ICAgICB8ICAgMTQgKysrKysrKysrKysrKysKIC4uLi9ibGFja2JlcnJ5L1dlYktpdFN1cHBvcnQv
U2VsZWN0aW9uSGFuZGxlci5jcHAgIHwgICAgMiArLQogMiBmaWxlcyBjaGFuZ2VkLCAxNSBpbnNl
cnRpb25zKCspLCAxIGRlbGV0aW9uKC0pCgpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYktpdC9ibGFj
a2JlcnJ5L0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQvYmxhY2tiZXJyeS9DaGFuZ2VMb2cKaW5k
ZXggZTcwNGRhZS4uNjBjYTM0MiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdC9ibGFja2JlcnJ5
L0NoYW5nZUxvZworKysgYi9Tb3VyY2UvV2ViS2l0L2JsYWNrYmVycnkvQ2hhbmdlTG9nCkBAIC0x
LDMgKzEsMTcgQEAKKzIwMTItMDYtMTQgIFNlYW4gV2FuZyAgPFh1ZXdlbi5XYW5nQHRvcmNobW9i
aWxlLmNvbS5jbj4KKworICAgICAgICBbQmxhY2tCZXJyeV0gU2VsZWN0aW9uIC0gQ3Jhc2ggd2hl
biBtYW5pcHVsYXRpbmcgc2VsZWN0aW9uIGJ5IGRyYWdnaW5nIGhhbmRsZQorICAgICAgICBodHRw
czovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9ODkxNjAKKworICAgICAgICBSSU1C
VUc6MTY0OTcwCisgICAgICAgIEF2b2lkIHRvIHNldCBwb3NpdGlvbiBpbiBzaGFkb3cgdHJlZSB0
byB0aGUgbmV3IHNlbGVjdGlvbidzIGJhc2UuIEl0CisgICAgICAgIGNyYXNoZXMgd2hlbiBzZXR0
aW5nIGEgc2hhZG93IHBvc2l0aW9uIHRvIGEgc2VsZWN0aW9uJyBiYXNlLgorCisgICAgICAgIFJl
dmlld2VkIGJ5IEFudG9uaW8gR29tZXMuCisKKyAgICAgICAgKiBXZWJLaXRTdXBwb3J0L1NlbGVj
dGlvbkhhbmRsZXIuY3BwOgorICAgICAgICAoQmxhY2tCZXJyeTo6V2ViS2l0Ojp2aXNpYmxlUG9z
aXRpb25Gb3JQb2ludElnbm9yaW5nQ2xpcHBpbmcpOgorCiAyMDEyLTA2LTE0ICBHZW5ldmlldmUg
TWFrICA8Z21ha0ByaW0uY29tPgogCiAgICAgICAgIEFsd2F5cyBjb252ZXJ0IHRvdWNoIGV2ZW50
cyB0byBtb3VzZSBldmVudHMgaWYgdGhlCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L2JsYWNr
YmVycnkvV2ViS2l0U3VwcG9ydC9TZWxlY3Rpb25IYW5kbGVyLmNwcCBiL1NvdXJjZS9XZWJLaXQv
YmxhY2tiZXJyeS9XZWJLaXRTdXBwb3J0L1NlbGVjdGlvbkhhbmRsZXIuY3BwCmluZGV4IDUxYzli
NzIuLjlkY2VjMWUgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvYmxhY2tiZXJyeS9XZWJLaXRT
dXBwb3J0L1NlbGVjdGlvbkhhbmRsZXIuY3BwCisrKyBiL1NvdXJjZS9XZWJLaXQvYmxhY2tiZXJy
eS9XZWJLaXRTdXBwb3J0L1NlbGVjdGlvbkhhbmRsZXIuY3BwCkBAIC0xNTIsNyArMTUyLDcgQEAg
c3RhdGljIFZpc2libGVQb3NpdGlvbiB2aXNpYmxlUG9zaXRpb25Gb3JQb2ludElnbm9yaW5nQ2xp
cHBpbmcoY29uc3QgRnJhbWUmIGZyYW0KICAgICAvLyBvdXRzaWRlIHRoZSB2aXNpYmxlIHJlY3Qu
IFRvIHdvcmsgYXJvdW5kIHRoZSBidWcsIHRoaXMgaXMgYSBjb3B5IG9mCiAgICAgLy8gdmlzaWJs
ZVBvc2l0aW9uQXRQb2ludCB3aGljaCB3aGljaCBwYXNzZXMgaWdub3JlQ2xpcHBpbmc9dHJ1ZS4K
ICAgICAvLyBTZWUgUklNIEJ1ZyAjNDMxNS4KLSAgICBIaXRUZXN0UmVzdWx0IHJlc3VsdCA9IGZy
YW1lLmV2ZW50SGFuZGxlcigpLT5oaXRUZXN0UmVzdWx0QXRQb2ludChmcmFtZVBvaW50LCB0cnVl
IC8qIGFsbG93U2hhZG93Q29udGVudCAqLywgdHJ1ZSAvKiBpZ25vcmVDbGlwcGluZyAqLyk7Cisg
ICAgSGl0VGVzdFJlc3VsdCByZXN1bHQgPSBmcmFtZS5ldmVudEhhbmRsZXIoKS0+aGl0VGVzdFJl
c3VsdEF0UG9pbnQoZnJhbWVQb2ludCwgZmFsc2UgLyogYWxsb3dTaGFkb3dDb250ZW50ICovLCB0
cnVlIC8qIGlnbm9yZUNsaXBwaW5nICovKTsKIAogICAgIE5vZGUqIG5vZGUgPSByZXN1bHQuaW5u
ZXJOb2RlKCk7CiAgICAgaWYgKCFub2RlIHx8IG5vZGUtPmRvY3VtZW50KCkgIT0gZnJhbWUuZG9j
dW1lbnQoKSkKLS0gCjEuNy45LjUKCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>