some browsers (notably IE up to version 8) don't support
standard addEventLister so we need to adjust the way event
for spreadsheet is wired in job detail view page

Signed-off-by: Julius Gawlas <[email protected]>
---
 .../client/src/autotest/afe/JobDetailView.java     |    9 ++++++++-
 1 files changed, 8 insertions(+), 1 deletions(-)

diff --git a/frontend/client/src/autotest/afe/JobDetailView.java 
b/frontend/client/src/autotest/afe/JobDetailView.java
index 4f1e8f6..84827a2 100644
--- a/frontend/client/src/autotest/afe/JobDetailView.java
+++ b/frontend/client/src/autotest/afe/JobDetailView.java
@@ -90,7 +90,14 @@ public class JobDetailView extends DetailView implements 
TableWidgetFactory, Tab
             
[email protected]::resizeResultsFrame(Ljava/lang/String;)(event.data);
         }
 
-        $wnd.addEventListener("message", $wnd.onSpreadsheetLoad, false);
+        // ataching listener unfortunatelly varies bc not everybody follows 
standard
+        if ($wnd.addEventListener) { //W3C
+           $wnd.addEventListener("message", $wnd.onSpreadsheetLoad, false);
+        } else if ($wnd.attachEvent) { //IE
+           $wnd.attachEvent("onmessage", $wnd.onSpreadsheetLoad);
+        } else { //last resort
+           $wnd["onmessage"] = $wnd.onSpreadsheetLoad;
+        }
     }-*/;
 
     @SuppressWarnings("unused") // called from native
-- 
1.7.7.6

_______________________________________________
Autotest-kernel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/autotest-kernel

Reply via email to