Bug 77967 - Web Inspector: Optional out arguments are not supported in the Web Inspector protocol, which breaks the implementation
Summary: Web Inspector: Optional out arguments are not supported in the Web Inspector ...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P1 Major
Assignee: Peter Rybin
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-02-07 04:53 PST by Alexander Pavlov (apavlov)
Modified: 2012-02-08 08:17 PST (History)
12 users (show)

See Also:


Attachments
Patch (2.17 KB, patch)
2012-02-07 14:09 PST, Peter Rybin
no flags Details | Formatted Diff | Diff
original InspectorBackendDispatcher.cpp (189.89 KB, text/plain)
2012-02-08 07:32 PST, Peter Rybin
no flags Details
new InspectorBackendDispatcher.cpp (189.83 KB, text/plain)
2012-02-08 07:33 PST, Peter Rybin
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Pavlov (apavlov) 2012-02-07 04:53:52 PST
Please fix ASAP.
Comment 1 Peter Rybin 2012-02-07 14:09:31 PST
Created attachment 125924 [details]
Patch
Comment 2 Peter Rybin 2012-02-07 14:15:32 PST
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.
Comment 3 Yury Semikhatsky 2012-02-08 06:03:09 PST
Could you attach generated files diff?
Comment 4 Peter Rybin 2012-02-08 07:32:18 PST
Created attachment 126080 [details]
original InspectorBackendDispatcher.cpp
Comment 5 Peter Rybin 2012-02-08 07:33:54 PST
Created attachment 126081 [details]
new InspectorBackendDispatcher.cpp
Comment 6 WebKit Review Bot 2012-02-08 08:17:36 PST
Comment on attachment 125924 [details]
Patch

Clearing flags on attachment: 125924

Committed r107091: <http://trac.webkit.org/changeset/107091>
Comment 7 WebKit Review Bot 2012-02-08 08:17:41 PST
All reviewed patches have been landed.  Closing bug.