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

Side by Side Diff: chrome/browser/sync/glue/sync_backend_host.h

Issue 26594002: Chrome sync: Put WeakPtrFactory members at the end (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 2 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 | 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_SYNC_GLUE_SYNC_BACKEND_HOST_H_ 5 #ifndef CHROME_BROWSER_SYNC_GLUE_SYNC_BACKEND_HOST_H_
6 #define CHROME_BROWSER_SYNC_GLUE_SYNC_BACKEND_HOST_H_ 6 #define CHROME_BROWSER_SYNC_GLUE_SYNC_BACKEND_HOST_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 501 matching lines...) Expand 10 before | Expand all | Expand 10 after
512 int type, 512 int type,
513 const content::NotificationSource& source, 513 const content::NotificationSource& source,
514 const content::NotificationDetails& details) OVERRIDE; 514 const content::NotificationDetails& details) OVERRIDE;
515 515
516 // InvalidationHandler implementation. 516 // InvalidationHandler implementation.
517 virtual void OnInvalidatorStateChange( 517 virtual void OnInvalidatorStateChange(
518 syncer::InvalidatorState state) OVERRIDE; 518 syncer::InvalidatorState state) OVERRIDE;
519 virtual void OnIncomingInvalidation( 519 virtual void OnIncomingInvalidation(
520 const syncer::ObjectIdInvalidationMap& invalidation_map) OVERRIDE; 520 const syncer::ObjectIdInvalidationMap& invalidation_map) OVERRIDE;
521 521
522 base::WeakPtrFactory<SyncBackendHost> weak_ptr_factory_;
523
524 content::NotificationRegistrar notification_registrar_; 522 content::NotificationRegistrar notification_registrar_;
525 523
526 // A reference to the MessageLoop used to construct |this|, so we know how 524 // A reference to the MessageLoop used to construct |this|, so we know how
527 // to safely talk back to the SyncFrontend. 525 // to safely talk back to the SyncFrontend.
528 base::MessageLoop* const frontend_loop_; 526 base::MessageLoop* const frontend_loop_;
529 527
530 Profile* const profile_; 528 Profile* const profile_;
531 529
532 // Name used for debugging (set from profile_->GetDebugName()). 530 // Name used for debugging (set from profile_->GetDebugName()).
533 const std::string name_; 531 const std::string name_;
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
574 // Temporary holder of sync manager's initialization results. Set by 572 // Temporary holder of sync manager's initialization results. Set by
575 // HandleSyncManagerInitializationOnFrontendLoop, and consumed when we pass 573 // HandleSyncManagerInitializationOnFrontendLoop, and consumed when we pass
576 // it via OnBackendInitialized in the final state of 574 // it via OnBackendInitialized in the final state of
577 // HandleInitializationCompletedOnFrontendLoop. 575 // HandleInitializationCompletedOnFrontendLoop.
578 syncer::WeakHandle<syncer::JsBackend> js_backend_; 576 syncer::WeakHandle<syncer::JsBackend> js_backend_;
579 syncer::WeakHandle<syncer::DataTypeDebugInfoListener> debug_info_listener_; 577 syncer::WeakHandle<syncer::DataTypeDebugInfoListener> debug_info_listener_;
580 578
581 invalidation::InvalidationService* invalidator_; 579 invalidation::InvalidationService* invalidator_;
582 bool invalidation_handler_registered_; 580 bool invalidation_handler_registered_;
583 581
582 base::WeakPtrFactory<SyncBackendHost> weak_ptr_factory_;
583
584 DISALLOW_COPY_AND_ASSIGN(SyncBackendHost); 584 DISALLOW_COPY_AND_ASSIGN(SyncBackendHost);
585 }; 585 };
586 586
587 } // namespace browser_sync 587 } // namespace browser_sync
588 588
589 #endif // CHROME_BROWSER_SYNC_GLUE_SYNC_BACKEND_HOST_H_ 589 #endif // CHROME_BROWSER_SYNC_GLUE_SYNC_BACKEND_HOST_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698