Time range is not selectable by mouse when “Clear recent history” is opened via shortcut

RESOLVED FIXED in mozilla13

Status

()

Core
Widget: Gtk
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: nicolas m, Assigned: karlt)

Tracking

({regression})

13 Branch
mozilla13
x86
Linux
regression
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

5 years ago
User Agent: Mozilla/5.0 (X11; Linux i686; rv:13.0a1) Gecko/20120212 Firefox/13.0a1
Build ID: 20120212031149

Steps to reproduce:

This bug appears when accessing the “Clear recent history” window with the keyboard shortcut (ctrl+shift+del) or via the “Privacy” tabs in the preference panel. It does _not_ happen when opening the window through the menu.

(Firefox 13/Linux)


Actual results:

The “Time range to clear” select menu is frozen to the last used option, making impossible to choose a time range.


Expected results:

The menu should be naturally openable and selectable, exactly like it worked before or when the window is accessed via the menu.

Comment 1

5 years ago
WFM:
Mozilla/5.0 (X11; Linux x86_64; rv:13.0a1) Gecko/20120212 Firefox/13.0a1

Comment 2

5 years ago
Confirmed on
http://hg.mozilla.org/mozilla-central/rev/7c0ba1c98ff7
Mozilla/5.0 (X11; Linux i686; rv:13.0a1) Gecko/20120209 Firefox/13.0a1 ID:20120209031242

Regression window(m-c)
Works:
http://hg.mozilla.org/mozilla-central/rev/02e4c23ae67d
Mozilla/5.0 (X11; Linux i686; rv:13.0a1) Gecko/20120207 Firefox/13.0a1 ID:20120207125604
Fails:
http://hg.mozilla.org/mozilla-central/rev/b45785802731
Mozilla/5.0 (X11; Linux i686; rv:13.0a1) Gecko/20120207 Firefox/13.0a1 ID:20120207174850
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=02e4c23ae67d&tochange=b45785802731

Triggered by;
b45785802731	Karl Tomlinson — b=724966 make sure to set sRetryGrabTime for retrying grabs r=roc
Blocks: 724966
Status: UNCONFIRMED → NEW
Component: Untriaged → Widget: Gtk
Ever confirmed: true
Keywords: regression
Product: Firefox → Core
QA Contact: untriaged → gtk

Comment 3

5 years ago
Sorry error in Comment 2

Confirmed on
http://hg.mozilla.org/mozilla-central/rev/9253f058824a
Mozilla/5.0 (X11; Linux i686; rv:13.0a1) Gecko/20120212 Firefox/13.0a1 ID:20120212031149

Comment 4

5 years ago
I can also reproduce on  x86_64 Ubuntu 10.04 LTS + GNOME 2.30.2
http://hg.mozilla.org/mozilla-central/rev/9253f058824a
Mozilla/5.0 (X11; Linux x86_64; rv:13.0a1) Gecko/20120212 Firefox/13.0a1 ID:20120212031149

Updated

5 years ago
Summary: Time range is not selectable when “Clear recent history” is opened via shortcut → Time range is not selectable by mouse when “Clear recent history” is opened via shortcut
(Assignee)

Updated

5 years ago
Assignee: nobody → karlt
Status: NEW → ASSIGNED
(Assignee)

Comment 5

5 years ago
gtk_get_current_event_time is returning the time of an old event due to a nested event loop in ShowModal.  The last event in the inner loop (button press) has popped off the current event list and the grab is run off a nsXULPopupShowingEvent. At that point, the event remaining on the current event list is the button release that opened the modal window.

A button press has happened since that button release event.  Each button press induces an implicit pointer grab, which i assume updates the last-pointer-grab time and so the old time is not new enough for a new grab.  I'll write something to come up with a better time than gtk_get_current_event_time.
(Assignee)

Comment 6

5 years ago
Created attachment 596561 [details] [diff] [review]
rework last event time calculation
Attachment #596561 - Flags: review?(roc)
(Assignee)

Updated

5 years ago
Blocks: 725685, 726410
Hey Karl,

Thanks for putting this together. I built TB with this patch, and it seemed to fix the problem.

However, I'm getting the following error when attempting to do a try build:

/buildbot/try-comm-cen-lnx/build/mozilla/widget/gtk2/nsWindow.cpp: In function 'void UpdateLastInputEventTime(const void*)':
/buildbot/try-comm-cen-lnx/build/mozilla/widget/gtk2/nsWindow.cpp:388:69: error: invalid conversion from 'const GdkEvent*' to 'GdkEvent*'
/buildbot/try-comm-cen-lnx/build/mozilla/widget/gtk2/nsWindow.cpp:388:69: error:   initializing argument 1 of 'guint32 gdk_event_get_time(GdkEvent*)'


Is the wrong type being passed somehow?

-Mike
(Assignee)

Comment 8

5 years ago
Created attachment 596576 [details] [diff] [review]
rework last event time calculation v1.1

The gdk_event_get_time on the build machines needs a GdkEvent* instead of const GdkEvent*.
Attachment #596561 - Attachment is obsolete: true
Attachment #596561 - Flags: review?(roc)
Attachment #596576 - Flags: review?(roc)
Attachment #596576 - Flags: review?(roc) → review+
Karl:

This patch seems to fix bug 726410 - however, we're seeing the same popup test failures as before (bug 725685).

-Mike

Updated

5 years ago
No longer blocks: 725685
(Assignee)

Comment 10

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/7d542384106f
Target Milestone: --- → mozilla13
https://hg.mozilla.org/mozilla-central/rev/7d542384106f
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
(Assignee)

Updated

5 years ago
Duplicate of this bug: 726410
(Assignee)

Updated

5 years ago
No longer blocks: 726410
You need to log in before you can comment on or make changes to this bug.