Closed
Bug 805697
Opened 12 years ago
Closed 12 years ago
Remove StCaretHider
Categories
(Core :: Layout, defect)
Tracking
()
RESOLVED
FIXED
mozilla19
People
(Reporter: roc, Assigned: roc)
References
Details
Attachments
(2 files)
(deleted),
patch
|
MatsPalmgren_bugz
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
MatsPalmgren_bugz
:
review+
|
Details | Diff | Splinter Review |
With the new painting setup, StCaretHider is completely worthless.
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #675404 -
Flags: review?(matspal)
Assignee | ||
Comment 2•12 years ago
|
||
Attachment #675405 -
Flags: review?(matspal)
Assignee | ||
Comment 3•12 years ago
|
||
More background: As far as I can tell, StCaretHider used to be needed because we painted the caret using some XOR magic ... a regular paint would not necessarily paint the caret correctly, and scrolling could mess things up somehow. But these days we only ever paint from the refresh driver so there's no way we could disturb the window during the scope of an StCaretHider.
Updated•12 years ago
|
Attachment #675404 -
Flags: review?(matspal) → review+
Comment 4•12 years ago
|
||
Comment on attachment 675405 [details] [diff] [review]
There is no point in checking for the existence of a caret in the presshell
Looks good to me, r=mats
I did notice one minor "regression" though for this content:
data:text/html,<input readonly value="value">body text
If you put the document in caret browsing mode (F7), and click
on "body text" (caret is displayed - ok), and then click on
"value" => caret does not move, so it looks like the <input>
isn't focused at all, although it is. If you type Shift+LEFT
to select some <input> text then the caret is erased.
"regression" in quotation marks because the above STR doesn't
really work perfectly now either... but with these patches it's
slightly worse. So please file a follow-up bug on this.
Also, it might be possible to remove nsCaret::mIgnoreUserModify
now. I think it was only used to workaround some paint issue
for "View Source" (in bug 394473).
Attachment #675405 -
Flags: review?(matspal) → review+
Assignee | ||
Comment 5•12 years ago
|
||
Filed bug 806278.
Assignee | ||
Comment 6•12 years ago
|
||
Comment 7•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/911658ea401e
https://hg.mozilla.org/mozilla-central/rev/f31f3948e433
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
You need to log in
before you can comment on or make changes to this bug.
Description
•