WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
10924
Layout test failures, extra WebViewDidChangeSelectionNotification sent
https://bugs.webkit.org/show_bug.cgi?id=10924
Summary
Layout test failures, extra WebViewDidChangeSelectionNotification sent
Timothy Hatcher
Reported
2006-09-18 16:43:27 PDT
These tests sent an extra WebViewDidChangeSelectionNotification notification when the selection changes: css2.1/t0801-c412-hz-box-00-b-a.html editing/pasteboard/copy-paste-bidi.html fast/dynamic/move-node-with-selection.html fast/events/dblclick-addEventListener.html fast/events/updateLayoutForHitTest.html EDITING DELEGATE: webViewDidChangeSelection:WebViewDidChangeSelectionNotification +EDITING DELEGATE: webViewDidChangeSelection:WebViewDidChangeSelectionNotification
Attachments
Add attachment
proposed patch, testcase, etc.
Adam Roben (:aroben)
Comment 1
2006-10-04 22:07:33 PDT
css2.1/t0801-c412-hz-box-00-b-a.html suffers from a separate problem (catalogued in
bug 11000
). Adittionally, fast/forms/display-none-in-onchange-keyboard.html seems to now be missing a call to this delegate notification.
Alice Liu
Comment 2
2006-10-17 12:15:21 PDT
These three tests have new results checked in: fast/dynamic/move-node-with-selection.html fast/events/dblclick-addEventListener.html fast/events/updateLayoutForHitTest.html these 2 are broken for different reasons, and separate bugs should be filed about them. css2.1/t0801-c412-hz-box-00-b-a.html editing/pasteboard/copy-paste-bidi.html resolving this bug since this bug has to do with the problem of extra calls to WebViewDidChangeSelectionNotification
Alice Liu
Comment 3
2006-10-17 12:16:35 PDT
r17084
Adam Roben (:aroben)
Comment 4
2006-10-18 01:05:10 PDT
The explanation, cribbed from Alice's ChangeLog: When nodes are removed, selections are cleared, and when
http://bugs.webkit.org/show_bug.cgi?id=6498
was fixed, we started sending didChangeSelection notifications.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug