rb: Remove \t from source. Are we barbarians?

Author: p0deje

rb/lib/selenium/client/base.rb

@@ -20,7 +20,7 @@
 module Selenium
   module Client
 
-		# Driver constructor and session management commands
+    # Driver constructor and session management commands
     module Base
       include Selenium::Client::Protocol
       include Selenium::Client::GeneratedDriver
@@ -133,7 +133,7 @@ def start(opts = {})
       end
 
       def stop
-	      close_current_browser_session
+        close_current_browser_session
       end
 
       def chrome_backend?
@@ -143,7 +143,7 @@ def chrome_backend?
       def javascript_extension=(new_javascript_extension)
         @extension_js = new_javascript_extension
       end
-	    alias :set_extension_js :javascript_extension=
+      alias :set_extension_js :javascript_extension=
 
     end
 

rb/lib/selenium/client/extensions.rb

@@ -23,22 +23,19 @@ module Client
     # Convenience methods not explicitly part of the protocol
     module Extensions
 
-	    # These for all Ajax request to finish (Only works if you are using prototype, the wait happens in the browser)
-	    #
+      # These for all Ajax request to finish (Only works if you are using prototype, the wait happens in the browser)
       def wait_for_ajax(options={})
-		    builder = JavascriptExpressionBuilder.new active_javascript_framework(options)
-	      wait_for_condition builder.no_pending_ajax_requests.script,
-	                         options[:timeout_in_seconds]
+        builder = JavascriptExpressionBuilder.new active_javascript_framework(options)
+        wait_for_condition builder.no_pending_ajax_requests.script, options[:timeout_in_seconds]
       end
 
-	    # Wait for all Prototype effects to be processed (the wait happens in the browser).
-	    #
-	    # Credits to http://github.com/brynary/webrat/tree/master
-			def wait_for_effects(options={})
-		    builder = JavascriptExpressionBuilder.new active_javascript_framework(options)
-			  wait_for_condition builder.no_pending_effects.script,
-			                     options[:timeout_in_seconds]
-			end
+      # Wait for all Prototype effects to be processed (the wait happens in the browser).
+      #
+      # Credits to http://github.com/brynary/webrat/tree/master
+      def wait_for_effects(options={})
+        builder = JavascriptExpressionBuilder.new active_javascript_framework(options)
+        wait_for_condition builder.no_pending_effects.script, options[:timeout_in_seconds]
+      end
 
       # Wait for an element to be present (the wait happens in the browser).
       def wait_for_element(locator, options={})
@@ -54,8 +51,8 @@ def wait_for_no_element(locator, options={})
         wait_for_condition builder.script, options[:timeout_in_seconds]
       end
 
-			# Wait for some text to be present (the wait is happening browser side).
-			#
+      # Wait for some text to be present (the wait is happening browser side).
+      #
       # wait_for_text will search for the given argument within the innerHTML
       # of the current DOM. Note that this method treats a single string
       # as a special case.
@@ -86,7 +83,7 @@ def wait_for_no_element(locator, options={})
       #
       def wait_for_text(pattern, options={})
         builder = JavascriptExpressionBuilder.new
-		    builder.find_text(pattern, options).append("text_match == true;")
+        builder.find_text(pattern, options).append("text_match == true;")
         wait_for_condition builder.script, options[:timeout_in_seconds]
       end
 
@@ -95,7 +92,7 @@ def wait_for_text(pattern, options={})
       # See wait_for_text for usage details.
       def wait_for_no_text(pattern, options={})
         builder = JavascriptExpressionBuilder.new
-		    builder.find_text(pattern, options).append("text_match == false;")
+        builder.find_text(pattern, options).append("text_match == false;")
         wait_for_condition builder.script, options[:timeout_in_seconds]
       end
 
@@ -132,4 +129,4 @@ def active_javascript_framework(options)
     end
 
   end
-end
+end

rb/lib/selenium/client/idiomatic.rb

@@ -20,9 +20,9 @@
 module Selenium
   module Client
 
-		# Provide a more idiomatic API than the generated Ruby driver.
-		#
-		# Work in progress...
+    # Provide a more idiomatic API than the generated Ruby driver.
+    #
+    # Work in progress...
     module Idiomatic
 
       # Return the text content of an HTML element (rendered text shown to
@@ -102,31 +102,31 @@ def wait_for_popup(window_id, timeout_in_seconds=nil)
       def wait_for(options)
         if options[:wait_for] == :page
           wait_for_page options[:timeout_in_seconds]
-	      elsif options[:wait_for] == :ajax
-	          wait_for_ajax options
-	      elsif options[:wait_for] == :element
-	          wait_for_element options[:element], options
-	      elsif options[:wait_for] == :no_element
-	          wait_for_no_element options[:element], options
-	      elsif options[:wait_for] == :text
-	          wait_for_text options[:text], options
-	      elsif options[:wait_for] == :no_text
+        elsif options[:wait_for] == :ajax
+          wait_for_ajax options
+        elsif options[:wait_for] == :element
+          wait_for_element options[:element], options
+        elsif options[:wait_for] == :no_element
+          wait_for_no_element options[:element], options
+        elsif options[:wait_for] == :text
+          wait_for_text options[:text], options
+        elsif options[:wait_for] == :no_text
           wait_for_no_text options[:text], options
-	      elsif options[:wait_for] == :effects
-	          wait_for_effects options
+        elsif options[:wait_for] == :effects
+          wait_for_effects options
         elsif options[:wait_for] == :popup
