Last Comment Bug 768019 - Cached CSS warnings are displayed as JavaScript warnings on Web Console open
: Cached CSS warnings are displayed as JavaScript warnings on Web Console open
Product: Firefox
Classification: Client Software
Component: Developer Tools: Console (show other bugs)
: Trunk
: x86 All
: -- normal (vote)
: Firefox 16
Assigned To: Mihai Sucan [:msucan]
: Brian Grinstead [:bgrins]
Depends on:
  Show dependency treegraph
Reported: 2012-06-25 08:42 PDT by Paul Rouget [:paul]
Modified: 2012-06-30 15:16 PDT (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

test (243 bytes, text/html)
2012-06-25 08:42 PDT, Paul Rouget [:paul]
no flags Details
screenshot (17.63 KB, image/png)
2012-06-25 08:43 PDT, Paul Rouget [:paul]
no flags Details
[in-fx-team] proposed patch (2.21 KB, patch)
2012-06-25 10:23 PDT, Mihai Sucan [:msucan]
paul: review+
Details | Diff | Splinter Review

Description Paul Rouget [:paul] 2012-06-25 08:42:48 PDT
Created attachment 636324 [details]
Comment 1 Paul Rouget [:paul] 2012-06-25 08:43:25 PDT
Created attachment 636325 [details]
Comment 2 Mihai Sucan [:msucan] 2012-06-25 09:40:39 PDT
How do you reproduce the bug?

I tried this and I only see one CSS warning in the Web Console and in the Error Console.
Comment 3 Paul Rouget [:paul] 2012-06-25 09:48:05 PDT
- open test.html in a new tab
- switch to the tab
- wait until the page is loaded
- open the web console

Actual Result:
- JS warning "unknown property"
Comment 4 Mihai Sucan [:msucan] 2012-06-25 09:57:49 PDT
Ah, so I have to open the Web Console after the error happens.

Thanks for the STR. I was able to reproduce now. I have a hunch what's causing this. Will look into fixing it ASAP.
Comment 5 Mihai Sucan [:msucan] 2012-06-25 10:23:16 PDT
Created attachment 636374 [details] [diff] [review]
[in-fx-team] proposed patch

This fixes the problem for me. It is an embarrassingly simple fix: I passed the wrong argument to the function that determines the cached message category.

Thanks for your report!
Comment 6 Mihai Sucan [:msucan] 2012-06-26 12:14:46 PDT
Thanks for the r+!
Comment 7 Mihai Sucan [:msucan] 2012-06-27 02:26:53 PDT
Comment on attachment 636374 [details] [diff] [review]
[in-fx-team] proposed patch
Comment 8 Tim Taubert [:ttaubert] 2012-06-30 15:16:42 PDT

Note You need to log in before you can comment on or make changes to this bug.