Add comment about nsLayoutUtils::GetCrossDocParentFrame usage in EventStateManager::DecideGestureEvent
Categories
(Core :: DOM: Events, task, P3)
Tracking
()
| Fission Milestone | Future |
People
(Reporter: hiro, Unassigned)
References
Details
(Whiteboard: [fission:m?])
Similar to bug 1725548, but for EventStateManager::DecideGestureEvent.
|
Reporter | |
Comment 1•3 years ago
|
||
Masayuki, again, do you know who is the right person to audit this?
|
||
Comment 2•3 years ago
|
||
Searchfox says this EventStateManager::DecideGestureEvent function is not covered by any tests:
Do these gesture events happen in content processes? If they only happen in the parent process, then this code should work correctly.
|
||
Comment 3•3 years ago
|
||
That particular code is parent process only.
I guess we can use this bug to add some comment there.
|
|
||
Updated•3 years ago
|
|
||
Updated•3 years ago
|
|
|
||
Comment 4•3 years ago
|
||
(In reply to Olli Pettay [:smaug] from comment #3)
That particular code is parent process only.
I guess we can use this bug to add some comment there.
If this code works as expected and this bug is just to add a comment, then this bug doesn't need to block Fission Milestone MVP.
|
|
||
Updated•3 years ago
|
|
|
||
Comment 5•3 years ago
|
||
(In reply to Chris Peterson [:cpeterson] from comment #4)
(In reply to Olli Pettay [:smaug] from comment #3)
That particular code is parent process only.
I guess we can use this bug to add some comment there.If this code works as expected and this bug is just to add a comment, then this bug doesn't need to block Fission Milestone MVP.
Since this bug is now about adding a code comment instead of auditing the function calls, I will update the bug summary.
Description
•