Summary: | Infinite loop when processing mouse events synchronously | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Carlos Garcia Campos <cgarcia> | ||||
Component: | WebKit2 | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | andersca, bugs-noreply, darin, sam | ||||
Priority: | P2 | ||||||
Version: | WebKit Local Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 153740 | ||||||
Attachments: |
|
Description
Carlos Garcia Campos
2016-02-08 11:06:00 PST
Created attachment 270867 [details]
Patch
Comment on attachment 270867 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=270867&action=review > Source/WebKit2/UIProcess/WebPageProxy.cpp:4581 > + auto nextMouseMoveEvent = WTFMove(m_nextMouseMoveEvent); > + handleMouseEvent(*nextMouseMoveEvent); Could also write this as a one-liner using std::exchange. (In reply to comment #2) > Comment on attachment 270867 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=270867&action=review > > > Source/WebKit2/UIProcess/WebPageProxy.cpp:4581 > > + auto nextMouseMoveEvent = WTFMove(m_nextMouseMoveEvent); > > + handleMouseEvent(*nextMouseMoveEvent); > > Could also write this as a one-liner using std::exchange. Right, good point. Committed r196264: <http://trac.webkit.org/changeset/196264> |