|Summary:||Regression: View Source not showing closing script tags|
|Product:||WebKit||Reporter:||Keishi Hattori <keishi>|
|Component:||WebCore Misc.||Assignee:||Thomas Sepez <tsepez>|
|Severity:||Normal||CC:||abarth, tsepez, webkit.review.bot|
|Version:||528+ (Nightly build)|
Description Keishi Hattori 2011-09-27 06:26:58 PDT
http://trac.webkit.org/changeset/95451 caused some</script> to disappear from the view-source mode. Test: Visit "view-source:http://www.roseindia.net/tutorialfiles/26911.Script_Tag.htm" and see that </script> is missing. http://code.google.com/p/chromium/issues/detail?id=98166
Comment 1 Adam Barth 2011-09-27 09:16:05 PDT
Tom, your patch to SourceTracker probably broke this. I thought we had tests covering this case, but I guess we don't.
Comment 2 Thomas Sepez 2011-09-27 15:24:29 PDT
Comment 3 Thomas Sepez 2011-09-27 15:29:29 PDT
Created attachment 108914 [details] Patch + be more grammatical
Comment 4 Adam Barth 2011-09-27 15:32:28 PDT
Comment on attachment 108914 [details] Patch + be more grammatical Ok. I guess discretion is the better part of valor. :)
Comment 5 Thomas Sepez 2011-09-28 10:00:02 PDT
Comment on attachment 108914 [details] Patch + be more grammatical Tests OK. Setting c-c: ?
Comment 6 WebKit Review Bot 2011-09-28 10:18:15 PDT
Comment on attachment 108914 [details] Patch + be more grammatical Clearing flags on attachment: 108914 Committed r96231: <http://trac.webkit.org/changeset/96231>
Comment 7 WebKit Review Bot 2011-09-28 10:18:19 PDT
All reviewed patches have been landed. Closing bug.