Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(719)

Issue 602703002: Fixed tests to match Chrome 38 font property change; font-stretch added.

Created:
6 years, 3 months ago by terry
Modified:
6 years, 2 months ago
Reviewers:
CC:
team_co19.biz
Visibility:
Public.

Description

Fixed tests to match Chrome 38 font property change; font-stretch added. There has been a change to the CSS font property; it now display font-stretch. Below is the new value format: [ [ <‘font-style’> || <font-variant-css21> || <‘font-weight’> || <‘font-stretch’ ]? <‘font-size’> [ /<‘line-height’> ]? <‘font-family’> ] | caption | icon | menu | message-box | small-caption | status-bar Previous format of font property was: font-style font-variant font-weight font-size/line-height|caption|icon|menu|message-box|small-caption|status-bar|initial|inherit; I also noticed that Blink's co19 tests (Javascript) have also changed to handle this extra property displayed.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+65 lines, -65 lines) Patch
M co19/tests/co19/src/LayoutTests/fast/css/font-shorthand-from-longhands_t01.dart View 2 chunks +9 lines, -9 lines 0 comments Download
M co19/tests/co19/src/LayoutTests/fast/css/getComputedStyle/computed-style-font_t01.dart View 1 chunk +56 lines, -56 lines 0 comments Download

Powered by Google App Engine
This is Rietveld 408576698