Opened 16 years ago
Closed 16 years ago
#3500 closed Bug (fixed)
Failed testcase dom.element test_getAttribute_ignoreExpandoAttributes
| Reported by: | Alfonso Martínez de Lizarrondo | Owned by: | Frederico Caldeira Knabben |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.0 |
| Component: | General | Version: | SVN (CKEditor) - OLD |
| Keywords: | Review+ | Cc: |
Description
Except IE8 all the browsers fail this test.
In IE7 it's due to the testcase itself, the rest of the browsers are failing because they return the _cke_expando as an attribute (just what the test is checking)
The patch fixes all the issues for me.
Attachments (2)
Change History (6)
Changed 16 years ago by
| Attachment: | 3500.patch added |
|---|
comment:1 Changed 16 years ago by
| Keywords: | Review- added; Review? removed |
|---|---|
| Owner: | changed from Alfonso Martínez de Lizarrondo to Frederico Caldeira Knabben |
| Status: | new → assigned |
The TC could be a bit simpler. Also, in the case of tabIndex, it's not a matter of making the test pass. Tests should identify possible weak points, so the right fix would be making it work properly with tabindex.
I'll come with a new patch for it.
Changed 16 years ago by
| Attachment: | 3500_2.patch added |
|---|
comment:2 Changed 16 years ago by
| Keywords: | Review? added; Review- removed |
|---|
comment:3 Changed 16 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
Note: See
TracTickets for help on using
tickets.
Proposed patch