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

Issue 2595833002: Replaced usages of WritingMode to use inline utility functions instead (Closed)

Created:
4 years ago by sashab
Modified:
4 years ago
Reviewers:
haraken
CC:
darktears, apavlov+blink_chromium.org, blink-reviews, blink-reviews-css, blink-reviews-layout_chromium.org, chromium-reviews, dglazkov+blink, eae+blinkwatch, jchaffraix+rendering, leviw+renderwatch, pdr+renderingwatchlist_chromium.org, rwlbuis, szager+layoutwatch_chromium.org, zoltan1, alancutter (OOO until 2018)
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Replaced usages of WritingMode to use inline utility functions instead Replaced simple usages of WritingMode to use inline utility functions instead of the enums directly. This is preparing to rename the values of WritingMode to match the writing-mode keywords, which are less self- documenting. BUG=628043 Committed: https://crrev.com/a5b97bd742ab2fbf7946c15aef175fe72ffaebae Cr-Commit-Position: refs/heads/master@{#440317}

Patch Set 1 #

Patch Set 2 : Small fixes #

Patch Set 3 : Rebase and fixed ::blink with just blink:: #

Unified diffs Side-by-side diffs Delta from patch set Stats (+25 lines, -24 lines) Patch
M third_party/WebKit/Source/core/css/CSSProperty.cpp View 1 2 3 chunks +4 lines, -4 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutBox.cpp View 1 2 2 chunks +4 lines, -4 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutFlexibleBox.cpp View 1 2 2 chunks +11 lines, -10 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutTableCell.cpp View 1 2 4 chunks +4 lines, -4 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutVTTCue.cpp View 2 chunks +2 lines, -2 lines 0 comments Download

Dependent Patchsets:

Messages

Total messages: 20 (11 generated)
sashab
haraken@ ptal. Decided to go ahead with the plan to make generated ComputedStyle use the ...
4 years ago (2016-12-21 05:33:09 UTC) #2
haraken
LGTM
4 years ago (2016-12-21 05:51:34 UTC) #3
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/2595833002/1
4 years ago (2016-12-21 23:29:51 UTC) #5
commit-bot: I haz the power
Try jobs failed on following builders: ios-device on master.tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds/127021) ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED, ...
4 years ago (2016-12-21 23:32:39 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/2595833002/20001
4 years ago (2016-12-22 00:34:02 UTC) #10
commit-bot: I haz the power
Try jobs failed on following builders: ios-device on master.tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds/127065) ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED, ...
4 years ago (2016-12-22 00:35:53 UTC) #12
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/2595833002/40001
4 years ago (2016-12-22 00:58:45 UTC) #15
commit-bot: I haz the power
Committed patchset #3 (id:40001)
4 years ago (2016-12-22 02:39:56 UTC) #18
commit-bot: I haz the power
4 years ago (2016-12-22 02:41:54 UTC) #20
Message was sent while issue was closed.
Patchset 3 (id:??) landed as
https://crrev.com/a5b97bd742ab2fbf7946c15aef175fe72ffaebae
Cr-Commit-Position: refs/heads/master@{#440317}

Powered by Google App Engine
This is Rietveld 408576698