Bug 86675

Summary: Move showStyle from CSSStyleDeclaration to StylePropertySet
Product: WebKit Reporter: Ryosuke Niwa <rniwa>
Component: CSSAssignee: Ryosuke Niwa <rniwa>
Status: RESOLVED FIXED    
Severity: Normal CC: bdakin, cmarcelo, enrica, kling, koivisto, macpherson, menard, ojan, rakuco, simon.fraser, tony, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Moved
none
Fixed build none

Description Ryosuke Niwa 2012-05-16 14:31:00 PDT
The whole of showStyle was so that we can use it while debugging editing code, which mutates CSS properties quite often. It would be useful if showStyle was in StylePropertySet instead of CSSStyleDeclaration now that the most of internal code uses StylePropertySet.

Also, this function is the only reason we have CSSStyleDeclaration.cpp. We can remove that file once we moved the function.
Comment 1 Ryosuke Niwa 2012-05-16 15:58:47 PDT
Created attachment 142359 [details]
Moved
Comment 2 Ryosuke Niwa 2012-05-16 16:00:45 PDT
Created attachment 142360 [details]
Fixed build
Comment 3 Andreas Kling 2012-05-16 16:03:51 PDT
Comment on attachment 142360 [details]
Fixed build

LGreatTM!
Comment 4 WebKit Review Bot 2012-05-16 17:29:53 PDT
Comment on attachment 142360 [details]
Fixed build

Clearing flags on attachment: 142360

Committed r117369: <http://trac.webkit.org/changeset/117369>
Comment 5 WebKit Review Bot 2012-05-16 17:29:59 PDT
All reviewed patches have been landed.  Closing bug.