[Webkit-unassigned] [Bug 39879] Geolocation activity started after frame has been disconnected can cause crash

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jun 8 08:35:02 PDT 2010


Jeremy Orlow <jorlow at chromium.org> changed:

           What    |Removed                     |Added
  Attachment #58137|review?                     |review-
               Flag|                            |

--- Comment #17 from Jeremy Orlow <jorlow at chromium.org>  2010-06-08 08:35:00 PST ---
(From update of attachment 58137)
 +      // This method is called at most once on a given GeoNotifier object.
Why is this getting removed?  What happens if the timer has already started for another error?  Etc...  This probably deserves mention in the change log.

 +          RefPtr<GeoNotifier> notifier = *it;
Why do you need to do this?  It doesn't seem like it saves any space.  I guess it's slightly more readable....

 +      copyToVector(m_oneShots, copy);
Do you need to null out the list?

 +      m_watchers.getNotifiersVector(copy);
This shouldn't have had get in the name...

 +      void cancelAllRequests(Vector<RefPtr<GeoNotifier> >&);
Why name this the same thing and make it public?  It seems like just a helper function.

 +  static const char serviceNoLongerAvailableErrorMessage[] = "Geolocation service no longer available";
Are these messages supposed to not have a period at the end?

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

More information about the webkit-unassigned mailing list