<?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>29981</bug_id>
          
          <creation_ts>2009-10-01 14:08:40 -0700</creation_ts>
          <short_desc>REGRESSION (r45945): After pressing Shift-PageDown, pressing Shift-Up extends the top of the selection upwards (but should shrink the selection instead)</short_desc>
          <delta_ts>2009-10-27 11:54:24 -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>HTML Editing</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc>data:text/html,%3Ctextarea%20rows=40%3Etest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%3C/textarea%3E</bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar, Regression</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Adam Roben (:aroben)">aroben</reporter>
          <assigned_to name="Enrica Casucci">enrica</assigned_to>
          <cc>adele</cc>
    
    <cc>darin</cc>
    
    <cc>enrica</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>151690</commentid>
    <comment_count>0</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2009-10-01 14:08:40 -0700</bug_when>
    <thetext>To reproduce:

1. Go to data:text/html,%3Ctextarea%20rows=40%3Etest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%0Atest%3C/textarea%3E
2. Click at least one line below the top of the textarea
3. Press Shift-PageDown
4. Press Shift-Up

The top of the selection extends upwards. But the bottom of the extension should shrink upwards, instead.

This happens on both Mac and Windows. It did not happen in Safari 4.0.3.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>151692</commentid>
    <comment_count>1</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2009-10-01 14:09:13 -0700</bug_when>
    <thetext>&lt;rdar://problem/7269075&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>157188</commentid>
    <comment_count>2</comment_count>
      <attachid>41733</attachid>
    <who name="Enrica Casucci">enrica</who>
    <bug_when>2009-10-23 11:08:19 -0700</bug_when>
    <thetext>Created attachment 41733
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>157232</commentid>
    <comment_count>3</comment_count>
      <attachid>41733</attachid>
    <who name="Adele Peterson">adele</who>
    <bug_when>2009-10-23 12:02:23 -0700</bug_when>
    <thetext>Comment on attachment 41733
Patch

I think it should be possible to add a layout test for this.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>157380</commentid>
    <comment_count>4</comment_count>
      <attachid>41733</attachid>
    <who name="Adele Peterson">adele</who>
    <bug_when>2009-10-23 17:14:47 -0700</bug_when>
    <thetext>Comment on attachment 41733
Patch

Clearing the review patch, since Enrica is going to upload a new version w/ a test</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>157607</commentid>
    <comment_count>5</comment_count>
    <who name="Darin Adler">darin</who>
    <bug_when>2009-10-25 13:04:05 -0700</bug_when>
    <thetext>My bad. Fix looks good.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>158297</commentid>
    <comment_count>6</comment_count>
    <who name="Adele Peterson">adele</who>
    <bug_when>2009-10-27 11:54:24 -0700</bug_when>
    <thetext>I added a test case, and checked in revision 50165.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>41733</attachid>
            <date>2009-10-23 11:08:19 -0700</date>
            <delta_ts>2009-10-23 17:14:46 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug7269075.txt</filename>
            <type>text/plain</type>
            <size>1250</size>
            <attacher name="Enrica Casucci">enrica</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiA0OTk4MykKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTUgQEAKKzIwMDktMTAtMjMgIEVucmljYSBDYXN1Y2NpICA8ZW5yaWNhQGFwcGxl
LmNvbT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBS
RUdSRVNTSU9OIChTYWZhcmkgNC4wLjMtVG9UKTogQWZ0ZXIgcHJlc3NpbmcgU2hpZnQtUGFnZURv
d24sIHByZXNzaW5nIFNoaWZ0LVVwIAorICAgICAgICBleHRlbmRzIHRoZSB0b3Agb2YgdGhlIHNl
bGVjdGlvbiB1cHdhcmRzIChidXQgc2hvdWxkIHNocmluayB0aGUgc2VsZWN0aW9uIGluc3RlYWQp
CisgICAgICAgIDxyZGFyOi8vcHJvYmxlbS83MjY5MDc1PgorICAgICAgICBodHRwczovL2J1Z3Mu
d2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9Mjk5ODEKKworICAgICAgICAqIGVkaXRpbmcvU2Vs
ZWN0aW9uQ29udHJvbGxlci5jcHA6CisgICAgICAgIChXZWJDb3JlOjpTZWxlY3Rpb25Db250cm9s
bGVyOjptb2RpZnkpOiBBZGRlZCBzZXR0aW5nIG1fbGFzdENoYW5nZVdhc0hvcml6b250YWxFeHRl
bnNpb24uCisKIDIwMDktMTAtMjMgIFN0ZXZlIEJsb2NrICA8c3RldmVibG9ja0Bnb29nbGUuY29t
PgogCiAgICAgICAgIFJldmlld2VkIGJ5IEVyaWMgU2VpZGVsLgpJbmRleDogV2ViQ29yZS9lZGl0
aW5nL1NlbGVjdGlvbkNvbnRyb2xsZXIuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvZWRpdGlu
Zy9TZWxlY3Rpb25Db250cm9sbGVyLmNwcAkocmV2aXNpb24gNDk5NjApCisrKyBXZWJDb3JlL2Vk
aXRpbmcvU2VsZWN0aW9uQ29udHJvbGxlci5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTcyNSw2ICs3
MjUsOCBAQCBib29sIFNlbGVjdGlvbkNvbnRyb2xsZXI6Om1vZGlmeShFQWx0ZXJhCiAgICAgaWYg
KHVzZXJUcmlnZ2VyZWQpCiAgICAgICAgIG1fZnJhbWUtPnNldFNlbGVjdGlvbkdyYW51bGFyaXR5
KENoYXJhY3RlckdyYW51bGFyaXR5KTsKIAorICAgIG1fbGFzdENoYW5nZVdhc0hvcml6b250YWxF
eHRlbnNpb24gPSBhbHRlciA9PSBFWFRFTkQ7CisKICAgICByZXR1cm4gdHJ1ZTsKIH0KIAo=
</data>

          </attachment>
      

    </bug>

</bugzilla>