Summary: | Move suspendAnimations to use Internals interface | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Gyuyoung Kim <gyuyoung.kim> | ||||||
Component: | Tools / Tests | Assignee: | Gyuyoung Kim <gyuyoung.kim> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | ap, rakuco, rniwa, roger_fong, tkent, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Gyuyoung Kim
2012-05-09 05:33:51 PDT
Created attachment 140927 [details]
Patch
Comment on attachment 140927 [details] Patch Attachment 140927 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/12642907 Created attachment 140936 [details]
Patch
CC'ing Ryosuke, Alexey Comment on attachment 140936 [details] Patch Clearing flags on attachment: 140936 Committed r116610: <http://trac.webkit.org/changeset/116610> All reviewed patches have been landed. Closing bug. This patch changed API that was being used by other apps, which consequently now causes these other apps to fail. Please be more careful in the future when making such changes. (In reply to comment #7) > This patch changed API that was being used by other apps, which consequently now causes these other apps to fail. Please be more careful in the future when making such changes. Could you add a message rule on Source/WebKit/win/WebFrame.h to http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/watchlist ? e.g. see AppleMacPublicApi http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/watchlist#L262 It wasn't obvious that this was a part of public API. (In reply to comment #7) > This patch changed API that was being used by other apps, which consequently now causes these other apps to fail. Please be more careful in the future when making such changes. There was similar problem in Bug 86704. I have tried to be more careful since the bug. Sorry for making inconvenience. |