Should move: - CSSStyleRule.attributeStyleMap -> CSSStyleRule.styleMap - Element.attributeStyleMap -> ElementCSSInlineStyle.attributeStyleMap (should not affect any existing code) Spec: https://github.com/w3c/css-houdini-drafts/issues/572
i will take this issue.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/f4f8b21412f49d28290744cc66f4a743f18ddacb commit f4f8b21412f49d28290744cc66f4a743f18ddacb Author: Hwanseung Lee <hs1217.lee@samsung.com> Date: Tue Jan 30 23:25:49 2018 [css-typed-om] rename attributeStyleMap to styleMap at CSSStyleRule rename attributeStyleMap to styleMap at CSSStyleRule https://github.com/w3c/css-houdini-drafts/issues/572 https://github.com/w3c/css-houdini-drafts/pull/596 Bug: 807008 Change-Id: I8e5f140914783c074f73f75ed94312989e90e0ef Reviewed-on: https://chromium-review.googlesource.com/893239 Reviewed-by: Darren Shen <shend@chromium.org> Commit-Queue: Hwanseung Lee <hs1217.lee@samsung.com> Cr-Commit-Position: refs/heads/master@{#533077} [modify] https://crrev.com/f4f8b21412f49d28290744cc66f4a743f18ddacb/third_party/WebKit/LayoutTests/external/wpt/css/css-typed-om/resources/testhelper.js [modify] https://crrev.com/f4f8b21412f49d28290744cc66f4a743f18ddacb/third_party/WebKit/LayoutTests/external/wpt/css/css-typed-om/the-stylepropertymap/declared/declared.tentative.html [modify] https://crrev.com/f4f8b21412f49d28290744cc66f4a743f18ddacb/third_party/WebKit/LayoutTests/external/wpt/css/css-typed-om/the-stylepropertymap/declared/test.tentative.html [modify] https://crrev.com/f4f8b21412f49d28290744cc66f4a743f18ddacb/third_party/WebKit/LayoutTests/external/wpt/css/css-typed-om/the-stylepropertymap/interface-expected.txt [modify] https://crrev.com/f4f8b21412f49d28290744cc66f4a743f18ddacb/third_party/WebKit/LayoutTests/external/wpt/css/css-typed-om/the-stylepropertymap/interface.html [modify] https://crrev.com/f4f8b21412f49d28290744cc66f4a743f18ddacb/third_party/WebKit/LayoutTests/typedcssom/resources/testhelper.js [modify] https://crrev.com/f4f8b21412f49d28290744cc66f4a743f18ddacb/third_party/WebKit/LayoutTests/typedcssom/the-stylepropertymap/declared/delete-rule-crash.html [modify] https://crrev.com/f4f8b21412f49d28290744cc66f4a743f18ddacb/third_party/WebKit/LayoutTests/webexposed/global-interface-listing-expected.txt [modify] https://crrev.com/f4f8b21412f49d28290744cc66f4a743f18ddacb/third_party/WebKit/Source/core/css/CSSStyleRule.cpp [modify] https://crrev.com/f4f8b21412f49d28290744cc66f4a743f18ddacb/third_party/WebKit/Source/core/css/CSSStyleRule.h [modify] https://crrev.com/f4f8b21412f49d28290744cc66f4a743f18ddacb/third_party/WebKit/Source/core/css/CSSStyleRule.idl [modify] https://crrev.com/f4f8b21412f49d28290744cc66f4a743f18ddacb/third_party/WebKit/Source/core/css/cssom/DeclaredStylePropertyMap.h
i think we should implement ElementCSSInlineStyle before Element.attributeStyleMap move. actually ElementCSSInlineStyle was not exist in chromium. https://drafts.csswg.org/cssom/#the-elementcssinlinestyle-interface https://cs.chromium.org/chromium/src/third_party/WebKit/Source/core/svg/SVGElement.idl?q=SVGElement.idl&sq=package:chromium&l=30 https://cs.chromium.org/chromium/src/third_party/WebKit/Source/core/html/HTMLElement.idl?q=HTMLElement.idl&sq=package:chromium&dr&l=59
Oops, sorry I didn't see this. Yep, this sounds good (I didn't know ElementCSSInlineStyle isn't implemented in Blink).
Comment 1 by hs1217....@samsung.com
, Jan 30 2018Owner: hs1217....@samsung.com
Status: Assigned (was: Available)