Ctrl++ or Ctrl+- on any page triggers: WARNING: Unable to test style tree integrity -- no content node: file /Users/jruderman/trunk/mozilla/layout/base/nsCSSFrameConstructor.cpp, line 10110 This even happens on the empty page data:text/html,
We're doing a style change on the viewport. That's rare, but can happen....
It's not rare any more... can we change the assertion so it doesn't fire in this case?
It's not an assertion... it's a warning. And it _is_ rare compared to style changes in general. ;) We could probably skip the warning if the frame is a viewport... or even altogether. David, sound OK?
We hit this at pageload time, for files with <meta charset="utf-8">, since the charset-change happens asynchronously and triggers a rebuild of all style data starting with the viewport frame. I agree that we should skip the warning if the frame is a viewport. (I was actually just about to file a bug on making that change, and then I ran across this one from bugzilla's new-bug-form dupe-finder.)
Minimal testcase that demonstrates the beginning of comment 4: data:text/html,<meta charset="utf-8">
Created attachment 763872 [details] [diff] [review] fix v1 This patch skips the warning if we have a frame and it's a viewport frame.
Assignee: nobody → dholbert
Status: NEW → ASSIGNED
Attachment #763872 - Flags: review?(bzbarsky)
Comment on attachment 763872 [details] [diff] [review] fix v1 r=me
Attachment #763872 - Flags: review?(bzbarsky) → review+
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla24
You need to log in before you can comment on or make changes to this bug.