Closed Bug 1667511 Opened 4 years ago Closed 4 years ago

[wpt-sync] Sync PR 25802 - Corrected border-image-shorthand-002 and 003 tests

Categories

(Core :: Layout: Images, Video, and HTML Frames, task, P4)

task

Tracking

()

RESOLVED INVALID

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 25802 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/25802
Details from upstream follow.

b'G\xc3\xa9rard Talbot <github@gtalbot.org>' wrote:

Corrected border-image-shorthand-002 and 003 tests

This is a followup of
https://github.com/web-platform-tests/wpt/pull/23808

css/css-backgrounds/border-image-shorthand-002.htm
css/css-backgrounds/border-image-shorthand-003.htm
css/css-backgrounds/reference/border-image-shorthand-002-ref.html
css/css-backgrounds/reference/border-image-shorthand-003-ref.html
css/css-backgrounds/support/9grid40-30-20-10-red.png

Following are @fantasai's review requirements from PR23808:

{
The 002 and 003 tests need a border-image-outset declaration in 002 and a border-image-width declaration in 003 which will be overridden by the specially coded/crafted shorthand.

Both tests should use the 9grid40-30-20-10-red.png version.
}

These review requirements are in this PR.

On my website, the files are:

http://www.gtalbot.org/BrowserBugsSection/CSS3Backgrounds/border-image-shorthand-002-GT.html

http://www.gtalbot.org/BrowserBugsSection/CSS3Backgrounds/border-image-shorthand-003-GT.html

http://www.gtalbot.org/BrowserBugsSection/CSS3Backgrounds/reference/border-image-shorthand-002-ref.html

http://www.gtalbot.org/BrowserBugsSection/CSS3Backgrounds/reference/border-image-shorthand-003-ref.html

Approving and merging this PR will allow me to close @clopez's
https://github.com/web-platform-tests/wpt/pull/21445

Component: web-platform-tests → Layout: Images, Video, and HTML Frames
Product: Testing → Core
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.