Closed Bug 795262 Opened 7 years ago Closed 7 years ago
Highlighter depends on Moz
After Paint to detect that the size or position of an element has changed
This is a problem with DLBI (bug 539356) since we never promised to fire MozAfterPaint for changes to invisible elements (and we already don't in some cases, e.g. offscreen elements). DLBI makes the browser_inspector_invalidate.js test fail because the DIV whose width is changed is invisible; nothing gets repainted, so we don't fire MozAfterPaint.
Comment on attachment 665847 [details] [diff] [review] fix test There's nothing special about these changes so asking for cjones to give a quick review now. If there are any issues, I'll fix them after this.
Attachment #665847 - Flags: review?(rcampbell)
Attachment #665847 - Flags: review?(jones.chris.g) → review+
This patch makes the test pass, but there's still a bug where highlighting invisible elements won't necessarily update promptly when the invisible element size or position changes (and nothing else does). Right now I'm not sure how to fix that. We don't have a good API to detect such changes. It might be good enough to simply use setTimeout or mozRequestAnimationFrame and poll for changes.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 18
Comment on attachment 665847 [details] [diff] [review] fix test post hoc r+ from me. Thanks for this.
Attachment #665847 - Flags: review?(rcampbell) → review+
You need to log in before you can comment on or make changes to this bug.