Make explicit which TextIndicator animations are driven manually, and which run autom...
[WebKit-https.git] / Source / WebKit2 / UIProcess / mac / PageClientImpl.mm
index ef9e8b30e45b7839e24f295732d7e08944b5f8c3..88abd4f494d34a642222b7c7cf69e11239366e0e 100644 (file)
@@ -569,9 +569,6 @@ void PageClientImpl::didPerformDictionaryLookup(const DictionaryPopupInfo& dicti
     RetainPtr<NSMutableDictionary> mutableOptions = adoptNS([(NSDictionary *)dictionaryPopupInfo.options.get() mutableCopy]);
 
     if (canLoadLUTermOptionDisableSearchTermIndicator() && dictionaryPopupInfo.textIndicator.contentImage) {
-        // Run the animations serially because attaching another subwindow breaks the bounce animation.
-        // We could consider making the bounce NSAnimationNonblockingThreaded instead, which seems
-        // to work, but need to consider all of the implications.
         [m_wkView _setTextIndicator:TextIndicator::create(dictionaryPopupInfo.textIndicator) fadeOut:NO];
         [mutableOptions setObject:@YES forKey:getLUTermOptionDisableSearchTermIndicator()];
         [getLULookupDefinitionModuleClass() showDefinitionForTerm:dictionaryPopupInfo.attributedString.string.get() atLocation:textBaselineOrigin options:mutableOptions.get()];