зеркало из https://github.com/mozilla/gecko-dev.git
3d6d2d9586
We never removed the event listeners (the code was there, lol, but the function that was supposed to call into the tooltip listener returned NS_ERROR_NOT_IMPLEMENTED instead). Furthermore, we added an event listener each time we reframed an element, which is insane. Basically, each time an element with tooltip / tooltiptext gets its frame tree reconstructed, we add the even listener, again, and we never free it. Xidorn pointed out that this is not such a huge deal because we deduplicate event listeners per spec, but still... Move the code from the RestyleManager and the frame constructor to AfterSetAttr / BindToTree / UnbindFromTree in nsXULElement to hopefully make this saner. MozReview-Commit-ID: 6BQbIQJ87qt |
||
|---|---|---|
| .. | ||
| crashtests | ||
| test | ||
| XULDocument.cpp | ||
| XULDocument.h | ||
| moz.build | ||
| nsForwardReference.h | ||
| nsIController.idl | ||
| nsIControllers.idl | ||
| nsIXULOverlayProvider.idl | ||
| nsIXULSortService.idl | ||
| nsXULCommandDispatcher.cpp | ||
| nsXULCommandDispatcher.h | ||
| nsXULContentSink.cpp | ||
| nsXULContentSink.h | ||
| nsXULContentUtils.cpp | ||
| nsXULContentUtils.h | ||
| nsXULControllers.cpp | ||
| nsXULControllers.h | ||
| nsXULElement.cpp | ||
| nsXULElement.h | ||
| nsXULPopupListener.cpp | ||
| nsXULPopupListener.h | ||
| nsXULPrototypeCache.cpp | ||
| nsXULPrototypeCache.h | ||
| nsXULPrototypeDocument.cpp | ||
| nsXULPrototypeDocument.h | ||
| nsXULSortService.cpp | ||
| nsXULSortService.h | ||