Bug 59630 - Web Inspector: Stop on Exception state does not persist
Summary: Web Inspector: Stop on Exception state does not persist
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Ilya Tikhonovsky
URL:
Keywords:
Depends on: 59757
Blocks:
  Show dependency treegraph
 
Reported: 2011-04-27 13:50 PDT by Ilya Tikhonovsky
Modified: 2014-01-25 15:22 PST (History)
10 users (show)

See Also:


Attachments
[patch] initial version (1.97 KB, patch)
2011-04-27 14:16 PDT, Ilya Tikhonovsky
pfeldman: review-
Details | Formatted Diff | Diff
Fixed version was relanded as r85272. (1.51 KB, patch)
2011-04-28 17:37 PDT, Ilya Tikhonovsky
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ilya Tikhonovsky 2011-04-27 13:50:31 PDT
What steps will reproduce the problem?
1. Open developer tools
2. Click on stop on exception until it turns purple
3. Close and reopen developer tools

What is the expected result?

Button is purple

What happens instead?

Button is black

http://code.google.com/p/chromium/issues/detail?id=79456
Comment 1 Ilya Tikhonovsky 2011-04-27 14:16:07 PDT
Created attachment 91347 [details]
[patch] initial version
Comment 2 Ilya Tikhonovsky 2011-04-27 15:00:53 PDT
Comment on attachment 91347 [details]
[patch] initial version

landed as r85101
Comment 3 Pavel Feldman 2011-04-27 22:36:31 PDT
Comment on attachment 91347 [details]
[patch] initial version

When we migrated to the string values for pauleOnExceptionState, we needed to switch the property to the one with new name (to prevent failure on the first launch with old prefs). What exactly does not work now?
Comment 4 Pavel Feldman 2011-04-27 22:38:45 PDT
Comment on attachment 91347 [details]
[patch] initial version

Ok, I see, Settings.js needed to be updated.
Comment 5 Ilya Tikhonovsky 2011-04-28 17:37:57 PDT
Created attachment 91605 [details]
Fixed version was relanded as r85272.