[TIMOB-6436] Android: change window.js addEventListener and addWrappedListener to use this.view rather than window
GitHub Issue | n/a |
---|---|
Type | Bug |
Priority | High |
Status | Closed |
Resolution | Fixed |
Resolution Date | 2012-01-04T15:36:47.000+0000 |
Affected Version/s | Release 1.8.0.1 |
Fix Version/s | Sprint 2011-48, Release 1.8.0.1 |
Components | Android |
Labels | branch-v8, module_window, qe-testadded |
Reporter | Opie Cyrus |
Assignee | Ping Wang |
Created | 2011-12-02T13:31:42.000+0000 |
Updated | 2012-04-17T12:12:40.000+0000 |
Description
In window.js, the addEventListener and addWrappedListener method should use this.view rather than this.window as this.window in the case of LW windows will not represent the actual view we are interested in getting events from. The implementation works but only due to bubbling.
Testing for this ticket should include re-verifying both timob-6096 and timob-6393 with the new change.
Use the code below to test removeEventListener. Longpress anywhere in the window (except the button and the labels), the bottom text should show "longpress triggered". Press the button and longpress in the window, the label with "longpress triggered" doesn't show up.
Bug fixed. Verified on: SDK: 1.8.0.1.v20111205011857 Runtimes: Android V8/Rhino Studio: 1.0.7.201112041107 OS: OS X Lion Devices Tested: Nexus One 2.2.2, Droid3 2.3.4, Slate 3.1, Emulator 4.0
Added label qe-testadded.