diff options
| author | Tor Arne Vestbø <tor.arne.vestbo@qt.io> | 2016-10-12 15:33:13 +0200 |
|---|---|---|
| committer | Tor Arne Vestbø <tor.arne.vestbo@qt.io> | 2016-10-13 16:52:01 +0000 |
| commit | c79710a1819d906bbcf98e3fa2a1133e5c5cb66e (patch) | |
| tree | cf7ff8e4c4de051ca9d242817e2747b65c409fff /src/plugins/platforms/cocoa/qnsviewaccessibility.mm | |
| parent | 2f505b79a49bdf5ba8d084e13ab339bcf956c849 (diff) | |
macOS: Remove QNSView member m_window in favor of going via m_platformWindow
The two should never be out of sync, but by having them as separate members
we risk that they do. By going though m_platformWindow for QWindow access,
it's also more clear in the callsites that we're dealing with a QWindow
instead of a NSWindow, as referenced though self.window. Finally, removing
the member slims down memory use of a QNSView, however small.
Change-Id: Iec96cebf813fae82d3af339331781419f234c28b
Reviewed-by: Morten Johan Sørvig <morten.sorvig@qt.io>
Diffstat (limited to 'src/plugins/platforms/cocoa/qnsviewaccessibility.mm')
| -rw-r--r-- | src/plugins/platforms/cocoa/qnsviewaccessibility.mm | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/plugins/platforms/cocoa/qnsviewaccessibility.mm b/src/plugins/platforms/cocoa/qnsviewaccessibility.mm index bfb4ba8124..73e1f41dd5 100644 --- a/src/plugins/platforms/cocoa/qnsviewaccessibility.mm +++ b/src/plugins/platforms/cocoa/qnsviewaccessibility.mm @@ -53,10 +53,10 @@ @implementation QNSView (QNSViewAccessibility) - (id)childAccessibleElement { - if (!m_window->accessibleRoot()) + if (!m_platformWindow->window()->accessibleRoot()) return nil; - QAccessible::Id childId = QAccessible::uniqueId(m_window->accessibleRoot()); + QAccessible::Id childId = QAccessible::uniqueId(m_platformWindow->window()->accessibleRoot()); return [QMacAccessibilityElement elementWithId: childId]; } |