In r122769 (bug 91410) we added a special-case handling for replaced elements in InlineFlowBox::placeBoxesInBlockDirection to work around positioning issues in a couple of svg tests. This was needed to compensate for a workaround in the table painting code that was fixed in r131358 (bug 99364). As such the special-case handling for replaced elements is no longer needed in InlineFlowBox::placeBoxesInBlockDirection. Furthermore this caused problems with the caret positioning in come cases.
Created attachment 168966 [details] Patch
Created attachment 168970 [details] Patch
Comment on attachment 168970 [details] Patch Rejecting attachment 168970 [details] from commit-queue. Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2 Last 500 characters of output: ls/ChangeLog CONFLICT (content): Merge conflict in Tools/ChangeLog Failed to merge in the changes. Patch failed at 0001 MediaStream API: Add the chromium API for RTCDataChannel When you have resolved this problem run "git rebase --continue". If you would prefer to skip this patch, instead run "git rebase --skip". To restore the original branch and stop rebasing run "git rebase --abort". rebase refs/remotes/origin/master: command returned error: 1 Died at Tools/Scripts/update-webkit line 164. Full output: http://queues.webkit.org/results/14384310
(In reply to comment #3) > CONFLICT (content): Merge conflict in Tools/ChangeLog > Failed to merge in the changes. How is this even possible? That file is not included in the patch and thus by definition cannot conflict.
Comment on attachment 168970 [details] Patch trying again as the commit queue seems to be very confused
Filed bug 99507 for the commit queue problem.
Comment on attachment 168970 [details] Patch Clearing flags on attachment: 168970 Committed r131503: <http://trac.webkit.org/changeset/131503>
All reviewed patches have been landed. Closing bug.