summaryrefslogtreecommitdiffstats
path: root/chrome/common
diff options
context:
space:
mode:
authorphajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-02-19 18:23:25 +0000
committerphajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-02-19 18:23:25 +0000
commit1933eb200c4bb16e4df535d779c98561f85c1e25 (patch)
tree0ec1fe2dc176b2159abf8e33c53eaf940fb310c4 /chrome/common
parentfd599eeeddf2a57a2b2d70e63725106ce93c4591 (diff)
downloadchromium_src-1933eb200c4bb16e4df535d779c98561f85c1e25.zip
chromium_src-1933eb200c4bb16e4df535d779c98561f85c1e25.tar.gz
chromium_src-1933eb200c4bb16e4df535d779c98561f85c1e25.tar.bz2
Clean up dns prefetch code, and also port it.
- remove slave threads and use HostResolver in asynchronous mode instead (while still limiting number of concurrent lookups) - make the implementation portable and make DnsMaster unit test compile and pass on Linux - add more tests to DnsMaster unit test to simulate various shutdown scenarios, concurrent lookups, and to verify that we don't exceed our limit of concurrent lookup requests) - remove some tests which relied on specifics of slaves' inner working - adjust initialization and shutdown of dns prefetching (now it relies on the IO message loop being present) Bonus: shutdown is almost instant now, no need to have a timeout. BUG=5687, 6683 Review URL: http://codereview.chromium.org/15076 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@10021 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/common')
-rw-r--r--chrome/common/temp_scaffolding_stubs.cc26
1 files changed, 3 insertions, 23 deletions
diff --git a/chrome/common/temp_scaffolding_stubs.cc b/chrome/common/temp_scaffolding_stubs.cc
index 78a223a..25520df 100644
--- a/chrome/common/temp_scaffolding_stubs.cc
+++ b/chrome/common/temp_scaffolding_stubs.cc
@@ -164,7 +164,7 @@ void OpenFirstRunDialog(Profile* profile) { NOTIMPLEMENTED(); }
GURL NewTabUIURL() {
NOTIMPLEMENTED();
- // TODO(port): returning a blank URL here confuses the page IDs so make sure
+ // TODO(port): returning a blank URL here confuses the page IDs so make sure
// we load something
return GURL("http://dev.chromium.org");
}
@@ -277,7 +277,7 @@ const std::wstring& TabContents::GetTitle() const {
NavigationEntry* entry = controller_->GetTransientEntry();
if (entry)
return entry->GetTitleForDisplay();
-
+
entry = controller_->GetLastCommittedEntry();
if (entry)
return entry->GetTitleForDisplay();
@@ -326,7 +326,7 @@ void TabContents::UpdateMaxPageID(int32 page_id) {
// testing.
if (GetSiteInstance())
GetSiteInstance()->UpdateMaxPageID(page_id);
-
+
if (AsWebContents())
AsWebContents()->process()->UpdateMaxPageID(page_id);
else
@@ -358,26 +358,6 @@ bool IsPluginProcess() {
//--------------------------------------------------------------------------
-namespace chrome_browser_net {
-
-void EnableDnsPrefetch(bool) { NOTIMPLEMENTED(); }
-
-void DnsPrefetchGetHtmlInfo(std::string* output) { NOTIMPLEMENTED(); }
-
-void DnsPrefetchList(const std::vector<std::string>& hostnames) {
- NOTIMPLEMENTED();
-}
-
-void SaveHostNamesForNextStartup(PrefService* local_state) { NOTIMPLEMENTED(); }
-
-void TrimSubresourceReferrers() { NOTIMPLEMENTED(); }
-
-void SaveSubresourceReferrers(PrefService* local_state) { NOTIMPLEMENTED(); }
-
-} // namespace chrome_browser_net
-
-//--------------------------------------------------------------------------
-
// This is from chrome_plugin_util.cc.
void CPB_Free(void* memory) { NOTIMPLEMENTED(); }