Bug 42523 - Web Inspector: the bindings generation helper script can be simplified a bit. Chromium.
Summary: Web Inspector: the bindings generation helper script can be simplified a bit....
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
Depends on: 42658
  Show dependency treegraph
Reported: 2010-07-18 13:22 PDT by Ilya Tikhonovsky
Modified: 2010-07-21 00:37 PDT (History)
10 users (show)

See Also:

[patch] initial version. (7.40 KB, patch)
2010-07-18 13:25 PDT, Ilya Tikhonovsky
pfeldman: review-
Details | Formatted Diff | Diff
[patch] second iteration (8.77 KB, patch)
2010-07-18 14:11 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-18 13:22:20 PDT
rule_binding.py does move operation for header files generated by CodeGeneratorV8.
It is a bit strange because its much easier to generate headers in the right folder by CodeGenerator itself.
Comment 1 Ilya Tikhonovsky 2010-07-18 13:25:04 PDT
Created attachment 61909 [details]
[patch] initial version.
Comment 2 Pavel Feldman 2010-07-18 13:33:35 PDT
Comment on attachment 61909 [details]
[patch] initial version.

A bunch of nits to fix.

 +      command.extend(['--outputHDir', hdir])
Here and below outputHeadersDir

 +      # Both the .cpp and .h were generated in cppdir, but if hdir is different,
Can you avoid changing this behavior?

 +      $codeGenerator = $ifaceName->new($object, $useOutputDir, $useLayerOnTop, $preprocessor, $writeDependencies, $verbose, $useOutputHDir);
I'd keep useOutputDir and useOutputHeadersDir next to each other.

 +      open(my $HEADER, ">$outputHDir/$frontendClassName.h") || die "Couldn't open file $outputDir/$frontendClassName.h";
print HeadersDir, not Dir in the message.
Comment 3 Ilya Tikhonovsky 2010-07-18 14:11:58 PDT
Created attachment 61912 [details]
[patch] second iteration
Comment 4 Yury Semikhatsky 2010-07-18 22:57:01 PDT
Comment on attachment 61912 [details]
[patch] second iteration

 +          WebInspector: Web Inspector: the bindings generation helper script can be
WebInspector: Web Inspector: ->  Web Inspector:
Comment 5 WebKit Review Bot 2010-07-20 12:33:27 PDT
http://trac.webkit.org/changeset/63764 might have broken Chromium Linux Release
The following changes are on the blame list:
Comment 6 Ilya Tikhonovsky 2010-07-21 00:37:36 PDT
Committed r63803
	M	WebCore/ChangeLog
	M	WebCore/WebCore.gyp/WebCore.gyp
	M	WebCore/WebCore.gyp/scripts/rule_binding.py
	M	WebCore/bindings/scripts/CodeGeneratorCPP.pm
	M	WebCore/bindings/scripts/generate-bindings.pl
	M	WebCore/bindings/scripts/CodeGeneratorJS.pm
	M	WebCore/bindings/scripts/CodeGeneratorObjC.pm
	M	WebCore/bindings/scripts/CodeGeneratorV8.pm
	M	WebCore/bindings/scripts/CodeGenerator.pm
	M	WebCore/inspector/CodeGeneratorInspector.pm
r63803 = f77b355757ca7fcdb5f175d980dca7784f35debc (refs/remotes/trunk)