Bug 57432

Summary: Split out handling of trailing spaces from layoutInlineChildren
Product: WebKit Reporter: Eric Seidel <eric@webkit.org>
Component: New BugsAssignee: Eric Seidel <eric@webkit.org>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth@webkit.org, commit-queue@webkit.org, mitz@webkit.org, rniwa@webkit.org
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: Mac OS X 10.5   
Attachments:
Description Flags
Patch
none
Patch for landing none

Description From 2011-03-30 02:09:44 PST
Split out handling of trailing spaces from layoutInlineChildren
------- Comment #1 From 2011-03-30 02:11:29 PST -------
Created an attachment (id=87484) [details]
Patch
------- Comment #2 From 2011-03-30 08:10:45 PST -------
(From update of attachment 87484 [details])
Nice. I would have made this a static function with a const RenderBlock* parameter.
------- Comment #3 From 2011-03-30 08:21:59 PST -------
(From update of attachment 87484 [details])
(In reply to comment #2)
> (From update of attachment 87484 [details] [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!
------- Comment #4 From 2011-03-30 08:24:23 PST -------
(From update of attachment 87484 [details])
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
------- Comment #5 From 2011-04-03 00:29:30 PST -------
Created an attachment (id=88001) [details]
Patch for landing
------- Comment #6 From 2011-04-03 01:30:00 PST -------
(From update of attachment 88001 [details])
Clearing flags on attachment: 88001

Committed r82792: <http://trac.webkit.org/changeset/82792>
------- Comment #7 From 2011-04-03 01:30:05 PST -------
All reviewed patches have been landed.  Closing bug.