<?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>123464</bug_id>
          
          <creation_ts>2013-10-29 13:58:14 -0700</creation_ts>
          <short_desc>Node shouldn&apos;t be selected if it&apos;s not a draggable element and has -webkit-user-select: none set</short_desc>
          <delta_ts>2025-05-30 16:25:00 -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>DOM</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>DUPLICATE</resolution>
          <dup_id>208682</dup_id>
          
          <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="Crystal Zhang">haizhang</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>ahmad.saleem792</cc>
    
    <cc>ntim</cc>
    
    <cc>staikos</cc>
    
    <cc>tonikitoo</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>944590</commentid>
    <comment_count>0</comment_count>
    <who name="Crystal Zhang">haizhang</who>
    <bug_when>2013-10-29 13:58:14 -0700</bug_when>
    <thetext>As the comment says, if it&apos;s a draggable element, then it should be allowed to be selected, so drag &amp; drop action can be performed, if it&apos;s not, and has -webkit-user-select: none set, then it shouldn&apos;t be allowed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>944603</commentid>
    <comment_count>1</comment_count>
      <attachid>215425</attachid>
    <who name="Crystal Zhang">haizhang</who>
    <bug_when>2013-10-29 14:10:31 -0700</bug_when>
    <thetext>Created attachment 215425
patch

patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>944604</commentid>
    <comment_count>2</comment_count>
    <who name="Crystal Zhang">haizhang</who>
    <bug_when>2013-10-29 14:16:12 -0700</bug_when>
    <thetext>Add Antonio to cc list.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>944618</commentid>
    <comment_count>3</comment_count>
      <attachid>215425</attachid>
    <who name="Crystal Zhang">haizhang</who>
    <bug_when>2013-10-29 14:42:23 -0700</bug_when>
    <thetext>Comment on attachment 215425
patch

Need have a second thought.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2033209</commentid>
    <comment_count>4</comment_count>
    <who name="Ahmad Saleem">ahmad.saleem792</who>
    <bug_when>2024-05-03 16:42:13 -0700</bug_when>
    <thetext>This patch was modifying this:

https://searchfox.org/wubkat/rev/3f1a725ef3d3f0a0ee05b56aff41fbfdae90f34f/Source/WebCore/dom/Node.cpp#1288

- It seems blink did the change:

https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/core/dom/node.cc;l=1654;drc=5180d4deb2bee0cc22d14ced5ac540e51135a2e5

** Blink:

    // We allow selections to begin within |user-select: text/all| sub trees
    // but not if the element is draggable.
    if (style.UserDrag() != EUserDrag::kElement &amp;&amp;
        (user_select == EUserSelect::kText || user_select == EUserSelect::kAll))
      return true;

** WebKit:

        // We allow selections to begin within an element that has -webkit-user-select: none set,
        // but if the element is draggable then dragging should take priority over selection.
        if (style.userDrag() == UserDrag::Element &amp;&amp; style.usedUserSelect() == UserSelect::None)
            return false;
    }

__</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2120827</commentid>
    <comment_count>5</comment_count>
    <who name="Tim Nguyen (:ntim)">ntim</who>
    <bug_when>2025-05-30 16:25:00 -0700</bug_when>
    <thetext>

*** This bug has been marked as a duplicate of bug 208682 ***</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>215425</attachid>
            <date>2013-10-29 14:10:31 -0700</date>
            <delta_ts>2013-10-29 14:42:23 -0700</delta_ts>
            <desc>patch</desc>
            <filename>0001-2013-10-29-Crystal-Zhang-haizhang-blackberry.com.patch</filename>
            <type>text/plain</type>
            <size>2553</size>
            <attacher name="Crystal Zhang">haizhang</attacher>
            
              <data encoding="base64">RnJvbSBhMzEwYTEzYjkyNmEzNjRiZGI2NWUwNmI0NjQ0NjhjNGQ1MjY3ZTcwIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBDcnlzdGFsIFpoYW5nIDxoYWl6aGFuZ0BibGFja2JlcnJ5LmNv
