[Webkit-unassigned] [Bug 117413] New: [CoordinatedGraphics] Typos in custom shader support

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=117413

           Summary: [CoordinatedGraphics] Typos in custom shader support
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: WebKit2
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: ralpht+bugs at gmail.com
                CC: noam at webkit.org, dongseong.hwang at intel.com,
                    jae.park at company100.net


CoordinatedLayerTreeHost.cpp:333 is

for (size_t j = 0; j < keyframes.size(); ++j) {
    const FilterAnimationValue& filterValue = static_cast<const FilterAnimationValue&>(keyframes.at(i));

but it should be:

for (size_t j = 0; j < keyframes.size(); ++j) {
    const FilterAnimationValue& filterValue = static_cast<const FilterAnimationValue&>(keyframes.at(j));


Also, WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsScene.cpp:730 has the same bug, it is:

for (size_t j = 0; j < keyframes.size(); ++j) {
    const FilterAnimationValue& filterValue = static_cast<const FilterAnimationValue&>(keyframes.at(i));

but should be:

for (size_t j = 0; j < keyframes.size(); ++j) {
    const FilterAnimationValue& filterValue = static_cast<const FilterAnimationValue&>(keyframes.at(j));

I'm not set up for submitting patches currently, but I'll try to get setup soon (in the meantime, if anyone else wants to submit  something for this then that would be wonderful ;) ).

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.