Closed
Bug 549097
Opened 14 years ago
Closed 14 years ago
Multiple location bar results appear selected when scrolling down
Categories
(Core :: Graphics, defect)
Tracking
()
VERIFIED
FIXED
People
(Reporter: u88484, Assigned: bas.schouten)
References
Details
(Keywords: regression)
Attachments
(2 files, 2 obsolete files)
958.21 KB,
image/png
|
Details | |
954 bytes,
patch
|
jrmuizel
:
review+
|
Details | Diff | Splinter Review |
Multiple location bar results appear selected when scrolling down STR: 1) Type g in location bar (or some other letter that you know will show a lot of results) 2) Press the down arrow key until the results begin to scroll 3) Note multiple results showing as being selected This does not happen in the downloads manager. Windows 7 64-bit NVIDIA GeForce 7150M / nForce 630M Drivers: 7.15.11.7967
Comment 1•14 years ago
|
||
Are they actually selected, or do they just appear as though they are?
Just appear. The last selection is what opens up when the enter key is pressed.
I can confirm this. Only seeing this after enabling Direct2D rendering. Windows 7 x64 ATI Radeon HD5770 Driver 8.690.0.0
Could this bug be related to multiple ctrl+tab multiple selections? http://img30.imageshack.us/img30/3489/ctrltab.jpg
Assignee | ||
Comment 5•14 years ago
|
||
So, this is interesting, the window is scrolled up correctly, but it seems the part of the window containing the URL that used to be selected is not invalidated, so the unselected URL is never drawn there. I don't have a good idea yet what causes this.
Assignee | ||
Comment 6•14 years ago
|
||
Add a screenshot of the effect.
Assignee: nobody → bas.schouten
Status: NEW → ASSIGNED
Updated•14 years ago
|
Status: ASSIGNED → NEW
Assignee | ||
Comment 7•14 years ago
|
||
This patch fixes the invalidation when scrolling, by doing what ScrollWindowEx seems to do for the updateRgn. That is, it 'smears' the invalidation area with the scroll. I essentially takes the union of the current update region and the update region moves by the scroll.
Attachment #430168 -
Flags: review?(jmuizelaar)
Can't you just move the region?
Assignee | ||
Comment 10•14 years ago
|
||
I believe that roc's right looking at our code. It doesn't seem to be what ScrollWindowEx does. But it seems to work just fine in all situations I tested, so I suggest we do this.
Attachment #430168 -
Attachment is obsolete: true
Attachment #430168 -
Flags: review?(jmuizelaar)
Assignee | ||
Updated•14 years ago
|
Attachment #430186 -
Flags: review?(jmuizelaar)
Comment 11•14 years ago
|
||
(In reply to comment #10) > Created an attachment (id=430186) [details] > Fix scrolling invalidation for D2D v2 > > I believe that roc's right looking at our code. It doesn't seem to be what > ScrollWindowEx does. But it seems to work just fine in all situations I tested, > so I suggest we do this. Can't we just use OffsetRgn instead of doing this by hand?
Updated•14 years ago
|
Attachment #430186 -
Flags: review?(jmuizelaar) → review-
Assignee | ||
Comment 12•14 years ago
|
||
Thanks for pointing me at the GDI function I couldn't find!!
Attachment #430186 -
Attachment is obsolete: true
Attachment #430343 -
Flags: review?(jmuizelaar)
Updated•14 years ago
|
Attachment #430343 -
Flags: review?(jmuizelaar) → review+
Assignee | ||
Comment 13•14 years ago
|
||
Pushed, changeset 103b23e9a222.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Comment 14•14 years ago
|
||
Verified fixed using: Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.3a3pre) Gecko/20100304 Minefield/3.7a3pre Firefox/3.6 ID:20100304141831 <latest hourly cset: 20100304141831 5125fc26166b
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•