Bug 89221 - CSSOM wrappers are not built for rules in the per-document and page group user stylesheets
Summary: CSSOM wrappers are not built for rules in the per-document and page group use...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Alexander Pavlov (apavlov)
URL:
Keywords:
Depends on: 89364
Blocks:
  Show dependency tree / graph
 
Reported: 2012-06-15 09:20 PDT by Alexander Pavlov (apavlov)
Modified: 2012-06-18 11:28 PDT (History)
21 users (show)

See Also:

Attachments
Patch (12.32 KB, patch)
2012-06-18 04:59 PDT, Alexander Pavlov (apavlov) 		no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ec2-cr-linux-04 (659.30 KB, application/zip)
2012-06-18 05:31 PDT, WebKit Review Bot 		no flags Details
Patch (7.84 KB, patch)
2012-06-18 05:53 PDT, Alexander Pavlov (apavlov) 		koivisto: review+
Details | Formatted Diff | Diff
Show Obsolete (1) 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) 2012-06-15 09:20:35 PDT 
Upstreaming http://code.google.com/p/chromium/issues/detail?id=132425
Comment 1 Alexey Proskuryakov 2012-06-17 09:38:09 PDT 
Alexander, could you please provide complete information about the issue here? Asking everyone looking at a bug to click through to an external site to get information is plain offensive.
Comment 2 Alexander Pavlov (apavlov) 2012-06-17 14:16:31 PDT 
(In reply to comment #1)
> Alexander, could you please provide complete information about the issue here? Asking everyone looking at a bug to click through to an external site to get information is plain offensive.

Yes, sorry.

Web Inspector does not display Chromium extension-injected CSS rules in the Styles pane.

I believe non-Chromium users will rarely encounter this issue.
Comment 3 Alexander Pavlov (apavlov) 2012-06-18 04:59:51 PDT 
Created attachment 148084 [details]
Patch
Comment 4 Gustavo Noronha (kov) 2012-06-18 05:10:24 PDT 
Comment on attachment 148084 [details]
Patch

Attachment 148084 [details] did not pass gtk-ews (gtk):
Output: http://queues.webkit.org/results/12973381
Comment 5 Build Bot 2012-06-18 05:19:20 PDT 
Comment on attachment 148084 [details]
Patch

Attachment 148084 [details] did not pass win-ews (win):
Output: http://queues.webkit.org/results/12971618
Comment 6 Early Warning System Bot 2012-06-18 05:24:41 PDT 
Comment on attachment 148084 [details]
Patch

Attachment 148084 [details] did not pass qt-wk2-ews (qt):
Output: http://queues.webkit.org/results/12978261
Comment 7 Early Warning System Bot 2012-06-18 05:27:40 PDT 
Comment on attachment 148084 [details]
Patch

Attachment 148084 [details] did not pass qt-ews (qt):
Output: http://queues.webkit.org/results/12970647
Comment 8 WebKit Review Bot 2012-06-18 05:31:13 PDT 
Comment on attachment 148084 [details]
Patch

Attachment 148084 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/12971620

New failing tests:
inspector/styles/inject-stylesheet.html
Comment 9 WebKit Review Bot 2012-06-18 05:31:18 PDT 
Created attachment 148086 [details]
Archive of layout-test-results from ec2-cr-linux-04

The attached test failures were seen while running run-webkit-tests on the chromium-ews.
Bot: ec2-cr-linux-04  Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'>  Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Comment 10 Gyuyoung Kim 2012-06-18 05:36:06 PDT 
Comment on attachment 148084 [details]
Patch

Attachment 148084 [details] did not pass efl-ews (efl):
Output: http://queues.webkit.org/results/12981080
Comment 11 Build Bot 2012-06-18 05:44:30 PDT 
Comment on attachment 148084 [details]
Patch

Attachment 148084 [details] did not pass mac-ews (mac):
Output: http://queues.webkit.org/results/12980110
Comment 12 Alexander Pavlov (apavlov) 2012-06-18 05:53:54 PDT 
Created attachment 148087 [details]
Patch
Comment 13 Alexander Pavlov (apavlov) 2012-06-18 09:04:27 PDT 
Committed r120600: <http://trac.webkit.org/changeset/120600>
Comment 14 Csaba Osztrogonác 2012-06-18 09:52:12 PDT 
(In reply to comment #13)
> Committed r120600: <http://trac.webkit.org/changeset/120600>

Something is wrong with the new test, because it fails on Qt and Apple Lion:

-CONSOLE MESSAGE: line 9: iframe loaded
 Tests that injected user stylesheets are reflected in the Styles pane.
 
+This test requires window.testRunner, exiting.

Could you check it, please?
Comment 15 Csaba Osztrogonác 2012-06-18 11:28:16 PDT 
New bug report filed on this failure, because reopening closed bugs and discussion in closed bugs isn't allowed: https://bugs.webkit.org/show_bug.cgi?id=89364