Bug 57432 – Split out handling of trailing spaces from layoutInlineChildren

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Eric Seidel (no email)]

Split out handling of trailing spaces from layoutInlineChildren

[Eric Seidel (no email)]

Created attachment 87484 [details]
Patch

Comment on attachment 87484 [details]
Patch

Nice. I would have made this a static function with a const RenderBlock* parameter.

[Eric Seidel (no email)]

Comment on attachment 87484 [details]
Patch

(In reply to comment #2)
> (From update of attachment 87484 [details])
> Nice. I would have made this a static function with a const RenderBlock* parameter.

I thought about that.  But this reads cleaner (but has more perf risk).  If the PLT bots tell me it's slower I'll fix to do just that.

Thanks!

[WebKit Commit Bot]

Comment on attachment 87484 [details]
Patch

Rejecting attachment 87484 [details] from commit-queue.

Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=cr-jail-4', 'apply-..." exit_code: 2

Last 500 characters of output:
s).
patching file Source/WebCore/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file Source/WebCore/rendering/RenderBlock.h
patching file Source/WebCore/rendering/RenderBlockLineLayout.cpp
Hunk #1 FAILED at 608.
Hunk #2 succeeded at 853 (offset 21 lines).
1 out of 2 hunks FAILED -- saving rejects to file Source/WebCore/rendering/RenderBlockLineLayout.cpp.rej

Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--reviewer', u'Dan Bernstein', u'--fo..." exit_code: 1

Full output: http://queues.webkit.org/results/8298330

[Eric Seidel (no email)]

Created attachment 88001 [details]
Patch for landing

[WebKit Commit Bot]

Comment on attachment 88001 [details]
Patch for landing

Clearing flags on attachment: 88001

Committed r82792: <http://trac.webkit.org/changeset/82792>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.