Bug 49731 - Web Inspector: [REGRESSION] Crash on opening WebInspector for a page with CSS containing @media with nested rules
Summary: Web Inspector: [REGRESSION] Crash on opening WebInspector for a page with CSS...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Alexander Pavlov (apavlov)
URL:
Keywords:
: 49738 49862 (view as bug list)
Depends on:
Blocks:
 
Reported: 2010-11-18 07:10 PST by Alexander Pavlov (apavlov)
Modified: 2010-11-24 14:27 PST (History)
13 users (show)

See Also:

Attachments
[PATCH] Suggested fix (7.37 KB, patch)
2010-11-19 04:42 PST, Alexander Pavlov (apavlov) 		pfeldman: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Pavlov (apavlov) 2010-11-18 07:10:52 PST 
1. Go to http://www.opera.com/
2. Open Web Inspector

Upstreaming http://code.google.com/p/chromium/issues/detail?id=63575
Comment 1 Alexander Pavlov (apavlov) 2010-11-19 04:42:27 PST 
Created attachment 74368 [details]
[PATCH] Suggested fix

This patch temporarily disables building of styles for nested rules. A corresponding regression bug will be filed once the crasher is gone.
Comment 2 Pavel Feldman 2010-11-19 04:44:26 PST 
Comment on attachment 74368 [details]
[PATCH] Suggested fix

View in context: https://bugs.webkit.org/attachment.cgi?id=74368&action=review

> WebCore/inspector/InspectorStyleSheet.cpp:682
> +        RefPtr<InspectorObject> bogusStyle = InspectorObject::create();

Please add FIXME here.
Comment 3 Alexander Pavlov (apavlov) 2010-11-19 06:56:56 PST 
Committed with comment addressed: <http://trac.webkit.org/changeset/72386>
Comment 4 Pavel Feldman 2010-11-21 07:33:52 PST 
*** Bug 49862 has been marked as a duplicate of this bug. ***
Comment 5 Alexander Pavlov (apavlov) 2010-11-24 14:27:51 PST 
*** Bug 49738 has been marked as a duplicate of this bug. ***