Bug 151373

Summary: Web Inspector: Probes sidebar panel sections should use NavigationBar
Product: WebKit Reporter: Matt Baker <mattbaker>
Component: Web InspectorAssignee: Matt Baker <mattbaker>
Severity: Normal CC: bburg, commit-queue, graouts, joepeck, mattbaker, nvasilyev, timothy, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 151377    
Description Flags
[Patch] Proposed Fix
[Patch] Proposed Fix
[Patch] Proposed Fix none

Description Matt Baker 2015-11-17 17:11:29 PST
Probes sidebar panel sections should use NavigationBar. Currently the add probe, clear samples, and remove probe buttons are added to the DOM manually, and lack tooltips and enabled/disabled state. The clear samples button should only be enabled when samples exist.
Comment 1 Radar WebKit Bug Importer 2015-11-17 17:12:21 PST
Comment 2 Matt Baker 2015-11-17 17:30:36 PST
Created attachment 265721 [details]
[Patch] Proposed Fix
Comment 3 Matt Baker 2015-11-17 19:16:29 PST
Created attachment 265727 [details]
[Patch] Proposed Fix
Comment 4 BJ Burg 2015-11-18 13:16:05 PST
Comment on attachment 265727 [details]
[Patch] Proposed Fix

Comment 5 WebKit Commit Bot 2015-11-18 14:03:55 PST
Comment on attachment 265727 [details]
[Patch] Proposed Fix

Rejecting attachment 265727 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-03', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 265727, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 500 characters of output:
fs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc ...
Currently at 192583 = 2cdccd4e25f76b98954bfe31ba68605844034e38
r192584 = 355e1772eddb165f3350675b090fc6b42d9696df
r192585 = d17e14a3625201fac8271edf221660edd2917f38
r192586 = cb5552d0076102582c50977bd8e82ce3e244a4d3
Done rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc
First, rewinding head to replay your work on top of it...
Fast-forwarded master to refs/remotes/origin/master.

Full output: http://webkit-queues.webkit.org/results/446845
Comment 6 Matt Baker 2015-11-18 14:50:49 PST
Created attachment 265783 [details]
[Patch] Proposed Fix
Comment 7 WebKit Commit Bot 2015-11-18 15:47:57 PST
Comment on attachment 265783 [details]
[Patch] Proposed Fix

Clearing flags on attachment: 265783

Committed r192594: <http://trac.webkit.org/changeset/192594>
Comment 8 WebKit Commit Bot 2015-11-18 15:48:00 PST
All reviewed patches have been landed.  Closing bug.