<?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>4609</bug_id>
          
          <creation_ts>2005-08-23 17:40:43 -0700</creation_ts>
          <short_desc>window.getSelection().toString() is undefined (range object returned from window.getSelection() is useless)</short_desc>
          <delta_ts>2006-01-03 15:05:13 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>HTML Editing</component>
          <version>420+</version>
          <rep_platform>Mac</rep_platform>
          <op_sys>OS X 10.4</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc>http://concisity.net</bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Major</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Berndt Jung">berndtj</reporter>
          <assigned_to name="Justin Garcia">justin.garcia</assigned_to>
          <cc>ap</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>17299</commentid>
    <comment_count>0</comment_count>
    <who name="Berndt Jung">berndtj</who>
    <bug_when>2005-08-23 17:40:43 -0700</bug_when>
    <thetext>This should get the text of the selection&apos;s start container.  The mozilla version works, and the Safari 
version used to work (old safari).  The two examples below are meant as bookmarklets.  Safari will report 
that the returned object is a range object, but I cannot do anything with this object, such as get the start/
end containers/offsets.  Interestingly toString() does not work either, yet if it is concatinated with a string, 
it does work.

safari: &lt;a href=&quot;javascript:d=window.getSelection();alert(d.startContainer.data);&quot;&gt;getStartContainerText()
&lt;/a&gt;
mozilla: &lt;a href=&quot;javascript:d=window.getSelection().getRangeAt(0);alert
(d.startContainer.data);&quot;&gt;getStartContainerText()&lt;/a&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>17300</commentid>
    <comment_count>1</comment_count>
      <attachid>3542</attachid>
    <who name="Berndt Jung">berndtj</who>
    <bug_when>2005-08-23 17:45:36 -0700</bug_when>
    <thetext>Created attachment 3542
test cases

Drag the links to the bookmark bar to test</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>18989</commentid>
    <comment_count>2</comment_count>
    <who name="Oliver Hunt">oliver</who>
    <bug_when>2005-09-07 04:29:36 -0700</bug_when>
    <thetext>Nice test case, although interestingly ffx seems to be able to do both mozilla+safari mechanisms for at 
least one test</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>26470</commentid>
    <comment_count>3</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2005-12-24 04:28:10 -0800</bug_when>
    <thetext>Actually, window.getSelection() does not return a Range object - it returns a Selection object (with type 
None, Caret or Range), which doesn&apos;t have startContainer &lt;http://developer.mozilla.org/en/docs/
DOM:Selection&gt;.

The patch in bug 4904 adds support for getRangeAt, so the &quot;Mozilla&quot; way starts to work.

Keeping this bug to track the problem with toString().</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>26471</commentid>
    <comment_count>4</comment_count>
      <attachid>5259</attachid>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2005-12-24 05:09:19 -0800</bug_when>
    <thetext>Created attachment 5259
test case</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>27674</commentid>
    <comment_count>5</comment_count>
    <who name="Justin Garcia">justin.garcia</who>
    <bug_when>2006-01-03 15:05:13 -0800</bug_when>
    <thetext>Landing the toString fix.  Turns out it was defined as a method on our JS Selection object, but we didn&apos;t 
handle it in Selection::callAsFunction

getRangeAt and toString should now behave like they do in FF.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="0"
              isprivate="0"
          >
            <attachid>3542</attachid>
            <date>2005-08-23 17:45:36 -0700</date>
            <delta_ts>2005-12-24 05:09:19 -0800</delta_ts>
            <desc>test cases</desc>
            <filename>getSelection.htm</filename>
            <type>text/html</type>
            <size>1195</size>
            <attacher name="Berndt Jung">berndtj</attacher>
            
              <data encoding="base64">PCFET0NUWVBFIEhUTUwgUFVCTElDICItLy9XM0MvL0RURCBIVE1MIDQuMDEgVHJhbnNpdGlvbmFs
