Changeset 69170 in webkit


Ignore:
Timestamp:
Oct 5, 2010 8:04:18 PM (13 years ago)
Author:
abarth@webkit.org
Message:

2010-10-05 Adam Barth <abarth@webkit.org>

Reviewed by Darin Adler.

Issue in treebuilder parsing related to table tags
https://bugs.webkit.org/show_bug.cgi?id=47190

Add test coverage.

  • html5lib/resources/webkit02.dat:

2010-10-05 Adam Barth <abarth@webkit.org>

Reviewed by Darin Adler.

Issue in treebuilder parsing related to table tags
https://bugs.webkit.org/show_bug.cgi?id=47190

Update ASSERT to match the spec and our behavior.

  • html/parser/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processEndTagForInCell):
Location:
trunk
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • trunk/LayoutTests/ChangeLog

    r69168 r69170  
     12010-10-05  Adam Barth  <abarth@webkit.org>
     2
     3        Reviewed by Darin Adler.
     4
     5        Issue in treebuilder parsing related to table tags
     6        https://bugs.webkit.org/show_bug.cgi?id=47190
     7
     8        Add test coverage.
     9
     10        * html5lib/resources/webkit02.dat:
     11
    1122010-10-05  Kenneth Russell  <kbr@google.com>
    213
  • trunk/LayoutTests/html5lib/resources/webkit02.dat

    r67962 r69170  
    4141|   <head>
    4242|   <body>
     43
     44#data
     45<table><td></tbody>A
     46#errors
     47#document
     48| <html>
     49|   <head>
     50|   <body>
     51|     "A"
     52|     <table>
     53|       <tbody>
     54|         <tr>
     55|           <td>
     56
     57#data
     58<table><td></thead>A
     59#errors
     60#document
     61| <html>
     62|   <head>
     63|   <body>
     64|     <table>
     65|       <tbody>
     66|         <tr>
     67|           <td>
     68|             "A"
     69
     70#data
     71<table><td></tfoot>A
     72#errors
     73#document
     74| <html>
     75|   <head>
     76|   <body>
     77|     <table>
     78|       <tbody>
     79|         <tr>
     80|           <td>
     81|             "A"
     82
     83#data
     84<table><thead><td></tbody>A
     85#errors
     86#document
     87| <html>
     88|   <head>
     89|   <body>
     90|     <table>
     91|       <thead>
     92|         <tr>
     93|           <td>
     94|             "A"
  • trunk/WebCore/ChangeLog

    r69166 r69170  
     12010-10-05  Adam Barth  <abarth@webkit.org>
     2
     3        Reviewed by Darin Adler.
     4
     5        Issue in treebuilder parsing related to table tags
     6        https://bugs.webkit.org/show_bug.cgi?id=47190
     7
     8        Update ASSERT to match the spec and our behavior.
     9
     10        * html/parser/HTMLTreeBuilder.cpp:
     11        (WebCore::HTMLTreeBuilder::processEndTagForInCell):
     12
    1132010-10-05  Victoria Kirst  <vrk@google.com>
    214
  • trunk/WebCore/html/parser/HTMLTreeBuilder.cpp

    r68116 r69170  
    18501850        || isTableBodyContextTag(token.name())) {
    18511851        if (!m_tree.openElements()->inTableScope(token.name())) {
    1852             ASSERT(isParsingFragment());
     1852            ASSERT(isTableBodyContextTag(token.name()) || isParsingFragment());
    18531853            parseError(token);
    18541854            return;
Note: See TracChangeset for help on using the changeset viewer.