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

Issue 23995017: Merge 157268 "Update containtingIsolate to go back all the way t..." (Closed)

Created:
7 years, 3 months ago by inferno
Modified:
7 years, 3 months ago
Reviewers:
jww
CC:
blink-reviews, dglazkov+blink, eae+blinkwatch, leviw+renderwatch, jchaffraix+rendering
Visibility:
Public.

Description

Merge 157268 "Update containtingIsolate to go back all the way t..." > Update containtingIsolate to go back all the way to top isolate from current root, rather than stopping at the first isolate it finds. This works because the current root is always updated with each isolate run. > > BUG=279277 > > Review URL: https://chromiumcodereview.appspot.com/23972003 TBR=jww@chromium.org Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=157785

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -5 lines) Patch
Source/core/rendering/InlineIterator.h View 1 chunk +1 line, -4 lines 0 comments Download
Source/core/rendering/RenderBlockLineLayout.cpp View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
inferno
7 years, 3 months ago (2013-09-13 20:57:58 UTC) #1
inferno
7 years, 3 months ago (2013-09-13 20:58:07 UTC) #2
Message was sent while issue was closed.
Committed patchset #0 manually as r157785.

Powered by Google App Engine
This is Rietveld 408576698