https://bugs.webkit.org/show_bug.cgi?id=131861
<rdar://problem/
15663617>
Reviewed by David Hyatt.
.:
* ManualTests/empty-inline-as-line-break-position.html: Added.
This issue wouldn't reproduce in WebKitTestRunner.
Source/WebCore:
* rendering/line/BreakingContextInlineHeaders.h:
(WebCore::BreakingContext::canBreakAtThisPosition):
Don't break at this position if this is an empty RenderInline.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@167628
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
+2014-04-18 Jon Honeycutt <jhoneycutt@apple.com>
+
+ Empty RenderInline objects should not be line break objects.
+
+ https://bugs.webkit.org/show_bug.cgi?id=131861
+ <rdar://problem/15663617>
+
+ Reviewed by David Hyatt.
+
+ * ManualTests/empty-inline-as-line-break-position.html: Added.
+ This issue wouldn't reproduce in WebKitTestRunner.
+
2014-04-19 Brent Fulgham <bfulgham@apple.com>
Revert unintended workspace change in my last commit.
--- /dev/null
+<div>
+ WebKit bug #<a href="https://bugs.webkit.org/show_bug.cgi?id=131861">131861</a>: Empty inlines should not be line break objects. This test must be run manually. On success, you will see "Test passed" below.
+</div>
+<script>
+ window.onload = function() {
+ document.body.offsetTop;
+ b.style.display = "inline-block";
+
+ document.body.offsetTop;
+ a.removeChild(c);
+
+ window.resizeTo(window.outerWidth + 1, window.outerHeight + 1);
+ setTimeout(function() { x.innerHTML = "Test passed"; }, 100);
+ }
+</script>
+<div style="width: 0px;" id="x">foo<span id="a">
+ <span id="b"></span><span>bar</span><span id="c"></span>
+ </span>
+ >
+
+2014-04-18 Jon Honeycutt <jhoneycutt@apple.com>
+
+ Empty RenderInline objects should not be line break objects.
+
+ https://bugs.webkit.org/show_bug.cgi?id=131861
+ <rdar://problem/15663617>
+
+ Reviewed by David Hyatt.
+
+ * rendering/line/BreakingContextInlineHeaders.h:
+ (WebCore::BreakingContext::canBreakAtThisPosition):
+ Don't break at this position if this is an empty RenderInline.
+
2014-04-21 Enrica Casucci <enrica@apple.com>
[iOS WebKit2] support replacements for misspelled words.
if (m_width.committedWidth() && !m_width.fitsOnLine(m_currentCharacterIsSpace) && m_currWS == NOWRAP)
return true;
+ // Avoid breaking on empty inlines.
+ if (m_current.renderer()->isRenderInline() && isEmptyInline(toRenderInline(*m_current.renderer())))
+ return false;
+
// Avoid breaking before empty inlines.
if (m_nextObject && m_nextObject->isRenderInline() && isEmptyInline(toRenderInline(*m_nextObject)))
return false;