Bug 94069 - Web Inspector: CodeMirrorTextEditor doesn't clear execution line
Summary: Web Inspector: CodeMirrorTextEditor doesn't clear execution line
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: Jan Keromnes
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-08-14 22:22 PDT by Jan Keromnes
Modified: 2012-08-14 23:25 PDT (History)
11 users (show)

See Also:


Attachments
Patch (1.47 KB, patch)
2012-08-14 22:25 PDT, Jan Keromnes
no flags Details | Formatted Diff | Diff
Patch (2.55 KB, patch)
2012-08-14 22:39 PDT, Jan Keromnes
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jan Keromnes 2012-08-14 22:22:04 PDT
One of the last changes in #93686 broke the clearing of execution lines. It added a typeof == "number" check on a handle.
Comment 1 Jan Keromnes 2012-08-14 22:25:41 PDT
Created attachment 158503 [details]
Patch
Comment 2 Jan Keromnes 2012-08-14 22:39:36 PDT
Created attachment 158505 [details]
Patch
Comment 3 WebKit Review Bot 2012-08-14 23:25:53 PDT
Comment on attachment 158505 [details]
Patch

Clearing flags on attachment: 158505

Committed r125650: <http://trac.webkit.org/changeset/125650>
Comment 4 WebKit Review Bot 2012-08-14 23:25:57 PDT
All reviewed patches have been landed.  Closing bug.