1. Go to search.twitter.com 2. Enter "toto" into the search box, hit enter An illegal read happens in PopupMenu::~PopupMenu().
Created attachment 52614 [details] Fix potential NULL pointer read.
Is it possible to add a layout test for this?
Created attachment 52692 [details] Adding a layout test
Comment on attachment 52692 [details] Adding a layout test The bug title should be prefixed with "[Chromium]" to indicate that this is for chromium only. > --- LayoutTests/ChangeLog (revision 57181) > +++ LayoutTests/ChangeLog (working copy) > @@ -1,3 +1,14 @@ > +2010-04-06 Jay Civelli <jcivelli@chromium.org> Garbage characters at the beginning. > --- LayoutTests/platform/chromium/fast/forms/search-popup-crasher.html (revision 0) > +++ LayoutTests/platform/chromium/fast/forms/search-popup-crasher.html (revision 0) > @@ -0,0 +1,27 @@ > +<HTML> > +<HEAD> > +<script> > +window.onload = function() > +{ > + if (window.layoutTestController) > + layoutTestController.dumpAsText(); > + > + Please remove the extra empty line. > + if (window.location.href.indexOf('formSubmitted') != -1) > + return; > + document.getElementById("searchBox").value= 'quentin'; Might be better to stick with double-quote than single-quote, for consistency. > + document.getElementById("searchForm").submit(); > +}; > +</script> > +</HEAD> > +<BODY> > + > +<p>This page tests that a page with a search popup does not crash, as reported in <a href='https://bugs.webkit.org/show_bug.cgi?id=37141'>this bug</a>. Below is a search input that the test automatically fills than submit. Pass if this does not crash.</p><br><br> Typo: than => then
Created attachment 52766 [details] Applying suggested changes.
Comment on attachment 52766 [details] Applying suggested changes. Please update both ChangeLog to reflect the bug title change. Do you want the commit-queue to land it for you? If so, please specify "?" for the "commit-queue" flag when you submit your new patch.
Created attachment 52777 [details] Added Chromium mention in ChangeLog files.
Comment on attachment 52777 [details] Added Chromium mention in ChangeLog files. r=me
Comment on attachment 52777 [details] Added Chromium mention in ChangeLog files. Clearing flags on attachment: 52777 Committed r57230: <http://trac.webkit.org/changeset/57230>
All reviewed patches have been landed. Closing bug.