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

Side by Side Diff: chrome/browser/profiles/profile_io_data.h

Issue 61323002: Fix broken threading model in CheckDesktopNotificationPermission (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Even more merge conflicts.. Created 7 years, 1 month 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 | Annotate | Revision Log
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 #ifndef CHROME_BROWSER_PROFILES_PROFILE_IO_DATA_H_ 5 #ifndef CHROME_BROWSER_PROFILES_PROFILE_IO_DATA_H_
6 #define CHROME_BROWSER_PROFILES_PROFILE_IO_DATA_H_ 6 #define CHROME_BROWSER_PROFILES_PROFILE_IO_DATA_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 11 matching lines...) Expand all
22 #include "chrome/common/content_settings_types.h" 22 #include "chrome/common/content_settings_types.h"
23 #include "content/public/browser/content_browser_client.h" 23 #include "content/public/browser/content_browser_client.h"
24 #include "content/public/browser/resource_context.h" 24 #include "content/public/browser/resource_context.h"
25 #include "net/cookies/cookie_monster.h" 25 #include "net/cookies/cookie_monster.h"
26 #include "net/http/http_network_session.h" 26 #include "net/http/http_network_session.h"
27 #include "net/url_request/url_request_job_factory.h" 27 #include "net/url_request/url_request_job_factory.h"
28 28
29 class ChromeHttpUserAgentSettings; 29 class ChromeHttpUserAgentSettings;
30 class ChromeNetworkDelegate; 30 class ChromeNetworkDelegate;
31 class CookieSettings; 31 class CookieSettings;
32 class DesktopNotificationService;
33 class HostContentSettingsMap; 32 class HostContentSettingsMap;
34 class ManagedModeURLFilter; 33 class ManagedModeURLFilter;
35 class Profile; 34 class Profile;
36 class ProtocolHandlerRegistry; 35 class ProtocolHandlerRegistry;
37 class SigninNamesOnIOThread; 36 class SigninNamesOnIOThread;
38 37
39 namespace chrome_browser_net { 38 namespace chrome_browser_net {
40 class LoadTimeStats; 39 class LoadTimeStats;
41 class ResourcePrefetchPredictorObserver; 40 class ResourcePrefetchPredictorObserver;
42 } 41 }
(...skipping 70 matching lines...) Expand 10 before | Expand all | Expand 10 after
113 ChromeURLRequestContext* app_context, 112 ChromeURLRequestContext* app_context,
114 const StoragePartitionDescriptor& partition_descriptor) const; 113 const StoragePartitionDescriptor& partition_descriptor) const;
115 114
116 // These are useful when the Chrome layer is called from the content layer 115 // These are useful when the Chrome layer is called from the content layer
117 // with a content::ResourceContext, and they want access to Chrome data for 116 // with a content::ResourceContext, and they want access to Chrome data for
118 // that profile. 117 // that profile.
119 extensions::InfoMap* GetExtensionInfoMap() const; 118 extensions::InfoMap* GetExtensionInfoMap() const;
120 CookieSettings* GetCookieSettings() const; 119 CookieSettings* GetCookieSettings() const;
121 HostContentSettingsMap* GetHostContentSettingsMap() const; 120 HostContentSettingsMap* GetHostContentSettingsMap() const;
122 121
123 #if defined(ENABLE_NOTIFICATIONS)
124 DesktopNotificationService* GetNotificationService() const;
125 #endif
126
127 IntegerPrefMember* session_startup_pref() const { 122 IntegerPrefMember* session_startup_pref() const {
128 return &session_startup_pref_; 123 return &session_startup_pref_;
129 } 124 }
130 125
131 SigninNamesOnIOThread* signin_names() const { 126 SigninNamesOnIOThread* signin_names() const {
132 return signin_names_.get(); 127 return signin_names_.get();
133 } 128 }
134 129
135 StringPrefMember* google_services_username() const { 130 StringPrefMember* google_services_username() const {
136 return &google_services_username_; 131 return &google_services_username_;
(...skipping 113 matching lines...) Expand 10 before | Expand all | Expand 10 after
250 base::FilePath path; 245 base::FilePath path;
251 IOThread* io_thread; 246 IOThread* io_thread;
252 scoped_refptr<CookieSettings> cookie_settings; 247 scoped_refptr<CookieSettings> cookie_settings;
253 scoped_refptr<HostContentSettingsMap> host_content_settings_map; 248 scoped_refptr<HostContentSettingsMap> host_content_settings_map;
254 scoped_refptr<net::SSLConfigService> ssl_config_service; 249 scoped_refptr<net::SSLConfigService> ssl_config_service;
255 scoped_refptr<net::CookieMonster::Delegate> cookie_monster_delegate; 250 scoped_refptr<net::CookieMonster::Delegate> cookie_monster_delegate;
256 scoped_refptr<extensions::InfoMap> extension_info_map; 251 scoped_refptr<extensions::InfoMap> extension_info_map;
257 scoped_ptr<chrome_browser_net::ResourcePrefetchPredictorObserver> 252 scoped_ptr<chrome_browser_net::ResourcePrefetchPredictorObserver>
258 resource_prefetch_predictor_observer_; 253 resource_prefetch_predictor_observer_;
259 254
260 #if defined(ENABLE_NOTIFICATIONS)
261 DesktopNotificationService* notification_service;
262 #endif
263
264 // This pointer exists only as a means of conveying a url job factory 255 // This pointer exists only as a means of conveying a url job factory
265 // pointer from the protocol handler registry on the UI thread to the 256 // pointer from the protocol handler registry on the UI thread to the
266 // the URLRequestContext on the IO thread. The consumer MUST take 257 // the URLRequestContext on the IO thread. The consumer MUST take
267 // ownership of the object by calling release() on this pointer. 258 // ownership of the object by calling release() on this pointer.
268 scoped_ptr<ProtocolHandlerRegistry::JobInterceptorFactory> 259 scoped_ptr<ProtocolHandlerRegistry::JobInterceptorFactory>
269 protocol_handler_interceptor; 260 protocol_handler_interceptor;
270 261
271 // We need to initialize the ProxyConfigService from the UI thread 262 // We need to initialize the ProxyConfigService from the UI thread
272 // because on linux it relies on initializing things through gconf, 263 // because on linux it relies on initializing things through gconf,
273 // and needs to be on the main thread. 264 // and needs to be on the main thread.
(...skipping 223 matching lines...) Expand 10 before | Expand all | Expand 10 after
497 mutable scoped_ptr<net::FraudulentCertificateReporter> 488 mutable scoped_ptr<net::FraudulentCertificateReporter>
498 fraudulent_certificate_reporter_; 489 fraudulent_certificate_reporter_;
499 mutable scoped_ptr<net::ProxyService> proxy_service_; 490 mutable scoped_ptr<net::ProxyService> proxy_service_;
500 mutable scoped_ptr<net::TransportSecurityState> transport_security_state_; 491 mutable scoped_ptr<net::TransportSecurityState> transport_security_state_;
501 mutable scoped_ptr<net::HttpServerProperties> 492 mutable scoped_ptr<net::HttpServerProperties>
502 http_server_properties_; 493 http_server_properties_;
503 #if defined(OS_CHROMEOS) 494 #if defined(OS_CHROMEOS)
504 mutable scoped_ptr<net::CertVerifier> cert_verifier_; 495 mutable scoped_ptr<net::CertVerifier> cert_verifier_;
505 #endif 496 #endif
506 497
507 #if defined(ENABLE_NOTIFICATIONS)
508 mutable DesktopNotificationService* notification_service_;
509 #endif
510
511 mutable scoped_ptr<net::TransportSecurityPersister> 498 mutable scoped_ptr<net::TransportSecurityPersister>
512 transport_security_persister_; 499 transport_security_persister_;
513 500
514 // These are only valid in between LazyInitialize() and their accessor being 501 // These are only valid in between LazyInitialize() and their accessor being
515 // called. 502 // called.
516 mutable scoped_ptr<ChromeURLRequestContext> main_request_context_; 503 mutable scoped_ptr<ChromeURLRequestContext> main_request_context_;
517 mutable scoped_ptr<ChromeURLRequestContext> extensions_request_context_; 504 mutable scoped_ptr<ChromeURLRequestContext> extensions_request_context_;
518 // One URLRequestContext per isolated app for main and media requests. 505 // One URLRequestContext per isolated app for main and media requests.
519 mutable URLRequestContextMap app_request_context_map_; 506 mutable URLRequestContextMap app_request_context_map_;
520 mutable URLRequestContextMap isolated_media_request_context_map_; 507 mutable URLRequestContextMap isolated_media_request_context_map_;
(...skipping 18 matching lines...) Expand all
539 526
540 // TODO(jhawkins): Remove once crbug.com/102004 is fixed. 527 // TODO(jhawkins): Remove once crbug.com/102004 is fixed.
541 bool initialized_on_UI_thread_; 528 bool initialized_on_UI_thread_;
542 529
543 bool is_incognito_; 530 bool is_incognito_;
544 531
545 DISALLOW_COPY_AND_ASSIGN(ProfileIOData); 532 DISALLOW_COPY_AND_ASSIGN(ProfileIOData);
546 }; 533 };
547 534
548 #endif // CHROME_BROWSER_PROFILES_PROFILE_IO_DATA_H_ 535 #endif // CHROME_BROWSER_PROFILES_PROFILE_IO_DATA_H_
OLDNEW
« no previous file with comments | « chrome/browser/notifications/desktop_notification_service_unittest.cc ('k') | chrome/browser/profiles/profile_io_data.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698