Changeset 46663 in webkit

Timestamp:

Jul 31, 2009 11:43:20 PM (15 years ago)

Author:

abarth@webkit.org

Message:

2009-07-31 Yong Li <​yong.li@torchmobile.com>

Reviewed by Adam Barth.

Fix Geolocation permission problem
​https://bugs.webkit.org/show_bug.cgi?id=26993

• page/Geolocation.cpp: (WebCore::Geolocation::requestPermission):

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• page/Geolocation.cpp (modified) (2 diffs)

trunk/WebCore/ChangeLog

@@ +1 @@
+2009-07-31  Yong Li  <yong.li@torchmobile.com>
+
+        Reviewed by Adam Barth.
+
+        Fix Geolocation permission problem
+        https://bugs.webkit.org/show_bug.cgi?id=26993
+
+        * page/Geolocation.cpp:
+        (WebCore::Geolocation::requestPermission):
+
 2009-07-31  Norbert Leser  <norbert.leser@nokia.com>
 

trunk/WebCore/page/Geolocation.cpp

@@ -1 +1 @@
 /*
  * Copyright (C) 2008, 2009 Apple Inc. All Rights Reserved.
+ * Copyright (C) 2009 Torch Mobile, Inc.
  *
  * Redistribution and use in source and binary forms, with or without
@@ -262 +263 @@
         return;
     
+    m_allowGeolocation = InProgress;
+
     // Ask the chrome: it maintains the geolocation challenge policy itself.
     page->chrome()->requestGeolocationPermissionForFrame(m_frame, this);
-    
-    m_allowGeolocation = InProgress;
 }