Summary: | Revert RenderApplet::intrinsicSize to integers | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Levi Weintraub <leviw> | ||||||
Component: | Layout and Rendering | Assignee: | Levi Weintraub <leviw> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | eae, eric, jchaffraix, simon.fraser, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 60318 | ||||||||
Attachments: |
|
Description
Levi Weintraub
2012-03-26 07:36:40 PDT
Created attachment 133809 [details]
Patch
Comment on attachment 133809 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=133809&action=review > Source/WebCore/rendering/RenderReplaced.cpp:-205 > -int RenderReplaced::computeIntrinsicLogicalWidth(RenderBox* contentRenderer, bool includeMaxWidth) const Really? Dead? (In reply to comment #2) > (From update of attachment 133809 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=133809&action=review > > > Source/WebCore/rendering/RenderReplaced.cpp:-205 > > -int RenderReplaced::computeIntrinsicLogicalWidth(RenderBox* contentRenderer, bool includeMaxWidth) const > > Really? Dead? I know... I was amazed, too, but I did my due diligence with grep. You can also note that they're non-virtual private functions and have now passed all EWS bots as well. Dead dead dead! Comment on attachment 133809 [details] Patch Rejecting attachment 133809 [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: rce/WebCore/rendering/RenderReplaced.cpp Hunk #1 FAILED at 202. 1 out of 1 hunk FAILED -- saving rejects to file Source/WebCore/rendering/RenderReplaced.cpp.rej patching file Source/WebCore/rendering/RenderReplaced.h Hunk #1 FAILED at 63. 1 out of 1 hunk FAILED -- saving rejects to file Source/WebCore/rendering/RenderReplaced.h.rej Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--force', u'--reviewer', u'Eric Seidel']" exit_code: 1 cwd: /mnt/git/webkit-commit-queue/ Full output: http://queues.webkit.org/results/12141453 (In reply to comment #4) > (From update of attachment 133809 [details]) > Rejecting attachment 133809 [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: > rce/WebCore/rendering/RenderReplaced.cpp > Hunk #1 FAILED at 202. > 1 out of 1 hunk FAILED -- saving rejects to file Source/WebCore/rendering/RenderReplaced.cpp.rej > patching file Source/WebCore/rendering/RenderReplaced.h > Hunk #1 FAILED at 63. > 1 out of 1 hunk FAILED -- saving rejects to file Source/WebCore/rendering/RenderReplaced.h.rej > > Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--force', u'--reviewer', u'Eric Seidel']" exit_code: 1 cwd: /mnt/git/webkit-commit-queue/ > > Full output: http://queues.webkit.org/results/12141453 Zimmerman beat me to it! I'll stick to the first part :) Created attachment 134018 [details]
Patch for landing
Comment on attachment 134018 [details] Patch for landing Clearing flags on attachment: 134018 Committed r112255: <http://trac.webkit.org/changeset/112255> All reviewed patches have been landed. Closing bug. |