New issue
Advanced search Search tips

Issue 44868 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Oct 2011
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug-Security
M-5

Restricted
  • Only users with EditIssue permission may comment.



Sign in to add a comment

Geolocation events fire after document deletion

Project Member Reported by jsc...@chromium.org, May 23 2010

Issue description

I stumbled on this when trying to land a fix for  bug 39985 . It's filed 
upstream as https://bugs.webkit.org/show_bug.cgi?id=39388

Reporting here so I remember to merge it on Friday.

 

Comment 1 by jsc...@chromium.org, May 23 2010

Summary: Geolocation events fire after document deletion

Comment 2 by bugdro...@gmail.com, May 25 2010

The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=48114 

------------------------------------------------------------------------
r48114 | inferno@chromium.org | 2010-05-24 18:29:30 -0700 (Mon, 24 May 2010) | 17 lines
Changed paths:
   A http://src.chromium.org/viewvc/chrome/branches/WebKit/375/LayoutTests/fast/dom/Geolocation/notimer-after-unload-expected.txt
   A http://src.chromium.org/viewvc/chrome/branches/WebKit/375/LayoutTests/fast/dom/Geolocation/notimer-after-unload.html
   A http://src.chromium.org/viewvc/chrome/branches/WebKit/375/LayoutTests/fast/dom/Geolocation/resources/notimer-after-unload.js
   M http://src.chromium.org/viewvc/chrome/branches/WebKit/375/WebCore/ChangeLog?r1=48114&r2=48113
   M http://src.chromium.org/viewvc/chrome/branches/WebKit/375/WebCore/page/Geolocation.cpp?r1=48114&r2=48113

Merge 59859 - 20100520  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/notimerafterunload.html, plus it should be possible to reenable Gtk LayoutTests.

        * page/Geolocation.cpp:
        (WebCore::Geolocation::disconnectFrame):


TBR=steveblock@google.com
BUG= 44868 

Review URL: http://codereview.chromium.org/2128029
------------------------------------------------------------------------

Labels: -NeedsMerge -M375
Labels: -Restrict-View-SecurityTeam
Status: Fixed
Fixed in 5.0.375.70

Comment 5 by jsc...@chromium.org, Dec 16 2010

Labels: SecSeverity-High

Comment 6 by jsc...@chromium.org, Mar 21 2011

Labels: Type-Security
Labels: SecImpacts-Stable
Batch update.
Project Member

Comment 8 by bugdroid1@chromium.org, Oct 13 2012

Labels: Restrict-AddIssueComment-Commit
Owner: ----
This issue has been closed for some time. No one will pay attention to new comments.
If you are seeing this bug or have new data, please click New Issue to start a new bug.
Project Member

Comment 9 by bugdroid1@chromium.org, Mar 10 2013

Labels: -Area-WebKit -Mstone-5 -SecSeverity-High -Type-Security -SecImpacts-Stable Cr-Content M-5 Security-Impact-Stable Type-Bug-Security Security-Severity-High
Project Member

Comment 10 by bugdroid1@chromium.org, Mar 13 2013

Labels: -Restrict-AddIssueComment-Commit Restrict-AddIssueComment-EditIssue
Project Member

Comment 11 by bugdroid1@chromium.org, Mar 21 2013

Labels: -Security-Severity-High Security_Severity-High
Project Member

Comment 12 by bugdroid1@chromium.org, Mar 21 2013

Labels: -Security-Impact-Stable Security_Impact-Stable
Project Member

Comment 13 by bugdroid1@chromium.org, Apr 6 2013

Labels: -Cr-Content Cr-Blink
Project Member

Comment 14 by sheriffbot@chromium.org, Oct 1 2016

This bug has been closed for more than 14 weeks. Removing security view restrictions.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 15 by sheriffbot@chromium.org, Oct 1 2016

Labels: Restrict-View-SecurityNotify
Project Member

Comment 16 by sheriffbot@chromium.org, Oct 2 2016

Labels: -Restrict-View-SecurityNotify
This bug has been closed for more than 14 weeks. Removing security view restrictions.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: allpublic
Project Member

Comment 18 by sheriffbot@chromium.org, Jul 29

Labels: -Pri-2 Pri-1

Sign in to add a comment