Bug 125061

Summary: Web Inspector: Remove unused functions from InspectorAgent
Product: WebKit Reporter: Seokju Kwon <seokju>
Component: Web InspectorAssignee: Seokju Kwon <seokju>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, graouts, joepeck, timothy, webkit-bug-importer
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch
none
Patch none

Seokju Kwon
Reported 2013-12-01 22:10:29 PST
Remove 'inspectedURLWithoutFragment()' from InspectorAgent as it's never called.
Attachments
Patch (2.01 KB, patch)
2013-12-01 22:13 PST, Seokju Kwon 		no flags
DetailsFormatted DiffDiff
Patch (2.62 KB, patch)
2013-12-02 00:39 PST, Seokju Kwon 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Seokju Kwon
Comment 1 2013-12-01 22:13:01 PST
Created attachment 218143 [details] Patch
Seokju Kwon
Comment 2 2013-12-02 00:36:37 PST
Remove 'inspectedURL()' as well. Patch is coming.
Seokju Kwon
Comment 3 2013-12-02 00:39:56 PST
Created attachment 218148 [details] Patch
Antoine Quint
Comment 4 2013-12-02 00:54:18 PST
Comment on attachment 218148 [details] Patch I couldn't find any use of these methods either. r+
Seokju Kwon
Comment 5 2013-12-02 00:56:08 PST
(In reply to comment #4) > (From update of attachment 218148 [details]) > I couldn't find any use of these methods either. r+ Thanks, I will land the patch after making build-bots green.
WebKit Commit Bot
Comment 6 2013-12-02 02:09:22 PST
Comment on attachment 218148 [details] Patch Clearing flags on attachment: 218148 Committed r159923: <http://trac.webkit.org/changeset/159923>
WebKit Commit Bot
Comment 7 2013-12-02 02:09:25 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.