WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
NEW
33615
Rename InjectedScript
https://bugs.webkit.org/show_bug.cgi?id=33615
Summary
Rename InjectedScript
Geoffrey Garen
Reported
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.
Attachments
Add attachment
proposed patch, testcase, etc.
Frances Cornwall
Comment 1
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.
Frances Cornwall
Comment 2
2023-10-31 11:39:50 PDT
I am seeing that there is "injectedScript" in InspectorFrontEndAPI.js.
Frances Cornwall
Comment 3
2023-11-06 16:42:34 PST
Pull request:
https://github.com/WebKit/WebKit/pull/20075
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug