Changeset 57230 in webkit


Ignore:
Timestamp:
Apr 7, 2010 2:10:58 PM (14 years ago)
Author:
eric@webkit.org
Message:

2010-04-07 Jay Civelli <jcivelli@chromium.org>

Reviewed by Jian Li.

[chromium] Add test for a popup related crasher.
https://bugs.webkit.org/show_bug.cgi?id=37141

  • platform/chromium/fast/forms: Added.
  • platform/chromium/fast/forms/search-popup-crasher-expected.txt: Added.
  • platform/chromium/fast/forms/search-popup-crasher.html: Added.

2010-04-07 Jay Civelli <jcivelli@chromium.org>

Reviewed by Jian Li.

[chromium] Fixing a NULL pointer being dereferenced in some cases.
https://bugs.webkit.org/show_bug.cgi?id=37141

Test: platform/chromium/fast/forms/search-popup-crasher.html

  • platform/chromium/PopupMenuChromium.cpp: (WebCore::PopupMenu::~PopupMenu):
Location:
trunk
Files:
3 added
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/LayoutTests/ChangeLog

    r57228 r57230  
     12010-04-07  Jay Civelli  <jcivelli@chromium.org>
     2
     3        Reviewed by Jian Li.
     4
     5        [chromium] Add test for a popup related crasher.
     6        https://bugs.webkit.org/show_bug.cgi?id=37141
     7
     8        * platform/chromium/fast/forms: Added.
     9        * platform/chromium/fast/forms/search-popup-crasher-expected.txt: Added.
     10        * platform/chromium/fast/forms/search-popup-crasher.html: Added.
     11
    1122010-04-07  Csaba Osztrogonác  <ossy@webkit.org>
    213
  • trunk/WebCore/ChangeLog

    r57229 r57230  
     12010-04-07  Jay Civelli  <jcivelli@chromium.org>
     2
     3        Reviewed by Jian Li.
     4
     5        [chromium] Fixing a NULL pointer being dereferenced in some cases.
     6        https://bugs.webkit.org/show_bug.cgi?id=37141
     7
     8        Test: platform/chromium/fast/forms/search-popup-crasher.html
     9
     10        * platform/chromium/PopupMenuChromium.cpp:
     11        (WebCore::PopupMenu::~PopupMenu):
     12
    1132010-04-07  Kinuko Yasuda  <kinuko@chromium.org>
    214
  • trunk/WebCore/platform/chromium/PopupMenuChromium.cpp

    r56451 r57230  
    12611261    // When the PopupMenu is destroyed, the client could already have been
    12621262    // deleted.
    1263     p.popup->listBox()->disconnectClient();
     1263    if (p.popup)
     1264        p.popup->listBox()->disconnectClient();
    12641265    hide();
    12651266}
Note: See TracChangeset for help on using the changeset viewer.