bT4KRGF0ZTogVHVlLCAyOSBPY3QgMjAxMyAxNzowNTowNCAtMDQwMApTdWJqZWN0OiBbUEFUQ0hd
IDIwMTMtMTAtMjkgIENyeXN0YWwgWmhhbmcgIDxoYWl6aGFuZ0BibGFja2JlcnJ5LmNvbT4KCiAg
ICAgICAgTm9kZSBzaG91bGRuJ3QgYmUgc2VsZWN0ZWQgaWYgaXQncyBub3QgYSBkcmFnZ2FibGUg
ZWxlbWVudCBhbmQgaGFzIC13ZWJraXQtdXNlci1zZWxlY3Q6IG5vbmUgc2V0LgogICAgICAgIGh0
dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMjM0NjQKCiAgICAgICAgUmV2
aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCgogICAgICAgIEFzIHRoZSBjb21tZW50IHNheXMsIGlm
IGl0J3MgYSBkcmFnZ2FibGUgZWxlbWVudCwgdGhlbiBpdCBzaG91bGQgYmUgYWxsb3dlZCB0byBi
ZSBzZWxlY3RlZCwgc28KICAgICAgICBkcmFnICYgZHJvcCBhY3Rpb24gY2FuIGJlIHBlcmZvcm1l
ZCwgaWYgaXQncyBub3QsIGFuZCBoYXMgLXdlYmtpdC11c2VyLXNlbGVjdDogbm9uZSBzZXQsIHRo
ZW4gaXQKICAgICAgICBzaG91bGRuJ3QgYmUgYWxsb3dlZC4KCiAgICAgICAgSklSQSAxNDM1MS4K
CiAgICAgICAgKiBkb20vTm9kZS5jcHA6CiAgICAgICAgKFdlYkNvcmU6Ok5vZGU6OmNhblN0YXJ0
U2VsZWN0aW9uKToKLS0tCiBTb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cgICAgfCAxNCArKysrKysr
KysrKysrKwogU291cmNlL1dlYkNvcmUvZG9tL05vZGUuY3BwIHwgIDIgKy0KIDIgZmlsZXMgY2hh
bmdlZCwgMTUgaW5zZXJ0aW9ucygrKSwgMSBkZWxldGlvbigtKQoKZGlmZiAtLWdpdCBhL1NvdXJj
ZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwppbmRleCBhMmQ5
YzE5Li40NTY2NGE1IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIv
U291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTcgQEAKKzIwMTMtMTAtMjkgIENy
eXN0YWwgWmhhbmcgIDxoYWl6aGFuZ0BibGFja2JlcnJ5LmNvbT4KKworICAgICAgICBOb2RlIHNo
b3VsZG4ndCBiZSBzZWxlY3RlZCBpZiBpdCdzIG5vdCBhIGRyYWdnYWJsZSBlbGVtZW50IGFuZCBo
YXMgLXdlYmtpdC11c2VyLXNlbGVjdDogbm9uZSBzZXQuCisgICAgICAgIGh0dHBzOi8vYnVncy53
ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMjM0NjQKKworICAgICAgICBSZXZpZXdlZCBieSBO
T0JPRFkgKE9PUFMhKS4KKworICAgICAgICBBcyB0aGUgY29tbWVudCBzYXlzLCBpZiBpdCdzIGEg
ZHJhZ2dhYmxlIGVsZW1lbnQsIHRoZW4gaXQgc2hvdWxkIGJlIGFsbG93ZWQgdG8gYmUgc2VsZWN0
ZWQsIHNvCisgICAgICAgIGRyYWcgJiBkcm9wIGFjdGlvbiBjYW4gYmUgcGVyZm9ybWVkLCBpZiBp
dCdzIG5vdCwgYW5kIGhhcyAtd2Via2l0LXVzZXItc2VsZWN0OiBub25lIHNldCwgdGhlbiBpdAor
ICAgICAgICBzaG91bGRuJ3QgYmUgYWxsb3dlZC4KKworICAgICAgICAqIGRvbS9Ob2RlLmNwcDoK
KyAgICAgICAgKFdlYkNvcmU6Ok5vZGU6OmNhblN0YXJ0U2VsZWN0aW9uKToKKwogMjAxMy0xMC0x
NiAgQW50dGkgS29pdmlzdG8gIDxhbnR0aUBhcHBsZS5jb20+CiAKICAgICAgICAgTW92ZSBjb2Rl
IGZvciBmaW5kaW5nIHJlbmRlcmVkIGNoYXJhY3RlciBvZmZzZXQgdG8gUmVuZGVyVGV4dExpbmVC
b3hlcwpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvZG9tL05vZGUuY3BwIGIvU291cmNlL1dl
YkNvcmUvZG9tL05vZGUuY3BwCmluZGV4IGZmM2IzOTIuLjEzM2YxZWQgMTAwNjQ0Ci0tLSBhL1Nv
dXJjZS9XZWJDb3JlL2RvbS9Ob2RlLmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9kb20vTm9kZS5j
cHAKQEAgLTkzOCw3ICs5MzgsNyBAQCBib29sIE5vZGU6OmNhblN0YXJ0U2VsZWN0aW9uKCkgY29u
c3QKICAgICAgICAgUmVuZGVyU3R5bGUqIHN0eWxlID0gcmVuZGVyZXIoKS0+c3R5bGUoKTsKICAg
ICAgICAgLy8gV2UgYWxsb3cgc2VsZWN0aW9ucyB0byBiZWdpbiB3aXRoaW4gYW4gZWxlbWVudCB0
aGF0IGhhcyAtd2Via2l0LXVzZXItc2VsZWN0OiBub25lIHNldCwKICAgICAgICAgLy8gYnV0IGlm
IHRoZSBlbGVtZW50IGlzIGRyYWdnYWJsZSB0aGVuIGRyYWdnaW5nIHNob3VsZCB0YWtlIHByaW9y
aXR5IG92ZXIgc2VsZWN0aW9uLgotICAgICAgICBpZiAoc3R5bGUtPnVzZXJEcmFnKCkgPT0gRFJB
R19FTEVNRU5UICYmIHN0eWxlLT51c2VyU2VsZWN0KCkgPT0gU0VMRUNUX05PTkUpCisgICAgICAg
IGlmIChzdHlsZS0+dXNlckRyYWcoKSAhPSBEUkFHX0VMRU1FTlQgJiYgc3R5bGUtPnVzZXJTZWxl
Y3QoKSA9PSBTRUxFQ1RfTk9ORSkKICAgICAgICAgICAgIHJldHVybiBmYWxzZTsKICAgICB9CiAg
ICAgcmV0dXJuIHBhcmVudE9yU2hhZG93SG9zdE5vZGUoKSA/IHBhcmVudE9yU2hhZG93SG9zdE5v
ZGUoKS0+Y2FuU3RhcnRTZWxlY3Rpb24oKSA6IHRydWU7Ci0tIAoxLjguNAoK
</data>

          </attachment>
      

    </bug>

</bugzilla>