Closed
Bug 871917
Opened 12 years ago
Closed 11 years ago
Image do not appear after navigation-back and/or switch back tab
Categories
(Core :: SVG, defect)
Tracking
()
VERIFIED
FIXED
mozilla24
Tracking | Status | |
---|---|---|
firefox22 | --- | unaffected |
firefox23 | + | verified |
firefox24 | + | verified |
People
(Reporter: alice0775, Assigned: mattwoodrow)
References
Details
(Keywords: regression, reproducible)
Attachments
(1 file)
1.40 KB,
patch
|
roc
:
review+
lsblakk
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
Build Identifier: http://hg.mozilla.org/mozilla-central/rev/7130e5134a6e Mozilla/5.0 (Windows NT 6.1; WOW64; rv:23.0) Gecko/20130513 Firefox/23.0 ID:20130513031057 When I test Bug 871729, a problem found. Steps To Reproduce: 0. Extract attachment 748979 [details] of Bug 871729 1. Open ffTest.html 2. Open FemaleSkeleton_225.jpg in the same tab 3. Navigate Back (Press BS / Click Back button etc) (mouse pointer should be positioned outside of the image region) 4. Wait for few seconds --- observe image 5. Scroll Up/Down by dragging thumb/mouse wheel (mouse pointer should be positioned outside of the image region) --- observe image Actual Results: Image do not appear forever. Image is repaints partially when page scroll. Image appears when mouse pointer hover over image region. Image appears when window focus change. Expected results: Image should appear after navigation back immediately Regression window(m-c) Good: http://hg.mozilla.org/mozilla-central/rev/690b5e0f6562 Mozilla/5.0 (Windows NT 6.1; WOW64; rv:23.0) Gecko/20130424 Firefox/23.0 ID:20130424184951 Bad: http://hg.mozilla.org/mozilla-central/rev/7f68735fc8da Mozilla/5.0 (Windows NT 6.1; WOW64; rv:23.0) Gecko/20130425 Firefox/23.0 ID:20130425183955 Pushlog: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=690b5e0f6562&tochange=7f68735fc8da Regression window(m-i) Good: http://hg.mozilla.org/integration/mozilla-inbound/rev/78bdcb813be1 Mozilla/5.0 (Windows NT 6.1; WOW64; rv:23.0) Gecko/20130425 Firefox/23.0 ID:20130425010450 Bad: http://hg.mozilla.org/integration/mozilla-inbound/rev/96c350f2c64a Mozilla/5.0 (Windows NT 6.1; WOW64; rv:23.0) Gecko/20130424 Firefox/23.0 ID:20130425021949 Pushlog: http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=78bdcb813be1&tochange=96c350f2c64a Regressed by: 96c350f2c64a Jonathan Watt β Bug 864832 - Get rid of the majority of the remaining nsSVGUtils::InvalidateBounds calls, except for those related to transforms and nsSVGTextFrame2. r=mattwoodrow
![]() |
Reporter | |
Comment 1•12 years ago
|
||
Alternate STR: 1. Open ffTest.html in tab[Tab A] 2. Open a new tab in the front tab[Tab B] 3. Wait for 10-15 seconds 4. Switch to tab[Tab A] Actual Results: Image do not appear forever. Expected results: Image should appear after switch tab immediately
Summary: Image do not appear after navigation-back → Image do not appear after navigation-back and/or switch back tab
I saw this as well but wasn't sure if it was related. Great find and thanks for opening this issue up Alice!
Updated•12 years ago
|
Assignee: nobody → jwatt
status-firefox23:
--- → affected
status-firefox24:
--- → affected
Keywords: reproducible
As noted in Bug 871729, this one is very crucial for us, especially since it results in images disappearing. Any status update on this one? Thanks!
jwatt is on vacation. This seems like a regression we should fix.
Assignee: jwatt → matt.woodrow
Assignee | ||
Comment 5•11 years ago
|
||
Bug 864832 removed calls to SVGUtils::InvalidateBounds in these locations (that function no longer exists), which wasn't correct. We still need to mark the frame is invalid for DLBI to pick up on it. Thanks for the regression range and STR Alice!
Attachment #759612 -
Flags: review?(roc)
Attachment #759612 -
Flags: review?(roc) → review+
Assignee | ||
Comment 6•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/b6e14855f129
Comment 7•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/b6e14855f129
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla24
Quick question on this one. When testing this out locally, I see the image still missing but it reappears within about 300-500ms which seems a little odd to me. Is this expected or should it never disappear?
Also FYI, tested this in branch 19 and I'm not seeing any delay in the image reappearing when swapping back to the other tab - it's always there.
Assignee | ||
Comment 10•11 years ago
|
||
Comment on attachment 759612 [details] [diff] [review] Invalidate the svg frame when we get an imagelib notification [Approval Request Comment] Bug caused by (feature/regressing bug #): Bug 864832 User impact if declined: Images missing at some tmes Testing completed (on m-c, etc.): Been on m-c for over a week. Risk to taking this patch (and alternatives if risky): Very low risk, just makes sure we repaint SVG images. String or IDL/UUID changes made by this patch: None
Attachment #759612 -
Flags: approval-mozilla-aurora?
Updated•11 years ago
|
Attachment #759612 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 12•11 years ago
|
||
Should I open up a new ticket for the images not reappearing immediately after tab switching? (200-300ms lag it seems)
Please do, and CC me and :tn.
Comment 15•11 years ago
|
||
Reproduced the issue on Nightly 2013-05-17 Win 7 x64. Verified fixed FF 23b6, 24.0a2 (2013-07-15) considering the remaining work from bug 887792.
You need to log in
before you can comment on or make changes to this bug.
Description
•