opal · Jan 26, 2014 · Jan 26, 2014
Showing with 3 additions and 28 deletions.

+0 −8 .travis.yml

+3 −13 opal/browser/dom/event.rb

+0 −7 opal/browser/support.rb
diff --git a/.travis.yml b/.travis.yml
@@ -38,14 +38,6 @@ env:
 matrix:
   fast_finish: true
 
-  allow_failures:
-    - env: SELENIUM_BROWSER=chrome SELENIUM_VERSION=31
-    - env: SELENIUM_BROWSER=chrome SELENIUM_VERSION=30
-
-    - env: SELENIUM_BROWSER=ie SELENIUM_VERSION=6
-    - env: SELENIUM_BROWSER=ie SELENIUM_VERSION=7
-    - env: SELENIUM_BROWSER=ie SELENIUM_VERSION=8
-
 before_install:
   - gem update bundler
   - bundle --version

diff --git a/opal/browser/dom/event.rb b/opal/browser/dom/event.rb
@@ -208,24 +208,14 @@ def on
     return @on if @on
 
     if @callback
-      Target.convert(@callback.target)
+      @callback.target
     else
       Target.convert(`#@native.currentTarget`)
     end
   end
 
-  if Browser.supports? 'Event.target'
-    def target
-      Target.convert(`#@native.target`)
-    end
-  elsif Browser.supports? 'Event.srcElement'
-    def target
-      Target.convert(`#@native.srcElement`)
-    end
-  else
-    def target
-      raise NotImplementedError, 'event target unsupported'
-    end
+  def target
+    Target.convert(`#@native.srcElement || #@native.target`)
   end
 
   def arguments

diff --git a/opal/browser/support.rb b/opal/browser/support.rb
@@ -156,13 +156,6 @@ def self.supports?(feature)
       when 'Event.fire'
         defined?(`document.fireEvent`)
 
-      # TODO: implement this
-      when 'Event.target'
-        supports? 'Event.constructor'
-
-      when 'Event.srcElement'
-        supports? 'Event.createObject'
-
       when /^Event\.([A-Z].*?)$/
         `(#{$1} + "Event") in window`