Closed Bug 1132464 Opened 6 years ago Closed 6 years ago
Get rid of Event
State Manager::Get Child Process Offset
This function is confusing because the name implies it returns an offset but really it returns the refPoint of the provided aEvent, transformed into the child process's root widget coordinate space. i.e. it returns an absolute position and not an offset. It only appears to be used from TabParent which has its own GetChildProcessOffset that *does* return an offset, so we should just switch code over to using that instead.
This is easier to do once bug 1125040 converts TabParent::GetChildProcessOffset to return a LayoutDeviceIntPoint.
Depends on: 1125040
Will wait until bug 1125040 lands.
Comment on attachment 8565112 [details] [diff] [review] Patch Please test things like context menu handling etc.
Attachment #8565112 - Flags: review?(bugs) → review+
Tested various scenarios an in e10s window on OS X, things seem to work fine. The try push is green as well. https://hg.mozilla.org/integration/mozilla-inbound/rev/f10460c33eb6
You need to log in before you can comment on or make changes to this bug.