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

Side by Side Diff: chrome/browser/sync_file_system/drive_backend/metadata_database_index.cc

Issue 188293005: [SyncFS] Add UMAs to track syncfs usage (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 9 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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/sync_file_system/drive_backend/metadata_database_index. h" 5 #include "chrome/browser/sync_file_system/drive_backend/metadata_database_index. h"
6 6
7 #include "base/metrics/histogram.h"
7 #include "chrome/browser/sync_file_system/drive_backend/drive_backend_constants. h" 8 #include "chrome/browser/sync_file_system/drive_backend/drive_backend_constants. h"
8 #include "chrome/browser/sync_file_system/drive_backend/metadata_database.pb.h" 9 #include "chrome/browser/sync_file_system/drive_backend/metadata_database.pb.h"
9 10
10 namespace sync_file_system { 11 namespace sync_file_system {
11 namespace drive_backend { 12 namespace drive_backend {
12 13
13 ParentIDAndTitle::ParentIDAndTitle() : parent_id(0) {} 14 ParentIDAndTitle::ParentIDAndTitle() : parent_id(0) {}
14 ParentIDAndTitle::ParentIDAndTitle(int64 parent_id, 15 ParentIDAndTitle::ParentIDAndTitle(int64 parent_id,
15 const std::string& title) 16 const std::string& title)
16 : parent_id(parent_id), title(title) {} 17 : parent_id(parent_id), title(title) {}
(...skipping 34 matching lines...) Expand 10 before | Expand all | Expand 10 after
51 } // namespace 52 } // namespace
52 53
53 MetadataDatabaseIndex::MetadataDatabaseIndex(DatabaseContents* content) { 54 MetadataDatabaseIndex::MetadataDatabaseIndex(DatabaseContents* content) {
54 for (size_t i = 0; i < content->file_metadata.size(); ++i) 55 for (size_t i = 0; i < content->file_metadata.size(); ++i)
55 StoreFileMetadata(make_scoped_ptr(content->file_metadata[i])); 56 StoreFileMetadata(make_scoped_ptr(content->file_metadata[i]));
56 content->file_metadata.weak_clear(); 57 content->file_metadata.weak_clear();
57 58
58 for (size_t i = 0; i < content->file_trackers.size(); ++i) 59 for (size_t i = 0; i < content->file_trackers.size(); ++i)
59 StoreFileTracker(make_scoped_ptr(content->file_trackers[i])); 60 StoreFileTracker(make_scoped_ptr(content->file_trackers[i]));
60 content->file_trackers.weak_clear(); 61 content->file_trackers.weak_clear();
62
63 UMA_HISTOGRAM_COUNTS("SyncFileSystem.MetadataNumber", metadata_by_id_.size());
64 UMA_HISTOGRAM_COUNTS("SyncFileSystem.TrackerNumber", tracker_by_id_.size());
65 UMA_HISTOGRAM_COUNTS_100("SyncFileSystem.RegisteredAppNumber",
66 app_root_by_app_id_.size());
61 } 67 }
62 68
63 MetadataDatabaseIndex::~MetadataDatabaseIndex() {} 69 MetadataDatabaseIndex::~MetadataDatabaseIndex() {}
64 70
65 const FileTracker* MetadataDatabaseIndex::GetFileTracker( 71 const FileTracker* MetadataDatabaseIndex::GetFileTracker(
66 int64 tracker_id) const { 72 int64 tracker_id) const {
67 return tracker_by_id_.get(tracker_id); 73 return tracker_by_id_.get(tracker_id);
68 } 74 }
69 75
70 const FileMetadata* MetadataDatabaseIndex::GetFileMetadata( 76 const FileMetadata* MetadataDatabaseIndex::GetFileMetadata(
(...skipping 389 matching lines...) Expand 10 before | Expand all | Expand 10 after
460 466
461 DVLOG(3) << " Remove from dirty_trackers_: " << tracker_id; 467 DVLOG(3) << " Remove from dirty_trackers_: " << tracker_id;
462 dirty_trackers_.erase(tracker_id); 468 dirty_trackers_.erase(tracker_id);
463 469
464 demoted_dirty_trackers_.erase(tracker_id); 470 demoted_dirty_trackers_.erase(tracker_id);
465 } 471 }
466 } 472 }
467 473
468 } // namespace drive_backend 474 } // namespace drive_backend
469 } // namespace sync_file_system 475 } // namespace sync_file_system
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698