Summary: | [TexMap] Redundant method in GraphicsLayerTextureMapper. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | peavo | ||||||
Component: | WebCore Misc. | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | alex.christensen, bfulgham, cmarcelo, commit-queue, kondapallykalyan, luiz, mrobinson, noam, pnormand, zan | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
peavo
2014-10-23 04:29:07 PDT
Created attachment 240343 [details]
Patch
Created attachment 242398 [details]
Patch
(In reply to comment #2) > Created attachment 242398 [details] > Patch Rebased. This seems ok, but I'd prefer to have someone who knows and works more with texmap review this. Why is startedAnimation there? Is anyone else using it? Were they? Will they? (In reply to comment #4) > This seems ok, but I'd prefer to have someone who knows and works more with > texmap review this. Why is startedAnimation there? Is anyone else using > it? Were they? Will they? Thanks for looking into this :) I added the startedAnimation method as part of the accelerated compositing patch for WinCairo, so I don't think anybody else is using it. It was pointed out to me that there already is a similar method for this. Comment on attachment 242398 [details]
Patch
In that case I'll put it in because it appears not to break Linux.
Comment on attachment 242398 [details] Patch Clearing flags on attachment: 242398 Committed r176748: <http://trac.webkit.org/changeset/176748> All reviewed patches have been landed. Closing bug. |