-            wait_for_popup options[:window], options[:timeout_in_seconds]
-            select_window options[:window] if options[:select]
+          wait_for_popup options[:window], options[:timeout_in_seconds]
+          select_window options[:window] if options[:select]
         elsif options[:wait_for] == :value
-            wait_for_field_value options[:element], options[:value], options
+          wait_for_field_value options[:element], options[:value], options
         elsif options[:wait_for] == :no_value
-            wait_for_no_field_value options[:element], options[:value], options
+          wait_for_no_field_value options[:element], options[:value], options
         elsif options[:wait_for] == :visible
-            wait_for_visible options[:element], options
+          wait_for_visible options[:element], options
         elsif options[:wait_for] == :not_visible
-            wait_for_not_visible options[:element], options
-	      elsif options[:wait_for] == :condition
-	          wait_for_condition options[:javascript], options[:timeout_in_seconds]
+          wait_for_not_visible options[:element], options
+        elsif options[:wait_for] == :condition
+          wait_for_condition options[:javascript], options[:timeout_in_seconds]
         end
       end
 
@@ -206,7 +206,7 @@ def field(locator)
 
       # Alias for +field+
       def value(locator)
-	      field locator
+        field locator
       end
 
       # Returns whether a toggle-button (checkbox/radio) is checked.
@@ -397,7 +397,7 @@ def cookie?(name)
       # the optionsString's format is "path=/path/, max_age=60, domain=.foo.com". The order of options are irrelevant, the unit      of the value of 'max_age' is second.  Note that specifying a domain that isn't a subset of the current domain will      usually fail.
       def create_cookie(name_value_pair, options="")
         if options.kind_of? Hash
-		      options = options.keys.collect {|key| "#{key}=#{options[key]}" }.sort.join(", ")
+          options = options.keys.collect {|key| "#{key}=#{options[key]}" }.sort.join(", ")
         end
         remote_control_command "createCookie", [name_value_pair,options,]
       end
@@ -417,8 +417,8 @@ def create_cookie(name_value_pair, options="")
       # 'optionsString' is options for the cookie. Currently supported options include 'path', 'domain'      and 'recurse.' The optionsString's format is "path=/path/, domain=.foo.com, recurse=true".      The order of options are irrelevant. Note that specifying a domain that isn't a subset of      the current domain will usually fail.
       def delete_cookie(name, options="")
         if options.kind_of? Hash
-		      ordered_keys = options.keys.sort {|a,b| a.to_s <=> b.to_s }
-		      options = ordered_keys.collect {|key| "#{key}=#{options[key]}" }.join(", ")
+          ordered_keys = options.keys.sort {|a,b| a.to_s <=> b.to_s }
+          options = ordered_keys.collect {|key| "#{key}=#{options[key]}" }.join(", ")
         end
         remote_control_command "deleteCookie", [name,options,]
       end

rb/lib/selenium/client/javascript_expression_builder.rb

@@ -28,18 +28,18 @@ def initialize(javascript_framework_name=nil)
         @script = ""
       end
 
-	    def append(text)
-	      @script << text
-	      self
-	    end
+      def append(text)
+        @script << text
+        self
+      end
 
       def no_pending_ajax_requests
         append window_script("#{@framework.ajax_request_tracker} == 0")
       end
 
-			def no_pending_effects
-			  append window_script("Effect.Queue.size() == 0")
-			end
+      def no_pending_effects
+        append window_script("Effect.Queue.size() == 0")
+      end
 
       def visible(locator)
         append "selenium.isVisible('#{quote_escaped(locator)}')"
@@ -132,4 +132,4 @@ def quote_escaped(a_string)
       end
     end
   end
-end
+end

rb/spec/integration/selenium/client/api/backward_compatible_api_spec.rb

@@ -21,21 +21,21 @@
 
 describe "Backward Compatible API" do
   it "provides legacy driver methods" do
-		page.open "http://localhost:4444/selenium-server/org/openqa/selenium/tests/html/test_click_page1.html"
+    page.open "http://localhost:4444/selenium-server/org/openqa/selenium/tests/html/test_click_page1.html"
 
     expect(page.get_title).to eql("Click Page 1")
-		expect(page.get_text("link").index("Click here for next page")).not_to be_nil
+    expect(page.get_text("link").index("Click here for next page")).not_to be_nil
 
-		links = page.get_all_links
-		expect(links.length).to be > 3
-		expect(links[3]).to eql("linkToAnchorOnThisPage")
+    links = page.get_all_links
+    expect(links.length).to be > 3
+    expect(links[3]).to eql("linkToAnchorOnThisPage")
 
-		page.click "link"
-		page.wait_for_page_to_load 5000
-		expect(page.get_location).to match(%r"/selenium-server/org/openqa/selenium/tests/html/test_click_page2.html")
+    page.click "link"
+    page.wait_for_page_to_load 5000
+    expect(page.get_location).to match(%r"/selenium-server/org/openqa/selenium/tests/html/test_click_page2.html")
 
-		page.click "previousPage"
-		page.wait_for_page_to_load 5000
-		expect(page.get_location).to match(%r"/selenium-server/org/openqa/selenium/tests/html/test_click_page1.html")
+    page.click "previousPage"
+    page.wait_for_page_to_load 5000
+    expect(page.get_location).to match(%r"/selenium-server/org/openqa/selenium/tests/html/test_click_page1.html")
   end
 end