Bug 98828

Summary: Fix coding style of layout test fast/dom/timer-throttling-hidden-page.html
Product: WebKit Reporter: Kiran Muppala <cmuppala>
Component: Tools / TestsAssignee: Kiran Muppala <cmuppala>
Status: RESOLVED FIXED    
Severity: Normal CC: cmuppala, ojan, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Kiran Muppala
Reported 2012-10-09 15:35:32 PDT
Remove tab characters and use spaces exclusively. Fix placement of braces according to WebKit coding style. Rename variables previousT, d and t to more appropriate previousTime, date and time respectively.
Attachments
Patch (4.87 KB, patch)
2012-10-09 15:46 PDT, Kiran Muppala 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Kiran Muppala
Comment 1 2012-10-09 15:46:56 PDT
Created attachment 167866 [details] Patch
Ojan Vafai
Comment 2 2012-10-09 15:56:25 PDT
Comment on attachment 167866 [details] Patch Much better. Thanks!
WebKit Review Bot
Comment 3 2012-10-09 16:49:03 PDT
Comment on attachment 167866 [details] Patch Clearing flags on attachment: 167866 Committed r130827: <http://trac.webkit.org/changeset/130827>
WebKit Review Bot
Comment 4 2012-10-09 16:49:06 PDT
All reviewed patches have been landed. Closing bug.
Kiran Muppala
Comment 5 2012-10-09 16:51:25 PDT
(In reply to comment #2) > (From update of attachment 167866 [details]) > Much better. Thanks! Thanks for pointing these out.
Note You need to log in before you can comment on or make changes to this bug.