Bug 15888 – Use computed style helper method in LayoutTests to support other browsers

• Component: Tools / Tests
• Status: NEW

[David Kilzer (:ddkilzer)]

Since different browsers support different methods of computing style for an element, it would be nice to have a library method for the layout tests that abstracts this process to make it easier to run layout tests using computed style on multiple browsers.

$ find LayoutTests -name platform -prune -o -name .svn -prune -o -exec grep -l -i computedStyle {} \;
LayoutTests/ChangeLog
LayoutTests/ChangeLog-2007-10-14
LayoutTests/editing/style/relative-font-size-change-001.html
LayoutTests/editing/style/relative-font-size-change-002.html
LayoutTests/editing/style/relative-font-size-change-003.html
LayoutTests/editing/style/relative-font-size-change-004.html
LayoutTests/fast/css/child-selector-implicit-tbody.html
LayoutTests/fast/css/computed-style-display-none.html
LayoutTests/fast/css/computed-style-negative-top.html
LayoutTests/fast/css/computed-style-without-renderer.html
LayoutTests/fast/css/computed-style.html
LayoutTests/fast/css/css-properties-case-insensitive.html
LayoutTests/fast/css/device-aspect-ratio-expected.txt
LayoutTests/fast/css/device-aspect-ratio.html
LayoutTests/fast/css/getComputedStyle-background-position.html
LayoutTests/fast/css/getComputedStyle-background-size.html
LayoutTests/fast/css/getComputedStyle-border-box-expected.txt
LayoutTests/fast/css/getComputedStyle-border-box.html
LayoutTests/fast/css/getComputedStyle-border-spacing.html
LayoutTests/fast/css/getComputedStyle-borderRadius.html
LayoutTests/fast/css/getComputedStyle-relayout-expected.txt
LayoutTests/fast/css/getComputedStyle-relayout.html
LayoutTests/fast/css/getComputedStyle-zIndex-auto.html
LayoutTests/fast/css/getFloatValueForUnit.html
LayoutTests/fast/css/hover-affects-child.html
LayoutTests/fast/css/invalid-cursor-property-crash.html
LayoutTests/fast/css/marginComputedStyle.html
LayoutTests/fast/css/max-device-aspect-ratio-expected.txt
LayoutTests/fast/css/max-device-aspect-ratio.html
LayoutTests/fast/css/min-device-aspect-ratio-expected.txt
LayoutTests/fast/css/min-device-aspect-ratio.html
LayoutTests/fast/css/pendingStylesheetFontSize.html
LayoutTests/fast/dom/computed-style-set-property.html
LayoutTests/fast/dom/defaultView-expected.txt
LayoutTests/fast/dom/defaultView.html
LayoutTests/fast/dom/resources/wrapper-identity-base.js
LayoutTests/fast/dom/Window/get-set-properties-expected.txt
LayoutTests/fast/dom/Window/get-set-properties.html
LayoutTests/fast/dom/Window/window-properties-expected.txt
LayoutTests/fast/dom/Window/window-property-shadowing-expected.txt
LayoutTests/fast/dom/Window/window-property-shadowing.html
LayoutTests/fast/dom/wrapper-identity-expected.txt
LayoutTests/fast/dynamic/hovered-detach.html
LayoutTests/fast/forms/radio-no-theme-padding.html
LayoutTests/fast/loader/local-CSS-from-local.html
LayoutTests/fast/table/cell-pref-width-invalidation.html
LayoutTests/http/tests/misc/css-accept-any-type.html
LayoutTests/http/tests/security/cross-frame-access-get-expected.txt
LayoutTests/http/tests/security/cross-frame-access-get.html
LayoutTests/http/tests/security/cross-frame-access-put-expected.txt
LayoutTests/http/tests/security/cross-frame-access-put.html
LayoutTests/http/tests/security/local-CSS-from-remote.html
LayoutTests/http/tests/security/local-user-CSS-from-remote.html
LayoutTests/http/tests/security/resources/cross-frame-iframe-for-put-test.html
LayoutTests/svg/css/getComputedStyle-basic.xhtml
LayoutTests/svg/css/glyph-orientation-rounding-test-expected.txt
LayoutTests/svg/css/glyph-orientation-rounding-test.xhtml
LayoutTests/tables/mozilla/bugs/bug157890.html

[David Kilzer (:ddkilzer)]

Layout tests for bug 14893 could also use this when they land:

LayoutTests/fast/css/device-aspect-ratio.html
LayoutTests/fast/css/min-device-aspect-ratio.html
LayoutTests/fast/css/max-device-aspect-ratio.html

[Ahmad Saleem]

@ap & @David - is this still an issue with test harness and js-test scripts? Or we can close this?