Remove an obsolete comment regarding magic frames from Geolocation
authorbenjamin@webkit.org <benjamin@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 23 Mar 2012 00:13:01 +0000 (00:13 +0000)
committerbenjamin@webkit.org <benjamin@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 23 Mar 2012 00:13:01 +0000 (00:13 +0000)
https://bugs.webkit.org/show_bug.cgi?id=81871

Patch by Benjamin Poulain <bpoulain@apple.com> on 2012-03-22
Reviewed by Alexey Proskuryakov.

Moving a frame into another document is no longer possible after r111361. The comment
is now irrelevant.

* Modules/geolocation/Geolocation.cpp:
(WebCore::Geolocation::stop):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@111789 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/Modules/geolocation/Geolocation.cpp

index cbbe9cc..9f57780 100644 (file)
@@ -1,3 +1,16 @@
+2012-03-22  Benjamin Poulain  <bpoulain@apple.com>
+
+        Remove an obsolete comment regarding magic frames from Geolocation
+        https://bugs.webkit.org/show_bug.cgi?id=81871
+
+        Reviewed by Alexey Proskuryakov.
+
+        Moving a frame into another document is no longer possible after r111361. The comment
+        is now irrelevant.
+
+        * Modules/geolocation/Geolocation.cpp:
+        (WebCore::Geolocation::stop):
+
 2012-03-22  Csaba Osztrogon√°c  <ossy@webkit.org>
 
         Actually move WTF files to their new home
index 9f64754..3a2e5d8 100644 (file)
@@ -256,12 +256,6 @@ Page* Geolocation::page() const
 
 void Geolocation::stop()
 {
-    // FIXME: We should ideally allow existing Geolocation activities to continue
-    // when the Geolocation's iframe is reparented. (Assuming we continue to
-    // support reparenting iframes.)
-    // See https://bugs.webkit.org/show_bug.cgi?id=55577
-    // and https://bugs.webkit.org/show_bug.cgi?id=52877
-
     Page* page = this->page();
     if (page && m_allowGeolocation == InProgress)
         page->geolocationController()->cancelPermissionRequest(this);