Summary: | [GTK] Port progressbar painting to GtkStyleContext | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Carlos Garcia Campos <cgarcia> | ||||
Component: | WebKitGTK | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | mrobinson | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | PC | ||||||
OS: | Linux | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 50820 | ||||||
Attachments: |
|
Description
Carlos Garcia Campos
2011-01-07 05:23:58 PST
Created attachment 78224 [details]
Use GtkStyleContext API to paint progressbars
Comment on attachment 78224 [details] Use GtkStyleContext API to paint progressbars View in context: https://bugs.webkit.org/attachment.cgi?id=78224&action=review Looks great, but please see below before landing this. > WebCore/platform/gtk/RenderThemeGtk3.cpp:369 > +static const int progressActivityBlocks = 5; > +static const int progressAnimationFrames = 10; > +static const double progressAnimationInterval = 0.125; Include a comment before these values which explains where they originate from. > WebCore/platform/gtk/RenderThemeGtk3.cpp:411 > + double progress = renderProgress->animationProgress(); I think this should be called animationProgress, to avoid confusing it with the actual progress position. Please just rename before landing. Thanks! Committed r75364: <http://trac.webkit.org/changeset/75364> |