Bug 141006

Summary:

Web Inspector: Change Main Frame Status Buttons when debugging Augmented JSContext

Product:

WebKit

Reporter:

Joseph Pecoraro <joepeck>

Component:

Web Inspector

Assignee:

Joseph Pecoraro <joepeck>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

graouts, joepeck, jonowells, mattbaker, nvasilyev, timothy, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
[PATCH] Proposed Fix	timothy: review+

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