Bug 22641 - Calling pauseTransitionAtTimeOnElementWithId() with garbage property name asserts
Summary: Calling pauseTransitionAtTimeOnElementWithId() with garbage property name ass...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P2 Normal
Assignee: Pierre-Olivier Latour
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-12-03 18:26 PST by Simon Fraser (smfr)
Modified: 2008-12-04 12:44 PST (History)
0 users

See Also:


Attachments
Patch v1 (1.79 KB, patch)
2008-12-03 19:01 PST, Pierre-Olivier Latour
mitz: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Simon Fraser (smfr) 2008-12-03 18:26:59 PST
If a LayoutTest calls pauseTransitionAtTimeOnElementWithId() with an unrecognized property name, like:

layoutTestController.pauseTransitionAtTimeOnElementWithId("foopy", 0.5, "target");

it asserts here:
ASSERTION FAILED: !HashTranslator::equal(KeyTraits::emptyValue(), key)
(/Volumes/InternalData/Development/webkit/WebKit.git/WebKitBuild/Debug/JavaScriptCore.framework/PrivateHeaders/HashTable.h:455 void WTF::HashTable<Key, Value, Extractor, HashFunctions, Traits, KeyTraits>::checkKey(const T&) [with T = int, HashTranslator = WTF::IdentityHashTranslator<int, std::pair<int, WTF::RefPtr<WebCore::ImplicitAnimation> >, WTF::IntHash<unsigned int> >, Key = int, Value = std::pair<int, WTF::RefPtr<WebCore::ImplicitAnimation> >, Extractor = WTF::PairFirstExtractor<std::pair<int, WTF::RefPtr<WebCore::ImplicitAnimation> > >, HashFunctions = WTF::IntHash<unsigned int>, Traits = WTF::PairHashTraits<WTF::HashTraits<int>, WTF::HashTraits<WTF::RefPtr<WebCore::ImplicitAnimation> > >, KeyTraits = WTF::HashTraits<int>])
Comment 1 Pierre-Olivier Latour 2008-12-03 19:01:57 PST
Created attachment 25732 [details]
Patch v1
Comment 2 Pierre-Olivier Latour 2008-12-04 12:44:48 PST
Sending        WebCore/ChangeLog
Sending        WebCore/page/animation/CompositeAnimation.cpp
Transmitting file data ..
Committed revision 39002.