| Summary: | Web Inspector: Optional out arguments are not supported in the Web Inspector protocol, which breaks the implementation | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Alexander Pavlov (apavlov) <apavlov> | ||||||||
| Component: | Web Inspector (Deprecated) | Assignee: | Peter Rybin <peter.rybin> | ||||||||
| Status: | RESOLVED FIXED | ||||||||||
| Severity: | Major | CC: | apavlov, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, prybin, rik, timothy, webkit.review.bot, yurys | ||||||||
| Priority: | P1 | ||||||||||
| Version: | 528+ (Nightly build) | ||||||||||
| Hardware: | All | ||||||||||
| OS: | All | ||||||||||
| Attachments: |
|
||||||||||
|
Description
Alexander Pavlov (apavlov)
2012-02-07 04:53:52 PST
Created attachment 125924 [details]
Patch
Alexander, I propose this patch. It support optional returns for RefPtr-based types. However you would have to manually clear RefPtr in order to not return a value. You might expect that the proper way would be to pass empty RefPtr in the first place, but unfortunately this is much harder to implement right now and it would interfere with a refactoring that I'm doing on the same part of generator code. Please consider using this schema as a temporary solution. You can put FIXME comment at each RefPtr clearing site. I'm going to iterate over all returns in the future when landing my refactoring. Could you attach generated files diff? Created attachment 126080 [details]
original InspectorBackendDispatcher.cpp
Created attachment 126081 [details]
new InspectorBackendDispatcher.cpp
Comment on attachment 125924 [details] Patch Clearing flags on attachment: 125924 Committed r107091: <http://trac.webkit.org/changeset/107091> All reviewed patches have been landed. Closing bug. |