> DLOG("Invalid application focused, discarding the event");
> not correct any more, please update comment
I disagree. It means discarding the request to focus the new app. The code that follows is just rolling back any side effects of handling the request. I only added a special case to that code anyway, rather than change its meaning.
I could change the comment to "refocusing previous focus" or something like that. But it sounds like a wordier version of the "not doing anything" meaning we already have.
> DLOG("Invalid application focused, discarding the event");
> not correct any more, please update comment
I disagree. It means discarding the request to focus the new app. The code that follows is just rolling back any side effects of handling the request. I only added a special case to that code anyway, rather than change its meaning.
I could change the comment to "refocusing previous focus" or something like that. But it sounds like a wordier version of the "not doing anything" meaning we already have.
> Please add the checklist too /wiki.ubuntu. com/Process/ Merges/ Checklists/ Unity-Mir
> https:/
Done.
> "Oo" <- as before please just make empty string
Done.