Changeset 85636 in webkit


Ignore:
Timestamp:
May 3, 2011 12:34:36 PM (13 years ago)
Author:
ojan@chromium.org
Message:

2011-05-03 Ojan Vafai <ojan@chromium.org>

Reviewed by Tony Chang.

Don't include the test itself in expanded test results
https://bugs.webkit.org/show_bug.cgi?id=60051

When loading from local disk, we can't force it to render as
plain text, which causes the page to actually run. Also,
the extra clutter of another iframe is probably not worth it.

If we decide we want this later, we can add it back in as a
preference that's off by default.

  • fast/harness/results.html:
Location:
trunk/LayoutTests
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/LayoutTests/ChangeLog

    r85635 r85636  
     12011-05-03  Ojan Vafai  <ojan@chromium.org>
     2
     3        Reviewed by Tony Chang.
     4
     5        Don't include the test itself in expanded test results
     6        https://bugs.webkit.org/show_bug.cgi?id=60051
     7
     8        When loading from local disk, we can't force it to render as
     9        plain text, which causes the page to actually run. Also,
     10        the extra clutter of another iframe is probably not worth it.
     11
     12        If we decide we want this later, we can add it back in as a
     13        preference that's off by default.
     14
     15        * fast/harness/results.html:
     16
    1172011-05-03  Alpha Lam  <hclam@chromium.org>
    218
  • trunk/LayoutTests/fast/harness/results.html

    r85633 r85636  
    261261    var newCell = document.createElement('td');
    262262    newCell.colSpan = row.querySelectorAll('td').length;
    263 
    264     newCell.insertAdjacentHTML('beforeEnd', resultIframe(row.querySelector('.test-link').href));   
    265263
    266264    var resultLinks = row.querySelectorAll('.result-link');
Note: See TracChangeset for help on using the changeset viewer.