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

Issue 2478583003: Remove StyleMatchedPropertiesCache runtime flag (status=stable) (Closed)

Created:
4 years, 1 month ago by rwlbuis
Modified:
4 years, 1 month ago
CC:
chromium-reviews, blink-reviews-style_chromium.org, blink-reviews-css, dglazkov+blink, apavlov+blink_chromium.org, darktears, blink-reviews
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Remove StyleMatchedPropertiesCache runtime flag (status=stable) BUG=402536 Committed: https://crrev.com/026f4b5637d371143efff48bac629f3c72f37056 Cr-Commit-Position: refs/heads/master@{#431256}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -4 lines) Patch
M third_party/WebKit/Source/core/css/resolver/StyleResolver.cpp View 2 chunks +1 line, -3 lines 0 comments Download
M third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 20 (9 generated)
rwlbuis
PTAL.
4 years, 1 month ago (2016-11-04 17:26:23 UTC) #3
rwlbuis
On 2016/11/04 17:26:23, rwlbuis wrote: > PTAL. Oh, looks like StyleSharing goes hand in hand ...
4 years, 1 month ago (2016-11-07 14:23:47 UTC) #4
rwlbuis
I think dstockwell may be out so pinging timloh.
4 years, 1 month ago (2016-11-08 20:22:49 UTC) #6
Timothy Loh
On 2016/11/08 20:22:49, rwlbuis wrote: > I think dstockwell may be out so pinging timloh. ...
4 years, 1 month ago (2016-11-10 03:36:50 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2478583003/1
4 years, 1 month ago (2016-11-10 11:08:31 UTC) #9
commit-bot: I haz the power
Try jobs failed on following builders: chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presubmit/builds/301287)
4 years, 1 month ago (2016-11-10 11:15:08 UTC) #11
rwlbuis
@haraken PTAL I guess I need Source/platform permission.
4 years, 1 month ago (2016-11-10 13:02:13 UTC) #13
haraken
LGTM
4 years, 1 month ago (2016-11-10 13:02:44 UTC) #14
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2478583003/1
4 years, 1 month ago (2016-11-10 13:05:39 UTC) #16
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 1 month ago (2016-11-10 15:00:51 UTC) #18
commit-bot: I haz the power
4 years, 1 month ago (2016-11-10 15:05:10 UTC) #20
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/026f4b5637d371143efff48bac629f3c72f37056
Cr-Commit-Position: refs/heads/master@{#431256}

Powered by Google App Engine
This is Rietveld 408576698