Last Comment Bug 767373 - The "Filter scripts" textbox is cleared when checking/unchecking the "Pause on exceptions" checkbox
: The "Filter scripts" textbox is cleared when checking/unchecking the "Pause o...
Product: Firefox
Classification: Client Software
Component: Developer Tools: Debugger (show other bugs)
: 15 Branch
: All All
P3 normal (vote)
: Firefox 16
Assigned To: Victor Porof [:vporof][:vp]
: Jason Laster [:jlast]
Depends on:
  Show dependency treegraph
Reported: 2012-06-22 07:33 PDT by Ioana (away)
Modified: 2012-07-06 04:01 PDT (History)
4 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

v1 (1.06 KB, patch)
2012-07-03 09:54 PDT, Victor Porof [:vporof][:vp]
no flags Details | Diff | Splinter Review
v1 (588 bytes, patch)
2012-07-03 13:33 PDT, Victor Porof [:vporof][:vp]
past: review+
Details | Diff | Splinter Review

Description User image Ioana (away) 2012-06-22 07:33:31 PDT
Mozilla/5.0 (Windows NT 6.1; rv:15.0) Gecko/20120621 Firefox/15.0a2 (20120621042006)

1. Load in the browser.
2. Open the Debugger (Tools->Web developer->Debugger, or press Ctrl+Shift+S).
3. Enter some text in the textbox to filter the scrips.
4. Check the "Pause on exceptions" checkbox.

The text entered in the textbox for filtering scripts is automatically removed when checking or unchecking the "Pause on exceptions" checkbox.
Comment 1 User image Victor Porof [:vporof][:vp] 2012-06-22 09:42:45 PDT
I'll take a look.
Comment 2 User image Victor Porof [:vporof][:vp] 2012-07-03 09:54:53 PDT
Created attachment 638777 [details] [diff] [review]

It seems that doing a updatePauseOnExceptions call has the effect of pausing and resuming the main thread, which triggers some events, which have the effect of clearing the searchbox.

We originally did this during the work week to avoid some inconsistencies, but it seems that everything is working ok so we can get rid of the behavior.
Comment 3 User image Victor Porof [:vporof][:vp] 2012-07-03 13:33:12 PDT
Created attachment 638843 [details] [diff] [review]

Wrong patch :)
Comment 4 User image Paul Rouget [:paul] 2012-07-05 08:15:06 PDT
Comment 5 User image Tim Taubert [:ttaubert] 2012-07-06 04:01:36 PDT

Note You need to log in before you can comment on or make changes to this bug.