Web Inspector: Make update-InspectorBackendCommands.rb only update legacy protocol versions

https://bugs.webkit.org/show_bug.cgi?id=130412

Patch by Joseph Pecoraro <[email protected]> on 2014-03-18
Reviewed by Timothy Hatcher.

Rename the script and simplify to only update Legacy protocol versions.

* Scripts/update-LegacyInspectorBackendCommands.rb: Renamed from Source/WebInspectorUI/Scripts/update-InspectorBackendCommands.rb.

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@165838 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Author: [email protected]

Source/WebInspectorUI/ChangeLog

@@ -1,3 +1,14 @@
+2014-03-18  Joseph Pecoraro  <[email protected]>
+
+        Web Inspector: Make update-InspectorBackendCommands.rb only update legacy protocol versions
+        https://bugs.webkit.org/show_bug.cgi?id=130412
+
+        Reviewed by Timothy Hatcher.
+
+        Rename the script and simplify to only update Legacy protocol versions.
+
+        * Scripts/update-LegacyInspectorBackendCommands.rb: Renamed from Source/WebInspectorUI/Scripts/update-InspectorBackendCommands.rb.
+
 2014-03-18  Joseph Pecoraro  <[email protected]>
 
         Web Inspector: sourceMapResourceLoaded expects statusCode parameter

Source/WebInspectorUI/Scripts/update-InspectorBackendCommands.rb renamed to Source/WebInspectorUI/Scripts/update-LegacyInspectorBackendCommands.rb

@@ -1,7 +1,6 @@
 #!/usr/bin/ruby
 
 require 'fileutils'
-require 'tempfile'
 require 'tmpdir'
 
 if ARGV.size != 0
@@ -11,26 +10,17 @@
 
 WEB_INSPECTOR_PATH = File.expand_path File.join(File.dirname(__FILE__), "..")
 JAVASCRIPTCORE_PATH = File.expand_path File.join(File.dirname(__FILE__), "..", "..", "JavaScriptCore")
-WEBCORE_PATH = File.expand_path File.join(File.dirname(__FILE__), "..", "..", "WebCore")
-
-JAVASCRIPT_INSPECTOR_PROTOCOLS_PATH = File.join JAVASCRIPTCORE_PATH, "inspector", "protocol"
-WEB_INSPECTOR_PROTOCOLS_PATH = File.join WEBCORE_PATH, "inspector", "protocol"
 
 $code_generator_path = File.join JAVASCRIPTCORE_PATH, "inspector", "scripts", "CodeGeneratorInspector.py"
-$inspector_json_generator_path = File.join JAVASCRIPTCORE_PATH, "inspector", "scripts", "generate-combined-inspector-json.py"
-$inspector_js_json_path = Tempfile.new("generated-InspectorJS.json").path
-$inspector_web_json_path = Tempfile.new("generated-InspectorWeb.json").path
 $versions_directory_path = File.join WEB_INSPECTOR_PATH, "Versions"
-$web_inspector_user_interface_path = File.join WEB_INSPECTOR_PATH, "UserInterface"
-$web_inspector_protocol_path = File.join WEB_INSPECTOR_PATH, "UserInterface", "Protocol"
+$web_inspector_protocol_legacy_path = File.join WEB_INSPECTOR_PATH, "UserInterface", "Protocol", "Legacy"
 
 class Task
-  def initialize(input_json_path, dependency_json_path, type, output_directory_path, verification)
+  def initialize(input_json_path, dependency_json_path, type, output_directory_path)
     @type = type
     @input_json_path = input_json_path
     @dependency_json_path = dependency_json_path
     @output_directory_path = output_directory_path
-    @verification = verification
   end
 
   def run
@@ -42,8 +32,7 @@ def run
 
     Dir.mktmpdir do |tmpdir|
       dependency = @dependency_json_path ? "'#{@dependency_json_path}'" : ""
-      cmd = "#{$code_generator_path} '#{@input_json_path}' #{dependency} --output_h_dir '#{tmpdir}' --output_cpp_dir '#{tmpdir}' --output_js_dir '#{tmpdir}' --write_always --output_type '#{@type}'"
-      cmd += " --no_verification" if !@verification
+      cmd = "#{$code_generator_path} '#{@input_json_path}' #{dependency} --no_verification --output_h_dir '#{tmpdir}' --output_cpp_dir '#{tmpdir}' --output_js_dir '#{tmpdir}' --write_always --output_type '#{@type}'"
       %x{ #{cmd} }
       if $?.exitstatus != 0
         puts "ERROR: Error Code (#{$?.exitstatus}) Evaluating: #{cmd}"
@@ -62,36 +51,15 @@ def run
   end
 end
 
-def generate_combined_inspector_json
-  # InspectorJS.json
-  output = %x{ #{$inspector_json_generator_path} "#{JAVASCRIPT_INSPECTOR_PROTOCOLS_PATH}" > "#{$inspector_js_json_path}" }
-  if $?.exitstatus != 0
-    puts "ERROR: Could not generate combined InspectorJS.json file"
-    exit 1
-  end
-
-  # InspectorWeb.json
-  output = %x{ #{$inspector_json_generator_path} "#{WEB_INSPECTOR_PROTOCOLS_PATH}" > "#{$inspector_web_json_path}" }
-  if $?.exitstatus != 0
-    puts "ERROR: Could not generate combined InspectorWeb.json file"
-    exit 1
-  end
-end
-
 def all_tasks
   tasks = []
 
-  # ToT Inspector json files.
-  generate_combined_inspector_json
-  tasks << Task.new($inspector_js_json_path, nil, "JavaScript", $web_inspector_protocol_path, true)
-  tasks << Task.new($inspector_web_json_path, $inspector_js_json_path, "Web", $web_inspector_protocol_path, true)
-
   had_error = false
   Dir.glob(File.join($versions_directory_path, "*.json")).each do |version_path|
     match = File.basename(version_path).match(/^Inspector(.*?)\-([^-]+?)\.json$/)
     if match
-      output_path = File.join $web_inspector_protocol_path, "Legacy", match[2]
-      tasks << Task.new(version_path, nil, "Web", output_path, false)
+      output_path = File.join $web_inspector_protocol_legacy_path, match[2]
+      tasks << Task.new(version_path, nil, "Web", output_path)
     else
       puts "ERROR: Version file (#{version_path}) did not match the template Inspector<ANYTHING>-<VERSION>.js"
       had_error = true