Summary: | Change RenderBlock to user RenderBox/RenderInline writing mode logic | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Emil A Eklund <eae> | ||||||||
Component: | Layout and Rendering | Assignee: | Emil A Eklund <eae> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | darin, eric, leviw, webkit.review.bot | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 85824 | ||||||||||
Attachments: |
|
Description
Emil A Eklund
2012-05-21 12:53:56 PDT
Created attachment 143081 [details]
Patch
Comment on attachment 143081 [details]
Patch
I might have named it "overrideStyle" instead of otherStyle. This seems OK. the usingStyle(style) pattern would be fine to.
Created attachment 143219 [details]
Patch for landing
Comment on attachment 143219 [details] Patch for landing Rejecting attachment 143219 [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: re::RenderStyle*)': Source/WebCore/rendering/RenderBox.cpp:171: error: 'otherStyle' was not declared in this scope Source/WebCore/rendering/RenderBox.cpp: In member function 'void WebCore::RenderBox::setMarginAfter(WebCore::LayoutUnit, const WebCore::RenderStyle*)': Source/WebCore/rendering/RenderBox.cpp:189: error: 'otherStyle' was not declared in this scope make: *** [out/Debug/obj.target/webcore_rendering/Source/WebCore/rendering/RenderBox.o] Error 1 make: *** Waiting for unfinished jobs.... Full output: http://queues.webkit.org/results/12730937 Created attachment 143243 [details]
Patch for landing
Comment on attachment 143243 [details] Patch for landing Clearing flags on attachment: 143243 Committed r117946: <http://trac.webkit.org/changeset/117946> All reviewed patches have been landed. Closing bug. |