Created attachment 467431 [details] Valgrind message Conditional jump or move depends on uninitialised value(s) [@ CSSLoaderImpl::ParseSheet] Affects 1.9.2 and 1.9.1, not trunk
Created attachment 467432 [details] [diff] [review] fix The typo fix is to make -DDEBUG_TRACEMALLOC_PRESARENA compile.
Comment on attachment 467432 [details] [diff] [review] fix r=dbaron
This seems like something that could break any chrome style sheets that have a syntax error in the first rule, which could be a problem for some extensions. Seems like something we should get in to the branch soon. It's a regression from bug 524223.
This would not block a release, but we would like to fix it as it is a regression. Feel free to nominate for for 3.6.10 and 3.5.13 when it is ready.
Comment on attachment 467432 [details] [diff] [review] fix Approved for 18.104.22.168 and 22.214.171.124, a=dveditz for release-drivers
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/bc021a2ec6a9 http://hg.mozilla.org/releases/mozilla-1.9.2/rev/2e3179e393e3 http://hg.mozilla.org/releases/mozilla-1.9.1/rev/05acd6e91c43
Is there anything for QA to do to verify this on branch?
(In reply to comment #7) > Is there anything for QA to do to verify this on branch? Probably not, you need a special build (ac_add_options --enable-valgrind) and then run unit tests under valgrind to see the reported error. Maybe it's possible to make a testcase based on comment 3, I don't know.