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

Issue 1145953003: [Effen] Move responsibility for inserting nodes to the OneChildListRenderNodeWrapper. (Closed)

Created:
5 years, 7 months ago by Hixie
Modified:
5 years, 7 months ago
Reviewers:
abarth-chromium
CC:
abarth-chromium, gregsimon, mojo-reviews_chromium.org, ojan, qsr+mojo_chromium.org
Base URL:
https://github.com/domokit/mojo.git@master
Target Ref:
refs/heads/master
Project:
mojo
Visibility:
Public.

Description

[Effen] Move responsibility for inserting nodes to the OneChildListRenderNodeWrapper. Before, OneChildListRenderNodeWrappers were responsible for removing the child nodes' RenderCSS nodes from their containers, and for moving those around when nodes were reordered. Now, they're also responsible for adding them. R=abarth@chromium.org Committed: https://chromium.googlesource.com/external/mojo/+/5418b3e0782abd2f724b637b0dfbb359633e0fa6

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+47 lines, -18 lines) Patch
M sky/sdk/lib/framework/fn.dart View 18 chunks +47 lines, -18 lines 0 comments Download

Messages

Total messages: 3 (1 generated)
abarth-chromium
lgtm
5 years, 7 months ago (2015-05-20 22:20:24 UTC) #2
Hixie
5 years, 7 months ago (2015-05-20 22:56:15 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
5418b3e0782abd2f724b637b0dfbb359633e0fa6 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698