Changeset 59859 in webkit


Ignore:
Timestamp:
May 20, 2010, 1:10:12 PM (14 years ago)
Author:
steveblock@google.com
Message:

2010-05-20 Marcus Bulach <bulach@chromium.org>

Reviewed by Steve Block.

Ensure timers are stopped on Geolocation::disconnectFrame()
https://bugs.webkit.org/show_bug.cgi?id=39388

fast/dom/Geolocation/notimer-after-unload.html, plus it should be possible to re-enable Gtk LayoutTests.

  • page/Geolocation.cpp: (WebCore::Geolocation::disconnectFrame):
Location:
trunk
Files:
3 added
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/WebCore/ChangeLog

    r59856 r59859  
     12010-05-20  Marcus Bulach  <bulach@chromium.org>
     2
     3        Reviewed by Steve Block.
     4
     5        Ensure timers are stopped on Geolocation::disconnectFrame()
     6        https://bugs.webkit.org/show_bug.cgi?id=39388
     7
     8        fast/dom/Geolocation/notimer-after-unload.html, plus it should be possible to re-enable Gtk LayoutTests.
     9
     10        * page/Geolocation.cpp:
     11        (WebCore::Geolocation::disconnectFrame):
     12
    1132010-05-20  James Robinson  <jamesr@chromium.org>
    214
  • trunk/WebCore/page/Geolocation.cpp

    r59216 r59859  
    223223    if (m_frame && m_frame->page() && m_allowGeolocation == InProgress)
    224224        m_frame->page()->chrome()->cancelGeolocationPermissionRequestForFrame(m_frame, this);
     225    stopTimers();
    225226    stopUpdating();
    226227    if (m_frame && m_frame->document())
Note: See TracChangeset for help on using the changeset viewer.