Bug 45308
| Summary: | Web Inspector: populateScriptObjects method of InspectorController is very big. | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ilya Tikhonovsky <loislo> |
| Component: | Web Inspector (Deprecated) | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED WONTFIX | ||
| Severity: | Normal | CC: | apavlov, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, rik, yurys |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | All | ||
| OS: | All | ||
| Bug Depends on: | 45759, 45887 | ||
| Bug Blocks: | |||
Ilya Tikhonovsky
At the inspector backend side we have some kind of magic method which push everything from backend to frontend.
It would be better to split this method.
there are number of things inside this method.
1) runtime settings like profilerEnable, monitoringXhr, searchForNode etc;
2) resource publishing mechanics;
3) scripts publishing mechanics;
etc.
all these things should be implemented as separate functions.
The first step of that is extracting runtime settings.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Alexander Pavlov (apavlov)
This method has been removed.