Merge lp:~azzar1/unity/fix-1308572 into lp:unity

Proposed by Andrea Azzarone
Status: Merged
Approved by: Brandon Schaefer
Approved revision: no longer in the source branch.
Merged at revision: 3787
Proposed branch: lp:~azzar1/unity/fix-1308572
Merge into: lp:unity
Diff against target: 23 lines (+5/-1)
1 file modified
lockscreen/UserPromptView.cpp (+5/-1)
To merge this branch: bzr merge lp:~azzar1/unity/fix-1308572
Reviewer Review Type Date Requested Status
Robert Ancell Approve
PS Jenkins bot (community) continuous-integration Needs Fixing
Brandon Schaefer (community) Approve
Review via email: mp+216216@code.launchpad.net

Commit message

Do not allow to activate twice the same entry!

Description of the change

Do not allow to activate twice the same entry. Damn you nux!!

To post a comment you must log in.
Revision history for this message
Brandon Schaefer (brandontschaefer) wrote :

Nice, fixes crash for me!

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Robert Ancell (robert-ancell) wrote :

Confirmed fixed here doing a full package build from this branch.
unity 7.2.0+14.04.20140414.1-0ubuntu1 crashes in approximately 10s by holding down enter in the lock screen. After crash session is unlocked.

Tested this branch for over a minute with no crashes occuring.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lockscreen/UserPromptView.cpp'
--- lockscreen/UserPromptView.cpp 2014-04-01 19:05:07 +0000
+++ lockscreen/UserPromptView.cpp 2014-04-16 21:34:37 +0000
@@ -348,6 +348,11 @@
348 nux::GetWindowCompositor().SetKeyFocusArea(text_entry);348 nux::GetWindowCompositor().SetKeyFocusArea(text_entry);
349349
350 text_entry->activated.connect([this, text_input, promise](){350 text_entry->activated.connect([this, text_input, promise](){
351 auto* text_entry = text_input->text_entry();
352
353 if (!text_entry->GetInputEventSensitivity())
354 return;
355
351 if (focus_queue_.size() == 1)356 if (focus_queue_.size() == 1)
352 {357 {
353 text_input->SetSpinnerVisible(true);358 text_input->SetSpinnerVisible(true);
@@ -355,7 +360,6 @@
355 }360 }
356361
357 focus_queue_.pop_front();362 focus_queue_.pop_front();
358 auto* text_entry = text_input->text_entry();
359 cached_focused_geo_ = text_entry->GetGeometry();363 cached_focused_geo_ = text_entry->GetGeometry();
360 text_entry->SetInputEventSensitivity(false);364 text_entry->SetInputEventSensitivity(false);
361 QueueRelayout();365 QueueRelayout();