Bug 42462 - Web Inspector: Clean-up Inspector interface generator.
Summary: Web Inspector: Clean-up Inspector interface generator.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Ilya Tikhonovsky
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-07-16 09:31 PDT by Ilya Tikhonovsky
Modified: 2010-07-16 12:42 PDT (History)
11 users (show)

See Also:


Attachments
[patch] initial version. (23.64 KB, patch)
2010-07-16 09:35 PDT, Ilya Tikhonovsky
yurys: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ilya Tikhonovsky 2010-07-16 09:31:39 PDT
%subj%

The current implementation of generator is not ready for generation Backend part of Inspector interface.
The full patch with Backend is quite big and I've split it.
Comment 1 Ilya Tikhonovsky 2010-07-16 09:35:43 PDT
Created attachment 61816 [details]
[patch] initial version.
Comment 2 Yury Semikhatsky 2010-07-16 09:50:29 PDT
Comment on attachment 61816 [details]
[patch] initial version.

WebCore/inspector/CodeGeneratorInspector.pm:40
 +  };
Maybe accessorSuffix to be more specific?

WebCore/inspector/CodeGeneratorInspector.pm:104
 +  my $callId = new domSignature();
domSignature sounds weird to me

WebCore/inspector/CodeGeneratorInspector.pm:176
 +      my @argsFiltered = grep($_->direction eq "out", @{$function->parameters});
Some comment would clarify this stuff.
Comment 3 Joseph Pecoraro 2010-07-16 09:57:23 PDT
(In reply to comment #2)
> (From update of attachment 61816 [details])
> WebCore/inspector/CodeGeneratorInspector.pm:40
>  +  };
> Maybe accessorSuffix to be more specific?

For now I like "pushSuffix". Its the method used to push the result into an
InspectorValue. But, it may also be used later for pulling out of an
InspectorValue, when generating a dispatch function. If so I don't know
whats best, accessorSuffix may be good enough.
Comment 4 Ilya Tikhonovsky 2010-07-16 11:41:33 PDT
Committed r63561
	M	WebCore/ChangeLog
	M	WebCore/inspector/CodeGeneratorInspector.pm
	M	WebCore/inspector/InspectorValues.h
	M	WebCore/inspector/Inspector.idl
	M	WebCore/inspector/InspectorValues.cpp
r63561 = 52486558854a03d87a6292b1d1bc050243e0e37c (refs/remotes/trunk)
Comment 5 WebKit Review Bot 2010-07-16 12:42:58 PDT
http://trac.webkit.org/changeset/63561 might have broken Chromium Win Release
The following changes are on the blame list:
http://trac.webkit.org/changeset/63561
http://trac.webkit.org/changeset/63562
http://trac.webkit.org/changeset/63563