Closed
Bug 545049
Opened 15 years ago
Closed 15 years ago
When dragging in the bookmarks toolbar sometimes the drop indicator does not appear.
Categories
(Core :: Web Painting, defect, P2)
Core
Web Painting
Tracking
()
VERIFIED
FIXED
mozilla1.9.3a4
Tracking | Status | |
---|---|---|
blocking2.0 | --- | final+ |
People
(Reporter: mak, Assigned: roc)
References
Details
(Keywords: regression)
Attachments
(3 files)
(deleted),
text/html
|
Details | |
(deleted),
patch
|
MatsPalmgren_bugz
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
MatsPalmgren_bugz
:
review+
|
Details | Diff | Splinter Review |
STRs:
- new profile
- drag Getting started from its position towards Latest Headlines.
expected
- the drop indicator should appear before Getting started
actual:
- the drop indicator does not appear
dragging back to the start of the toolbar and then forward again makes it appear again.
i noticed this while working on bug 544047. So the drop indicator has 2 regressions, but they are not the same.
regrange:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=120667a01fd2&tochange=a43e2f7eda8f
Excluding the TM merge (for now) the most plausible bugs are bug 526394 or bug
529670.
I've checked that our code is acting correctly, but looks like the indicator even if it's in the right position is not drawn on screen.
Still investigating to reduce the range further.
Reporter | ||
Comment 3•15 years ago
|
||
restricted between 8c1587eac4b2 (works, Part18) and 47cd92d616d7 (broken, Part36)
Updated•15 years ago
|
Assignee: mak77 → nobody
Component: Bookmarks & History → Layout: View Rendering
Product: Firefox → Core
QA Contact: bookmarks → layout.view-rendering
Reporter | ||
Comment 4•15 years ago
|
||
restricted more between fada8c5cef07 (works, Part30) and and 47cd92d616d7 (broken,
Part36)
Reporter | ||
Comment 5•15 years ago
|
||
i cannot restrict more since looks like it does not compile in the middle (i did not try all 5 changesets though).
final range: between fada8c5cef07 (works, Part30) and and 9584d6bce9c1
(broken, Part34)
Reporter | ||
Comment 6•15 years ago
|
||
Part 34 is innocent (it's already broken at Part 33).
Updated•15 years ago
|
blocking2.0: --- → ?
Assignee | ||
Updated•15 years ago
|
Assignee: nobody → roc
Comment 7•15 years ago
|
||
I've noticed that the indicator only appears if you are dragging from left to the right. If you drag from right to the left it doesn't appear, only if you go ahead and come back (the same as if you drag from left to the right)
Assignee | ||
Comment 8•15 years ago
|
||
The problem is actually a fundamental one with our implementation of CSS transforms, as far as I can tell.
When the transform changes, we need to invalidate the old and new areas covered by the transformed frame. We invalidate the old overflow area in DoApplyRenderingChangeToTree. But the new overflow area isn't calculated until we reflow the transformed frame, and when that reflow occurs, nothing really moves, and nothing triggers invalidation of the new overflow area.
Assignee | ||
Comment 9•15 years ago
|
||
I think that whenever we have a transformed frame and the overflow area changes, we should invalidate the entire overflow area.
Assignee | ||
Comment 10•15 years ago
|
||
We already have code to invalidate the entire overflow area when it changes, used for SVG effects and outline rects which already have this problem. Using it for transforms too makes sense.
Attachment #434154 -
Flags: review?(matspal)
Assignee | ||
Comment 11•15 years ago
|
||
While I was looking into this, I noticed we have some places that do aFrame->Invalidate(aFrame->GetOverflowRect()). This is obviously supposed to just invalidate the entire drawn area of the frame subtree, so we can change these to aFrame->InvalidateOverflowArea and make InvalidateOverflowArea specifically invalidate GetOverflowAreaRelativeToSelf(). This is basically just cleanup, since Invalidate is already hacked to accept coordinates either pre-transform or post-transform... But we should fix that one day.
Attachment #434156 -
Flags: review?(matspal)
Assignee | ||
Updated•15 years ago
|
Whiteboard: [needs review]
Comment 12•15 years ago
|
||
Comment on attachment 434154 [details] [diff] [review]
fix
r=mats
Attachment #434154 -
Flags: review?(matspal) → review+
Updated•15 years ago
|
Attachment #434156 -
Flags: review?(matspal) → review+
Updated•15 years ago
|
Whiteboard: [needs review]
Assignee | ||
Updated•15 years ago
|
Whiteboard: [needs landing]
Assignee | ||
Comment 13•15 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/f25fbbe3e978
http://hg.mozilla.org/mozilla-central/rev/b5ebe19e4ecf
Status: NEW → RESOLVED
Closed: 15 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Whiteboard: [needs landing]
Comment 14•15 years ago
|
||
Verified fixed with with builds on Windows and OS X like Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.3a4pre) Gecko/20100328 Minefield/3.7a4pre ID:20100328204715
Coved by Litmus in https://litmus.mozilla.org/show_test.cgi?id=10006
Status: RESOLVED → VERIFIED
Flags: in-litmus+
Target Milestone: --- → mozilla1.9.3a4
Reporter | ||
Comment 15•15 years ago
|
||
hum, i can still reproduce in Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.3a4pre) Gecko/20100328 Minefield/3.7a4pre
i i drag toward left i don't see the indicator... am i missing something? Henrik?
Reporter | ||
Comment 16•15 years ago
|
||
uh. it's probably in the next build, sorry!
Why the user agent has not time indication but only date :\
Comment 17•15 years ago
|
||
I stumbled about the same problem this morning. And yes, I should be in the todays build. The verification happened with a tinderbox build from 8:45pm which already includes this fix.
Reporter | ||
Comment 18•15 years ago
|
||
indeed this is perfectly fixed in Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.3a4pre) Gecko/20100329 Minefield/3.7a4pre
Thank you Robert!
Henrik, what about filing a bug to get back times in nightlies UA?
Comment 19•15 years ago
|
||
The reason the time is not in the user agent string is that there was a bug where the time in user agent string screwed up the user agent sniffing of some sites. I can't locate the bug now unfortunately.
Comment 20•15 years ago
|
||
Assignee | ||
Updated•15 years ago
|
blocking2.0: ? → final+
Priority: -- → P2
Updated•6 years ago
|
Component: Layout: View Rendering → Layout: Web Painting
You need to log in
before you can comment on or make changes to this bug.
Description
•