Ly9FTiINImh0dHA6Ly93d3cudzMub3JnL1RSL2h0bWw0L2xvb3NlLmR0ZCI+DTxodG1sPg08aGVh
ZD4NPHRpdGxlPlVudGl0bGVkIERvY3VtZW50PC90aXRsZT4NPG1ldGEgaHR0cC1lcXVpdj0iQ29u
dGVudC1UeXBlIiBjb250ZW50PSJ0ZXh0L2h0bWw7IGNoYXJzZXQ9dXRmLTgiPg08L2hlYWQ+DQ08
Ym9keT4NPGRpdj48cD5CZWxvdyBhcmUgc29tZSBib29rbWFya2xldHM6PC9wPg0JPHA+VGhpcyBz
aG91bGQgZ3JhYiB0aGUgdGV4dCB5b3UgaGF2ZSBzZWxlY3RlZDogKHdvcmtzIGluIHNhZmFyaSk8
L3A+DQk8cD5zYWZhcmk6IDxhIGhyZWY9ImphdmFzY3JpcHQ6ZD13aW5kb3cuZ2V0U2VsZWN0aW9u
KCk7YWxlcnQoZCk7Ij5nZXRTZWxlY3Rpb25UZXh0KCk8L2E+PGJyIC8+bW96aWxsYTogPGEgaHJl
Zj0iamF2YXNjcmlwdDpkPXdpbmRvdy5nZXRTZWxlY3Rpb24oKS5nZXRSYW5nZUF0KDApO2FsZXJ0
KGQpOyI+Z2V0U2VsZWN0aW9uVGV4dCgpPC9hPjwvcD4NCTxwPlRoaXMgdmVyaWZpZXMgdGhhdCB0
aGUgc2VsZWN0ZWQgdGV4dCBpcyBhIHJhbmdlIG9iamVjdDogKHdvcmtzIGluIHNhZmFyaSk8L3A+
DQk8cD5zYWZhcmk6IDxhIGhyZWY9ImphdmFzY3JpcHQ6ZD13aW5kb3cuZ2V0U2VsZWN0aW9uKCk7
YWxlcnQoZC50eXBlKTsiPnZlcmlmeVJhbmdlKCk8L2E+PGJyIC8+bW96aWxsYTogPGEgaHJlZj0i
amF2YXNjcmlwdDpkPXdpbmRvdy5nZXRTZWxlY3Rpb24oKS5nZXRSYW5nZUF0KDApO2FsZXJ0KGQu
dHlwZSk7Ij52ZXJpZnlSYW5nZSgpPC9hPjwvcD4NCTxwPlRoaXMgc2hvdWxkIGdldCB0aGUgdGV4
dCBvZiB0aGUgc2VsZWN0aW9uJiM4MjE3O3Mgc3RhcnQgY29udGFpbmVyOiAoYnJva2VuKTwvcD4N
CTxwPnNhZmFyaTogPGEgaHJlZj0iamF2YXNjcmlwdDpkPXdpbmRvdy5nZXRTZWxlY3Rpb24oKTth
bGVydChkLnN0YXJ0Q29udGFpbmVyLmRhdGEpOyI+Z2V0U3RhcnRDb250YWluZXJUZXh0KCk8L2E+
PGJyIC8+DSAgbW96aWxsYTogPGEgaHJlZj0iamF2YXNjcmlwdDpkPXdpbmRvdy5nZXRTZWxlY3Rp
b24oKS5nZXRSYW5nZUF0KDApO2FsZXJ0KGQuc3RhcnRDb250YWluZXIuZGF0YSk7Ij5nZXRTdGFy
dENvbnRhaW5lclRleHQoKTwvYT48L3A+DTwvZGl2Pg08L2Rpdj4NPC9ib2R5Pg08L2h0bWw+DQ==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>5259</attachid>
            <date>2005-12-24 05:09:19 -0800</date>
            <delta_ts>2005-12-24 05:09:19 -0800</delta_ts>
            <desc>test case</desc>
            <filename>selection-tostring.html</filename>
            <type>text/html</type>
            <size>608</size>
            <attacher name="Alexey Proskuryakov">ap</attacher>
            
              <data encoding="base64">PGh0bWw+IAo8aGVhZD4KCjxzY3JpcHQ+Cgo8L3NjcmlwdD4KCjx0aXRsZT53aW5kb3cuZ2V0U2Vs
ZWN0aW9uKCkudG9TdHJpbmcoKTwvdGl0bGU+IAo8L2hlYWQ+IAo8Ym9keSBpZD10PgpIZWxsbwo8
c2NyaXB0PgoKICAgIHRyeSB7IC8vIEZpcmVmb3ggd2F5CiAgICAgIHJhbmdlID0gZG9jdW1lbnQu
Y3JlYXRlUmFuZ2UoKTsKICAgICAgcmFuZ2Uuc2VsZWN0Tm9kZShkb2N1bWVudC5nZXRFbGVtZW50
QnlJZCgidCIpKTsKICAgICAgd2luZG93LmdldFNlbGVjdGlvbigpLmFkZFJhbmdlKHJhbmdlKTsK
ICAgIH0gY2F0Y2ggKGV4KSB7fQogICAgCiAgICB0cnkgeyAvLyBTYWZhcmkgd2F5CiAgICAgIHdp
bmRvdy5nZXRTZWxlY3Rpb24oKS5zZXRCYXNlQW5kRXh0ZW50KGRvY3VtZW50LmdldEVsZW1lbnRC
eUlkKCJ0IiksIDAsIGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCJ0IiksIDEpOwogICAgfSBjYXRj
aCAoZXgpIHt9CiAgICAKICAgIGRvY3VtZW50LndyaXRlKCI8cD53aW5kb3cuZ2V0U2VsZWN0aW9u
KCkudG9TdHJpbmcoKTogIiArIHdpbmRvdy5nZXRTZWxlY3Rpb24oKS50b1N0cmluZygpICsgIjwv
cD4iKTsKICAgIAo8L3NjcmlwdD4KCjwvYm9keT4KPC9odG1sPgo=
</data>

          </attachment>
      

    </bug>

</bugzilla>