Summary: | Introduce the "stacking container" concept. | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | vollick | ||||
Component: | WebKit Misc. | Assignee: | vollick | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | eric, noam, ojan.autocc, simon.fraser, webkit.review.bot, zeno | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 106142 | ||||||
Attachments: |
|
Description
vollick
2013-01-23 14:06:04 PST
Created attachment 184307 [details]
Patch
Comment on attachment 184307 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=184307&action=review > Source/WebCore/rendering/RenderLayer.h:434 > + // A stacking container can have z-order lists. All stacking contexts are > + // stacking containers, but the converse is not true. I presume you'll amend this comment to say when it's not true, later. Comment on attachment 184307 [details] Patch (In reply to comment #2) > (From update of attachment 184307 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=184307&action=review > > > Source/WebCore/rendering/RenderLayer.h:434 > > + // A stacking container can have z-order lists. All stacking contexts are > > + // stacking containers, but the converse is not true. > > I presume you'll amend this comment to say when it's not true, later. Yes. When I broaden the definition of stacking container in the next patch, I will make this comment more specific. Comment on attachment 184307 [details] Patch Clearing flags on attachment: 184307 Committed r140620: <http://trac.webkit.org/changeset/140620> All reviewed patches have been landed. Closing bug. |