Comment #11 on issue 7900 by bugdro...@chromium.org: Opening a recently  
saved html page triggers multiple tabs to open
http://code.google.com/p/chromium/issues/detail?id=7900

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

------------------------------------------------------------------------
r10201 | phajdan...@chromium.org | 2009-02-23 11:22:13 -0800 (Mon, 23 Feb  
2009) | 13 lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/browser_process_impl.cc?r1=10201&r2=10200
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/net/dns_global.cc?r1=10201&r2=10200
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/net/dns_global.h?r1=10201&r2=10200

Fix a crash when dns prefetching was being shut down without being started.

This happened when BrowserMain exited before the point when we initialized  
dns prefetching, but after BrowserProcessImpl was created. We do shutdown  
of dns prefetching in ~BrowserProcessImpl.

Example way to trigger the crash: open a saved HTML page while Chrome is  
already running.

Change ShutdownDnsPrefetch to EnsureDnsPrefetchingShutdown which does  
nothing
if dns prefetching  has not been initialized. This is the simplest way
to prevent the crash and we don't want to complicate the startup sequence  
even more.

BUG=7900

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


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to