Bug 206721

Summary: Web Inspector: consolidate UIProcess/WebProcess files into Inspector folders
Product: WebKit Reporter: Devin Rousso <hi>
Component: Web InspectorAssignee: Devin Rousso <hi>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, annulen, Basuke.Suzuki, bburg, cdumez, cgarcia, chris.reid, commit-queue, dbates, don.olmstead, ews-watchlist, gyuyoung.kim, hi, Hironori.Fujii, inspector-bugzilla-changes, japhet, joepeck, mcatanzaro, mkwst, pnormand, ross.kirsling, ryuan.choi, sergio, timothy, webkit-bug-importer, zan
Priority: P2 Keywords: InRadar
Version: WebKit Local Build   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch
achristensen: review+
Patch
none
Patch none

Description Devin Rousso 2020-01-23 17:23:05 PST
All of the Web Inspector code should really live in
 - Source/WebKit/UIProcess/Inspector
 - Source/WebKit/WebProcess/Inspector
rather than be spread around.

This will make it easier for future Web Inspector developers to find code related to Web Inspector.
Comment 1 Devin Rousso 2020-01-23 18:39:30 PST
Created attachment 388635 [details]
Patch
Comment 2 Devin Rousso 2020-01-23 18:42:10 PST
Created attachment 388636 [details]
Patch
Comment 3 Devin Rousso 2020-01-23 18:43:17 PST
Created attachment 388637 [details]
Patch
Comment 4 Devin Rousso 2020-01-23 18:54:38 PST
Created attachment 388640 [details]
Patch
Comment 5 Devin Rousso 2020-01-23 19:03:27 PST
Created attachment 388644 [details]
Patch
Comment 6 Devin Rousso 2020-01-23 20:24:08 PST
Created attachment 388647 [details]
Patch
Comment 7 Devin Rousso 2020-01-23 23:44:46 PST
Created attachment 388658 [details]
Patch
Comment 8 Don Olmstead 2020-01-24 09:12:43 PST
Any reason this is a cq? Devin? Socket side seems fine.
Comment 9 Devin Rousso 2020-01-24 10:44:23 PST
(In reply to Don Olmstead from comment #8)
> Any reason this is a cq? Devin?
I was waiting for EWS to be all green.

> Socket side seems fine.
Thanks for taking a look! =D
Comment 10 WebKit Commit Bot 2020-01-26 22:29:34 PST Comment hidden (obsolete)
Comment 11 Devin Rousso 2020-01-26 23:14:35 PST
Created attachment 388831 [details]
Patch
Comment 12 WebKit Commit Bot 2020-01-27 17:13:39 PST
Comment on attachment 388831 [details]
Patch

Clearing flags on attachment: 388831

Committed r255214: <https://trac.webkit.org/changeset/255214>
Comment 13 WebKit Commit Bot 2020-01-27 17:13:41 PST
All reviewed patches have been landed.  Closing bug.
Comment 14 Radar WebKit Bug Importer 2020-01-27 17:14:17 PST
<rdar://problem/58940928>
Comment 15 Devin Rousso 2020-01-28 01:46:54 PST
Unreviewed, speculative gtk build fix: <https://trac.webkit.org/r255231>