[Webkit-unassigned] [Bug 147721] New: Web Inspector: Cached external stylesheets do not (re)output parser warnings for stylesheet to console

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=147721

            Bug ID: 147721
           Summary: Web Inspector: Cached external stylesheets do not
                    (re)output parser warnings for stylesheet to console
    Classification: Unclassified
           Product: WebKit
           Version: 528+ (Nightly build)
          Hardware: All
                OS: All
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: Web Inspector
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: joepeck at webkit.org
                CC: bburg at apple.com, graouts at webkit.org,
                    joepeck at webkit.org, koivisto at iki.fi,
                    mattbaker at apple.com, nvasilyev at apple.com,
                    timothy at apple.com, webkit-bug-importer at group.apple.com

* SUMMARY
Cached external stylesheets do not (re)output parser warnings for stylesheet to console.

* STEPS TO REPRODUCE
1. Inspect inspector-protocol/console/css-source-locations.html via http
  => console warnings for external stylesheet errors.css and inline style
2. Reload
  => console warnings for just inline style

* NOTES
- When errors.css was cached the warnings are not output in the new page load

- HTMLLinkElement::setCSSStyleSheet has a block of code that seems to re-use a cachable sheet, and doesn't re-parse it, without re-parsing (parseAuthorStyleSheet) it the warnings are not re-logged to the console

>    void HTMLLinkElement::setCSSStyleSheet(const String& href, const URL& baseURL, const String& charset, const CachedCSSStyleSheet* cachedStyleSheet)
>    {
>        ...
>        if (RefPtr<StyleSheetContents> restoredSheet = const_cast<CachedCSSStyleSheet*>(cachedStyleSheet)->restoreParsedStyleSheet(parserContext, cachePolicy)) {
>            ...
>            m_loading = false;
>            sheetLoaded();
>            notifyLoadedSheetAndAllCriticalSubresources(false);
>            return;
>        }
>        ...
>        styleSheet.get().parseAuthorStyleSheet(cachedStyleSheet, document().securityOrigin());
>        ...
>    }

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20150806/e0b8daa2/attachment.html>