Closed Bug 1252260 Opened 4 years ago Closed 4 years ago

get rid of HTML table CacheChildren

Categories

(Core :: Disability Access APIs, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla47
Tracking Status
firefox47 --- fixed

People

(Reporter: surkov, Assigned: surkov)

Details

Attachments

(1 file)

Attached patch patchSplinter Review
No description provided.
Attachment #8724951 - Flags: review?(mzehe)
Comment on attachment 8724951 [details] [diff] [review]
patch

r=me with the following question answered/fixed:

>+<body>
>+  <p id="display"></p>
>+  <div id="content" style="display: none"></div>
>+  <pre id="test">
>+  </pre>
No link to a bug here?
Attachment #8724951 - Flags: review?(mzehe) → review+
bug links in our case are not very useful, because we don't follow "one bug one test" practice in general, it's much easier to check hg to figure out what this test is about. Then this test passes if the patch of this bug is not applied (it would if we inserted accessilbes rather than re-cached them all together)
All right, thanks! Go ahead and land at your discretion.
https://hg.mozilla.org/mozilla-central/rev/4ddf9f83cb51
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
Assignee: nobody → surkov.alexander
You need to log in before you can comment on or make changes to this bug.