Bug 141006 - Web Inspector: Change Main Frame Status Buttons when debugging Augmented JSContext
Summary: Web Inspector: Change Main Frame Status Buttons when debugging Augmented JSCo...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Joseph Pecoraro
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2015-01-28 12:23 PST by Joseph Pecoraro
Modified: 2015-01-28 18:01 PST (History)
7 users (show)

See Also:

Attachments
[PATCH] Proposed Fix (4.75 KB, patch)
2015-01-28 12:25 PST, Joseph Pecoraro 		timothy: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Joseph Pecoraro 2015-01-28 12:23:48 PST 
* SUMMARY
Change Main Frame Status Buttons when debugging Augmented JSContext. The Download Web Archive button doesn't really make sense. And change "Reload..." to "Restart...".
Comment 1 Radar WebKit Bug Importer 2015-01-28 12:24:12 PST 
<rdar://problem/19635014>
Comment 2 Joseph Pecoraro 2015-01-28 12:25:32 PST 
Created attachment 245563 [details]
[PATCH] Proposed Fix
Comment 3 Timothy Hatcher 2015-01-28 16:04:35 PST 
Comment on attachment 245563 [details]
[PATCH] Proposed Fix

View in context: https://bugs.webkit.org/attachment.cgi?id=245563&action=review

> Source/WebInspectorUI/UserInterface/Views/TreeElementStatusButton.js:60
> +        this._element.classList.toggle("hidden", flag);

Woot!
Comment 4 Joseph Pecoraro 2015-01-28 18:01:29 PST 
http://trac.webkit.org/changeset/179322