Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 223834 - [Browser] Modifying the DOM (execute javascript) does not work
Summary: [Browser] Modifying the DOM (execute javascript) does not work
Status: RESOLVED FIXED
Alias: None
Product: RAP
Classification: RT
Component: RWT (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 1.3 M3   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-03-25 10:36 EDT by Ivan Furnadjiev CLA
Modified: 2009-10-13 10:57 EDT (History)
0 users

See Also:


Attachments
Snippet161.java (2.33 KB, text/plain)
2008-03-25 10:38 EDT, Ivan Furnadjiev CLA
no flags Details
Fix (939 bytes, patch)
2009-10-13 10:06 EDT, Tim Buschtoens CLA
ruediger.herrmann: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ivan Furnadjiev CLA 2008-03-25 10:36:16 EDT
When the snippet is executed:
1. The background color is not initial set to "yellow". If you change the document.bgColor = "yellow" to  document.bgColor = "red" and press the execute button nothing happens - no background change.
2. The new text node is not added when you press the execute button.
Comment 1 Ivan Furnadjiev CLA 2008-03-25 10:38:32 EDT
Created attachment 93389 [details]
Snippet161.java
Comment 2 Ivan Furnadjiev CLA 2008-03-25 10:42:33 EDT
browser.execute(text.getText()) returns false.
Comment 3 Benjamin Muskalla CLA 2009-08-09 16:39:32 EDT
Ivan, could you check this again on Windows with IE? Works for me as expected with FF and Chrome on Linux. I suspect this got fixed by bug 234683.
Comment 4 Ivan Furnadjiev CLA 2009-08-10 03:19:57 EDT
Benny, just checked it - does not work in IE8 on Windows Vista. Works with FF, Chrome and Safari on Windows.
Comment 5 Tim Buschtoens CLA 2009-10-13 10:06:48 EDT
Created attachment 149441 [details]
Fix

In IE, "eval" is not a member of "window". However, execScript is.
Comment 6 Rüdiger Herrmann CLA 2009-10-13 10:56:58 EDT
Applied patch to CVS HEAD