This causes unnecessary style recalc of descendants for inheritance when an element with such a property gets its style recalculated.
https://codereview.chromium.org/1887613002/
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/3b6513eaede077dc4171f5a05157a8658279a68a commit 3b6513eaede077dc4171f5a05157a8658279a68a Author: rune <rune@opera.com> Date: Wed Apr 13 15:58:35 2016 Compare font-feature-settings as part of Font::operator==(). We only did a pointer comparison which always failed when doing a style recalc since a style recalc always creates a new FontFeatureSetting object. R=eae@chromium.org BUG= 602802 , 602959 Review URL: https://codereview.chromium.org/1887613002 Cr-Commit-Position: refs/heads/master@{#387001} [add] https://crrev.com/3b6513eaede077dc4171f5a05157a8658279a68a/third_party/WebKit/LayoutTests/fast/css/font-feature-settings-assert-expected.txt [add] https://crrev.com/3b6513eaede077dc4171f5a05157a8658279a68a/third_party/WebKit/LayoutTests/fast/css/font-feature-settings-assert.html [add] https://crrev.com/3b6513eaede077dc4171f5a05157a8658279a68a/third_party/WebKit/LayoutTests/fast/css/invalidation/font-feature-settings-expected.txt [add] https://crrev.com/3b6513eaede077dc4171f5a05157a8658279a68a/third_party/WebKit/LayoutTests/fast/css/invalidation/font-feature-settings.html [modify] https://crrev.com/3b6513eaede077dc4171f5a05157a8658279a68a/third_party/WebKit/Source/platform/fonts/FontDescription.h [modify] https://crrev.com/3b6513eaede077dc4171f5a05157a8658279a68a/third_party/WebKit/Source/platform/fonts/FontFeatureSettings.cpp [modify] https://crrev.com/3b6513eaede077dc4171f5a05157a8658279a68a/third_party/WebKit/Source/platform/fonts/FontFeatureSettings.h
Issue 612656 has been merged into this issue.
Comment 1 by r...@opera.com
, Apr 13 2016