Removing more dead methods

lib/webrat/core/elements/label.rb

@@ -8,14 +8,6 @@ module Webrat
     def self.xpath_search
       ".//label"
     end
-    
-    def text
-      str = Webrat::XML.all_inner_text(@element)
-      str.gsub!("\n","")
-      str.strip!
-      str.squeeze!(" ")
-      str
-    end
 
     def for_id
       Webrat::XML.attribute(@element, "for")
@@ -25,6 +17,8 @@ module Webrat
       Field.load(@session, field_element)
     end
     
+  protected
+  
     def field_element
       if for_id.blank?
         Webrat::XML.xpath_at(@element, *Field.xpath_search)

lib/webrat/core/locators/field_labeled_locator.rb

@@ -7,15 +7,13 @@ module Webrat
     class FieldLabeledLocator < Locator
   
       def locate
-        matching_fields.min { |a, b| a.label_text.length <=> b.label_text.length }
+        matching_labels.any? && matching_labels.first.field
-      end
-      
-      def matching_fields
-        matching_labels.map(&:field).compact.uniq
       end
       
       def matching_labels
-        matching_label_elements.map do |label_element|
+        matching_label_elements.sort_by do |label_element|
+          text(label_element).length
+        end.map do |label_element|
           Label.load(@scope.session, label_element)
         end
       end