Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(290)

Side by Side Diff: chrome/browser/io_thread.cc

Issue 1818603002: Keep track of ChannelIDService in CookieStore (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: s/DCHECK(false)/NOTREACHED()/ Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | chrome/browser/profiles/off_the_record_profile_io_data.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/io_thread.h" 5 #include "chrome/browser/io_thread.h"
6 6
7 #include <utility> 7 #include <utility>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/base64.h" 10 #include "base/base64.h"
(...skipping 736 matching lines...) Expand 10 before | Expand all | Expand 10 after
747 // TODO(erikchen): Remove ScopedTracker below once http://crbug.com/466432 747 // TODO(erikchen): Remove ScopedTracker below once http://crbug.com/466432
748 // is fixed. 748 // is fixed.
749 tracked_objects::ScopedTracker tracking_profile12( 749 tracked_objects::ScopedTracker tracking_profile12(
750 FROM_HERE_WITH_EXPLICIT_FUNCTION( 750 FROM_HERE_WITH_EXPLICIT_FUNCTION(
751 "466432 IOThread::InitAsync::CreateCookieStore::End")); 751 "466432 IOThread::InitAsync::CreateCookieStore::End"));
752 // In-memory channel ID store. 752 // In-memory channel ID store.
753 globals_->system_channel_id_service.reset( 753 globals_->system_channel_id_service.reset(
754 new net::ChannelIDService( 754 new net::ChannelIDService(
755 new net::DefaultChannelIDStore(NULL), 755 new net::DefaultChannelIDStore(NULL),
756 base::WorkerPool::GetTaskRunner(true))); 756 base::WorkerPool::GetTaskRunner(true)));
757 globals_->system_cookie_store->SetChannelIDServiceID(
758 globals_->system_channel_id_service->GetUniqueID());
757 // TODO(erikchen): Remove ScopedTracker below once http://crbug.com/466432 759 // TODO(erikchen): Remove ScopedTracker below once http://crbug.com/466432
758 // is fixed. 760 // is fixed.
759 tracked_objects::ScopedTracker tracking_profile12_1( 761 tracked_objects::ScopedTracker tracking_profile12_1(
760 FROM_HERE_WITH_EXPLICIT_FUNCTION( 762 FROM_HERE_WITH_EXPLICIT_FUNCTION(
761 "466432 IOThread::InitAsync::CreateDnsProbeService")); 763 "466432 IOThread::InitAsync::CreateDnsProbeService"));
762 globals_->dns_probe_service.reset(new chrome_browser_net::DnsProbeService()); 764 globals_->dns_probe_service.reset(new chrome_browser_net::DnsProbeService());
763 // TODO(erikchen): Remove ScopedTracker below once http://crbug.com/466432 765 // TODO(erikchen): Remove ScopedTracker below once http://crbug.com/466432
764 // is fixed. 766 // is fixed.
765 tracked_objects::ScopedTracker tracking_profile12_2( 767 tracked_objects::ScopedTracker tracking_profile12_2(
766 FROM_HERE_WITH_EXPLICIT_FUNCTION( 768 FROM_HERE_WITH_EXPLICIT_FUNCTION(
(...skipping 989 matching lines...) Expand 10 before | Expand all | Expand 10 after
1756 std::move(job_factory); 1758 std::move(job_factory);
1757 1759
1758 context->set_job_factory( 1760 context->set_job_factory(
1759 globals->proxy_script_fetcher_url_request_job_factory.get()); 1761 globals->proxy_script_fetcher_url_request_job_factory.get());
1760 1762
1761 // TODO(rtenneti): We should probably use HttpServerPropertiesManager for the 1763 // TODO(rtenneti): We should probably use HttpServerPropertiesManager for the
1762 // system URLRequestContext too. There's no reason this should be tied to a 1764 // system URLRequestContext too. There's no reason this should be tied to a
1763 // profile. 1765 // profile.
1764 return context; 1766 return context;
1765 } 1767 }
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/profiles/off_the_record_profile_io_data.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698