Summary: | Counter still gets incremented when counter-increment is set to none | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Yi Shen <max.hong.shen> | ||||||
Component: | CSS | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | carol, commit-queue, esprehn+autocc, esprehn, hyatt, inferno, jchaffraix, macpherson, menard, ojan.autocc, rniwa, tony, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
Yi Shen
2012-04-26 12:01:20 PDT
Created attachment 139048 [details]
First try
Comment on attachment 139048 [details]
First try
This patch looks correct. Is there a reason it has not been reviewed? Does it violate some aspect of the counter specification?
Elliott was in the counters code recently, maybe he would like to do an unofficial review? I'll review later today, but looking quickly this doesn't seem right since you transform none into counter-increment: 0 for every counter which creates implicit scopes. Thanks for comments, guys. It is an old patch and I will rework on it. (In reply to comment #4) > I'll review later today, but looking quickly this doesn't seem right since you transform none into counter-increment: 0 for every counter which creates implicit scopes. Comment on attachment 139048 [details]
First try
Punting the patch out of the review queue based on Elliot's comment. If you come up with a new patch covering implicit scopes, I would be happy to look at it.
Created attachment 194371 [details]
clear counter increment when its value is none
Comment on attachment 194371 [details] clear counter increment when its value is none Clearing flags on attachment: 194371 Committed r147930: <http://trac.webkit.org/changeset/147930> All reviewed patches have been landed. Closing bug. |