If InjectedScript is to be moved down to JavaScriptCore it must be scrubbed of DOM references. The last references that exist are "Node" related. They are all just wrappers around existing functions. Eliminate the wrappers, and where possible move Node <-> JSNode conversions to InspectorDOMAgent static interfaces.
<rdar://problem/15712401>
Created attachment 219807 [details] [PATCH] Proposed Fix Yay! This is the last of a set of "in WebCore" cleanup patches. The next patch will be moving a BUNCH of files into JavaScriptCore to get RuntimeAgent working there. These recent patches will make it so when the files move there are as few as possible diffs between then files, making reviewing easier.
Created attachment 219810 [details] [PATCH] Proposed Fix Rebaselined.
Comment on attachment 219810 [details] [PATCH] Proposed Fix Clearing flags on attachment: 219810 Committed r160955: <http://trac.webkit.org/changeset/160955>
All reviewed patches have been landed. Closing bug.
This change breaks the Windows Release build, because the compiler can't decide if the PropertyDescriptor referenced in the code refers to JSC::PropertyDescriptor, or Inspector::TypeBuilder::Runtime::PropertyDescriptor, likewise with CallFrame. You and I can figure it out (as can the Debug build), but the Release build is missing something (despite the 'using' declaration.) My laptop decided to stop booting yesterday, so I can't fix it remotely. could you please check in a fix?
I think I got a fix in with: <http://trac.webkit.org/changeset/161091>