Code review comment for lp:~mzanetti/unity-mir/screenshotting-focusing-api

Revision history for this message
Gerry Boland (gerboland) wrote :

+void ApplicationManager::screenshotUpdated()
+ if (!m_nextFocusedAppId.isEmpty()) {
+ Q_EMIT focusRequested(m_nextFocusedAppId);
+ m_nextFocusedAppId.clear();
+ }
I don't follow this. Why emit a focusRequest after screenshot is updated? Is it just the best time to do it?

review: Needs Information

« Back to merge proposal