https://bugs.webkit.org/show_bug.cgi?id=89818
Reviewed by Antti Koivisto.
Patch works by fixing treating handling of CSSPropertyID == CSSPropertyVariable as a special case,
and looking up the author-defined property name from the CSSValue.
Added test inspector/styles/variables/css-variables.html that inspects an element using CSS variables.
Test is skipped when variables are compiled out.
* css/CSSProperty.cpp:
(WebCore::CSSProperty::cssName):
(WebCore):
(WebCore::CSSProperty::cssText):
* css/CSSProperty.h:
(CSSProperty):
* css/PropertySetCSSStyleDeclaration.cpp:
(WebCore::PropertySetCSSStyleDeclaration::item):
* css/StylePropertySet.cpp:
(WebCore::StylePropertySet::asText):
* inspector/InspectorStyleSheet.cpp:
(WebCore::InspectorStyle::populateAllProperties):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@121874
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
--- /dev/null
+Tests that webkit css variables can be loaded correctly.
+
+Text
+[expanded]
+color: green;
+ #inspected - -webkit-var(a) css-variables.html:7
+display: block;
+ div - block user agent stylesheet
+
+[expanded]
+element.style { ()
+
+======== Matched CSS Rules ========
+[expanded]
+#inspected { (css-variables.html:7)
+-webkit-var-a: green;
+color: -webkit-var(a);
+
+[expanded]
+div { (user agent stylesheet)
+display: block;
+
+
+
--- /dev/null
+<html>
+<head>
+<script>
+internals.settings.setCSSVariablesEnabled(true);
+</script>
+<style>
+#inspected {
+ -webkit-var-a: green;
+ color: -webkit-var(a);
+}
+
+</style>
+<script src="../../http/tests/inspector/inspector-test.js"></script>
+<script src="../../http/tests/inspector/elements-test.js"></script>
+<script>
+
+function test()
+{
+ WebInspector.showPanel("elements");
+ InspectorTest.selectNodeAndWaitForStylesWithComputed("inspected", dumpAllStyles);
+
+ function dumpAllStyles()
+ {
+ InspectorTest.dumpSelectedElementStyles();
+ InspectorTest.completeTest();
+ }
+}
+
+</script>
+</head>
+
+<body onload="runTest()">
+<p>
+Tests that webkit css variables can be loaded correctly.
+</p>
+
+<div id="inspected">Text</div>
+
+</body>
+</html>
// CSS Variables are not yet enabled.
BUGWK85580 SKIP : fast/css/variables = PASS
+BUGWK85580 SKIP : inspector/styles/variables = PASS
// CSS image-resolution is not yet enabled.
BUGWK85262 SKIP : fast/css/image-resolution = PASS
// CSS Variables are not yet enabled.
BUGWK85580 SKIP : fast/css/variables = PASS TEXT
+BUGWK85580 SKIP : inspector/styles/variables = PASS
// CSS image-resolution is not yet enabled.
BUGWK85262 SKIP : fast/css/image-resolution = PASS TEXT
// CSS Variables are not yet enabled.
BUGWK85580 SKIP : fast/css/variables = PASS
+BUGWK85580 SKIP : inspector/styles/variables = PASS
// UndoManager is not yet enabled.
BUGWK87908 SKIP : editing/undomanager = PASS
// CSS Variables are not yet enabled.
BUGWK85580 SKIP : fast/css/variables = PASS
+BUGWK85580 SKIP : inspector/styles/variables = PASS
// UndoManager is not yet enabled.
BUGWK87908 SKIP : editing/undomanager = PASS
+2012-07-04 Luke Macpherson <macpherson@chromium.org>
+
+ Inspector crashes when trying to inspect a page with CSS variables
+ https://bugs.webkit.org/show_bug.cgi?id=89818
+
+ Reviewed by Antti Koivisto.
+
+ Patch works by fixing treating handling of CSSPropertyID == CSSPropertyVariable as a special case,
+ and looking up the author-defined property name from the CSSValue.
+
+ Added test inspector/styles/variables/css-variables.html that inspects an element using CSS variables.
+ Test is skipped when variables are compiled out.
+
+ * css/CSSProperty.cpp:
+ (WebCore::CSSProperty::cssName):
+ (WebCore):
+ (WebCore::CSSProperty::cssText):
+ * css/CSSProperty.h:
+ (CSSProperty):
+ * css/PropertySetCSSStyleDeclaration.cpp:
+ (WebCore::PropertySetCSSStyleDeclaration::item):
+ * css/StylePropertySet.cpp:
+ (WebCore::StylePropertySet::asText):
+ * inspector/InspectorStyleSheet.cpp:
+ (WebCore::InspectorStyle::populateAllProperties):
+
2012-07-04 Anthony Scian <ascian@rim.com>
Web Inspector [JSC]: Implement ScriptCallStack::stackTrace
CSSParser* p = static_cast<CSSParser*>(parser);
p->storeVariableDeclaration($1, p->sinkFloatingValueList($4), $5);
$$ = true;
- p->markPropertyEnd($5, $$);
+ p->markPropertyEnd($5, true);
#else
$$ = false;
#endif
void CSSParser::storeVariableDeclaration(const CSSParserString& name, PassOwnPtr<CSSParserValueList> value, bool important)
{
+ ASSERT(name.length > 12);
+ AtomicString variableName = String(name.characters + 12, name.length - 12);
+
StringBuilder builder;
for (unsigned i = 0, size = value->size(); i < size; i++) {
if (i)
builder.append(' ');
builder.append(value->valueAt(i)->createCSSValue()->cssText());
}
- addProperty(CSSPropertyVariable, CSSVariableValue::create(name, builder.toString()), important, false);
+ addProperty(CSSPropertyVariable, CSSVariableValue::create(variableName, builder.toString()), important, false);
}
#endif
}
case CharacterDash:
-#if ENABLE(CSS_VARIABLES)
- if (cssVariablesEnabled() && isEqualToCSSIdentifier(m_currentCharacter, "webkit-var") && m_currentCharacter[10] == '-' && isIdentifierStartAfterDash(m_currentCharacter + 11)) {
- // handle variable declarations
- m_currentCharacter += 11;
- parseIdentifier(result, hasEscape);
- m_token = VAR_DEFINITION;
- yylval->string.characters = m_tokenStart;
- yylval->string.length = result - m_tokenStart;
- } else
-#endif
if (isIdentifierStartAfterDash(m_currentCharacter)) {
--m_currentCharacter;
parseIdentifier(result, hasEscape);
m_token = IDENT;
+#if ENABLE(CSS_VARIABLES)
+ if (cssVariablesEnabled() && isEqualToCSSIdentifier(m_tokenStart + 1, "webkit-var") && m_tokenStart[11] == '-' && isIdentifierStartAfterDash(m_tokenStart + 12))
+ m_token = VAR_DEFINITION;
+ else
+#endif
if (*m_currentCharacter == '(') {
m_token = FUNCTION;
if (!hasEscape)
#include "RenderStyleConstants.h"
#include "StylePropertyShorthand.h"
+#if ENABLE(CSS_VARIABLES)
+#include "CSSVariableValue.h"
+#endif
+
namespace WebCore {
struct SameSizeAsCSSProperty {
COMPILE_ASSERT(sizeof(CSSProperty) == sizeof(SameSizeAsCSSProperty), CSSProperty_should_stay_small);
+String CSSProperty::cssName() const
+{
+#if ENABLE(CSS_VARIABLES)
+ if (id() == CSSPropertyVariable) {
+ ASSERT(value()->isVariableValue());
+ return "-webkit-var-" + static_cast<CSSVariableValue*>(value())->name();
+ }
+#endif
+ return String(getPropertyName(id()));
+}
+
String CSSProperty::cssText() const
{
- return String(getPropertyName(id())) + ": " + m_value->cssText() + (isImportant() ? " !important" : "") + "; ";
+ return cssName() + ": " + m_value->cssText() + (isImportant() ? " !important" : "") + "; ";
}
void CSSProperty::wrapValueInCommaSeparatedList()
CSSValue* value() const { return m_value.get(); }
+ String cssName() const;
String cssText() const;
void wrapValueInCommaSeparatedList();
{
if (i >= m_propertySet->propertyCount())
return "";
- return getPropertyName(m_propertySet->propertyAt(i).id());
+ return m_propertySet->propertyAt(i).cssName();
}
String PropertySetCSSStyleDeclaration::cssText() const
String value;
switch (propertyID) {
+#if ENABLE(CSS_VARIABLES)
+ case CSSPropertyVariable:
+ result.append(prop.cssText());
+ continue;
+#endif
case CSSPropertyBackgroundPositionX:
positionXProp = ∝
continue;
return false;
RefPtr<StyleSheetContents> newStyleSheet = StyleSheetContents::create();
- CSSParser p(CSSStrictMode);
+ CSSParser p(m_pageStyleSheet->ownerDocument());
OwnPtr<RuleSourceDataList> ruleSourceDataResult = adoptPtr(new RuleSourceDataList());
p.parseSheet(newStyleSheet.get(), m_parsedStyleSheet->text(), 0, ruleSourceDataResult.get());
m_parsedStyleSheet->setSourceData(ruleSourceDataResult.release());
"WebCoreHas3DRendering": ["animations/3d", "transforms/3d"],
"WebGLShader": ["fast/canvas/webgl", "compositing/webgl", "http/tests/canvas/webgl"],
"MHTMLArchive": ["mhtml"],
- "CSSVariableValue": ["fast/css/variables"],
+ "CSSVariableValue": ["fast/css/variables", "inspector/styles/variables"],
}
def _has_test_in_directories(self, directory_lists, test_list):
"http/tests/canvas/webgl", # Requires WebGLShader
"mhtml", # Requires MHTMLArchive
"fast/css/variables", # Requires CSS Variables
+ "inspector/styles/variables", # Requires CSS Variables
])
result_directories = set(TestWebKitPort(symbols_string, None)._skipped_tests_for_unsupported_features(test_list=['mathml/foo.html']))
projects / profile / ivi / webkit-efl.git / commitdiff