Bug 105757 - [BlackBerry] Caret doesn't update position when scrolling text in Input element
Summary: [BlackBerry] Caret doesn't update position when scrolling text in Input element
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit BlackBerry (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-12-26 00:39 PST by Sean Wang
Modified: 2012-12-26 09:01 PST (History)
5 users (show)

See Also:

Attachments
patch (2.47 KB, patch)
2012-12-26 08:25 PST, Sean Wang 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Sean Wang 2012-12-26 00:39:16 PST 
When scrolling text in a input element, the text position is updated, but the caret is still at the origin position.

HOW TO REPRODUCE:
1. Input a long text in the input element to make the text exceed the input visible length.
2. Tap on the text to trigger the caret.
3. Touch Scroll on the text

Expects:
The Text and the caret should be scrolled synchronously.

Actual:
The text position is scrolled, the caret is still at the origin position.
Comment 1 Sean Wang 2012-12-26 08:25:14 PST 
Created attachment 180750 [details]
patch

Call FrameSelection::updateAppearence() instead of SelectionHandler::selectionPositionChanged(). The updateAppearence will call selectionPositionChanged at last.
Comment 2 WebKit Review Bot 2012-12-26 09:01:45 PST 
Comment on attachment 180750 [details]
patch

Clearing flags on attachment: 180750

Committed r138478: <http://trac.webkit.org/changeset/138478>
Comment 3 WebKit Review Bot 2012-12-26 09:01:49 PST 
All reviewed patches have been landed.  Closing bug.