Bug 33615 - Rename InjectedScript
Summary: Rename InjectedScript
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore JavaScript (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-01-13 12:19 PST by Geoffrey Garen
Modified: 2023-11-06 16:42 PST (History)
9 users (show)

See Also:

Attachments
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 Geoffrey Garen 2010-01-13 12:19:18 PST 
From https://bugs.webkit.org/show_bug.cgi?id=33469:

> 1. I don't think "InjectedScript", which is a fairly general phrase, is a good
> name to give to this technology. I'd prefer a name that conveyed the fact that
> this technology is only safe to use in limited circumstances. How about
> "InspectorHost" (inside the inspected page) and "InspectorClient" (the Web
> Inspector UI).
> 
I agree with you that the naming is not the best. Currently the inspector
front-end delegate is called InjectedScriptHost. We should come up with
something more specific.
Comment 1 Frances Cornwall 2023-10-31 11:33:50 PDT 
If this wasn't renamed yet, let's rename InjectedScript to InspectorScript, it is only in use in the inspector parts.
Comment 2 Frances Cornwall 2023-10-31 11:39:50 PDT 
I am seeing that there is "injectedScript" in InspectorFrontEndAPI.js.
Comment 3 Frances Cornwall 2023-11-06 16:42:34 PST 
Pull request: https://github.com/WebKit/WebKit/pull/20075