Bug 704246 - JavaScript Slow in IE8
Summary: JavaScript Slow in IE8
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise MRG
Classification: Red Hat
Component: cumin
Version: Development
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: 2.0
: ---
Assignee: Chad Roberts
QA Contact: Jan Sarenik
URL:
Whiteboard:
Depends On: 704253
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-05-12 14:38 UTC by Jan Sarenik
Modified: 2011-06-27 14:29 UTC (History)
4 users (show)

Fixed In Version: cumin-0.1.4765-1
Doc Type: Bug Fix
Doc Text:
Cause: The version of the mootools.js library that we are using is rather old (1.2.4) and has a bug that affects IE8. Consequence: IE8 complains about "slow running script" and performs badly when submitting forms (or potentially exiting any page where we use the <object> tag) Fix: We have patched the mootools.js library to address this specific problem without upgrading to the latest version to avoid introducing a great deal of work at this time. Result: IE8 no longer complains about long-running scripts.
Clone Of:
Environment:
Last Closed: 2011-06-27 14:29:45 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Proposed patch to get IE8 working properly (1.93 KB, patch)
2011-05-13 19:48 UTC, Chad Roberts
no flags Details | Diff

Description Jan Sarenik 2011-05-12 14:38:55 UTC
When I want to add or then remove broker link, I get following
warning (yes/no) after pressing submit in IE8 (Server2003):

  Stop running this script?

  A script on this page is causing Internet Explorer to run slowly.
  If it continues to run, your computer might become
  unresponsive.

           YES     NO


cumin-0.1.4746-1.el5

How reproducible: 100%

Steps to Reproduce:
1. In IE8: Messaging -> broker -> Broker Links -> Broker Link Add
2. add some valid broker name, then press Submit

Actual results: The above-reprinted warning shows and the only help
  is to press Yes. Then I am immediately transferred where I should
  normally (seeing "Add broker link: OK" in yellow box, the Operational
  link in the list).

Expected results: The underlying scripts should not make IE8 think
  there is something wrong.

Comment 1 Jan Sarenik 2011-05-12 15:15:14 UTC
The same happens when I submit a new Submission from
Grid -> Submissions -> Submit job

The full error which follows right after
clicking on the "Yes" button (as to not
continue running the slow script):
----------------------------------------
User Agent: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 5.2; Trident/4.0; .NET CLR 1.1.4322; .NET CLR 2.0.50727; .NET CLR 3.0.4506.2152; .NET CLR 3.5.30729)
Timestamp: Thu, 12 May 2011 18:13:41 UTC


Message: Syntax error
Line: 138
Char: 1
Code: 0
URI: http://cumin-host/index.html?frame=main.grid;main.m=grid;main.grid.id=1;main.grid.view.body.m=pool_submissions
----------------------------------------

Comment 3 Jan Sarenik 2011-05-12 15:53:54 UTC
The same error: Messaging -> amqp broker -> Add queue

Comment 5 Chad Roberts 2011-05-13 19:55:16 UTC
Fix available in revision 4763 (or via the attached patch to mootools.js).  

See the note for the attachment for details of the patch.

Comment 6 Chad Roberts 2011-05-13 19:55:16 UTC
    Technical note added. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    New Contents:
Cause:  The version of the mootools.js library that we are using is rather old (1.2.4) and has a bug that affects IE8.

Consequence:  IE8 complains about "slow running script" and performs badly when submitting forms (or potentially exiting any page where we use the <object> tag)

Fix:  We have patched the mootools.js library to address this specific problem without upgrading to the latest version to avoid introducing a great deal of work at this time.

Result:  IE8 no longer complains about long-running scripts.

Comment 7 Jan Sarenik 2011-05-18 12:29:54 UTC
Verified in cumin-0.1.4767-1.el5


Note You need to log in before you can comment on or make changes to this bug.