Bug 125061 - Web Inspector: Remove unused functions from InspectorAgent
Summary: Web Inspector: Remove unused functions from InspectorAgent
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: Seokju Kwon
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-12-01 22:10 PST by Seokju Kwon
Modified: 2013-12-02 02:09 PST (History)
5 users (show)

See Also:

Attachments
Patch (2.01 KB, patch)
2013-12-01 22:13 PST, Seokju Kwon 		no flags Details | Formatted Diff | Diff
Patch (2.62 KB, patch)
2013-12-02 00:39 PST, Seokju Kwon 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) 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 Seokju Kwon 2013-12-01 22:10:29 PST 
Remove 'inspectedURLWithoutFragment()' from InspectorAgent as it's never called.
Comment 1 Seokju Kwon 2013-12-01 22:13:01 PST 
Created attachment 218143 [details]
Patch
Comment 2 Seokju Kwon 2013-12-02 00:36:37 PST 
Remove 'inspectedURL()' as well. Patch is coming.
Comment 3 Seokju Kwon 2013-12-02 00:39:56 PST 
Created attachment 218148 [details]
Patch
Comment 4 Antoine Quint 2013-12-02 00:54:18 PST 
Comment on attachment 218148 [details]
Patch

I couldn't find any use of these methods either. r+
Comment 5 Seokju Kwon 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.
Comment 6 WebKit Commit Bot 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>
Comment 7 WebKit Commit Bot 2013-12-02 02:09:25 PST 
All reviewed patches have been landed.  Closing bug.