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

Issue 688223002: Get rid of isWritingModeRoot. (Closed)

Created:
6 years, 1 month ago by ojan
Modified:
6 years, 1 month ago
Reviewers:
esprehn
CC:
mojo-reviews_chromium.org, esprehn, abarth-chromium
Base URL:
git@github.com:domokit/mojo.git@master
Project:
mojo
Visibility:
Public.

Description

Get rid of isWritingModeRoot. This function always returns false now that we don't have writing modes. TBR=esprehn@chromium.org Remove flipForWritingMode. It's a noop now that we don't have writing modes. Committed: https://chromium.googlesource.com/external/mojo/+/98686a4b7e6d7a4af9151845240fb97a417d694c

Patch Set 1 #

Patch Set 2 : merge to ToT #

Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -45 lines) Patch
M sky/engine/core/rendering/RenderBlock.cpp View 1 6 chunks +9 lines, -32 lines 0 comments Download
M sky/engine/core/rendering/RenderBlockFlow.cpp View 3 chunks +4 lines, -9 lines 0 comments Download
M sky/engine/core/rendering/RenderBox.h View 1 1 chunk +0 lines, -2 lines 0 comments Download
M sky/engine/core/rendering/RenderBox.cpp View 1 1 chunk +1 line, -1 line 0 comments Download
M sky/engine/core/rendering/RenderFlexibleBox.cpp View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 1 (0 generated)
ojan
6 years, 1 month ago (2014-10-30 02:53:09 UTC) #1
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
98686a4b7e6d7a4af9151845240fb97a417d694c (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698