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

Unified Diff: chrome/browser/sync/internal_api/sync_manager.cc

Issue 9836100: Add full text regex searching to chrome://sync (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix typo Created 8 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/sync/internal_api/sync_manager.cc
diff --git a/chrome/browser/sync/internal_api/sync_manager.cc b/chrome/browser/sync/internal_api/sync_manager.cc
index 871abaf4bd1630572e61f71046cd890f91019958..30dbce97cd2f9164b8f97622f5b87bdf5ed2f534 100644
--- a/chrome/browser/sync/internal_api/sync_manager.cc
+++ b/chrome/browser/sync/internal_api/sync_manager.cc
@@ -176,12 +176,12 @@ class SyncManager::SyncInternal
"getNodeDetailsById",
&SyncManager::SyncInternal::GetNodeDetailsById);
BindJsMessageHandler(
+ "getAllNodes",
+ &SyncManager::SyncInternal::GetAllNodes);
+ BindJsMessageHandler(
"getChildNodeIds",
&SyncManager::SyncInternal::GetChildNodeIds);
BindJsMessageHandler(
- "findNodesContainingString",
- &SyncManager::SyncInternal::FindNodesContainingString);
- BindJsMessageHandler(
"getClientServerTraffic",
&SyncManager::SyncInternal::GetClientServerTraffic);
}
@@ -526,10 +526,10 @@ class SyncManager::SyncInternal
JsArgList GetNotificationState(const JsArgList& args);
JsArgList GetNotificationInfo(const JsArgList& args);
JsArgList GetRootNodeDetails(const JsArgList& args);
+ JsArgList GetAllNodes(const JsArgList& args);
JsArgList GetNodeSummariesById(const JsArgList& args);
JsArgList GetNodeDetailsById(const JsArgList& args);
JsArgList GetChildNodeIds(const JsArgList& args);
- JsArgList FindNodesContainingString(const JsArgList& args);
JsArgList GetClientServerTraffic(const JsArgList& args);
FilePath database_path_;
@@ -2291,6 +2291,25 @@ JsArgList SyncManager::SyncInternal::GetNodeDetailsById(
return GetNodeInfoById(args, GetUserShare(), &BaseNode::GetDetailsAsValue);
}
+JsArgList SyncManager::SyncInternal::GetAllNodes(
+ const JsArgList& args) {
+ ListValue return_args;
+ ListValue* result = new ListValue();
+ return_args.Append(result);
+
+ ReadTransaction trans(FROM_HERE, GetUserShare());
+ std::vector<const syncable::EntryKernel*> entry_kernels;
+ trans.GetDirectory()->GetAllEntryKernels(trans.GetWrappedTrans(),
+ &entry_kernels);
+
+ for (std::vector<const syncable::EntryKernel*>::const_iterator it =
+ entry_kernels.begin(); it != entry_kernels.end(); ++it) {
+ result->Append((*it)->ToValue());
+ }
+
+ return JsArgList(&return_args);
+}
+
JsArgList SyncManager::SyncInternal::GetChildNodeIds(
const JsArgList& args) {
ListValue return_args;
@@ -2311,39 +2330,6 @@ JsArgList SyncManager::SyncInternal::GetChildNodeIds(
return JsArgList(&return_args);
}
-JsArgList SyncManager::SyncInternal::FindNodesContainingString(
- const JsArgList& args) {
- std::string query;
- ListValue return_args;
- if (!args.Get().GetString(0, &query)) {
- return_args.Append(new ListValue());
- return JsArgList(&return_args);
- }
-
- // Convert the query string to lower case to perform case insensitive
- // searches.
- std::string lowercase_query = query;
- StringToLowerASCII(&lowercase_query);
-
- ListValue* result = new ListValue();
- return_args.Append(result);
-
- ReadTransaction trans(FROM_HERE, GetUserShare());
- std::vector<const syncable::EntryKernel*> entry_kernels;
- trans.GetDirectory()->GetAllEntryKernels(trans.GetWrappedTrans(),
- &entry_kernels);
-
- for (std::vector<const syncable::EntryKernel*>::const_iterator it =
- entry_kernels.begin(); it != entry_kernels.end(); ++it) {
- if ((*it)->ContainsString(lowercase_query)) {
- result->Append(new StringValue(base::Int64ToString(
- (*it)->ref(syncable::META_HANDLE))));
- }
- }
-
- return JsArgList(&return_args);
-}
-
void SyncManager::SyncInternal::OnEncryptedTypesChanged(
syncable::ModelTypeSet encrypted_types,
bool encrypt_everything) {
« no previous file with comments | « chrome/browser/resources/sync_internals/sync_search.js ('k') | chrome/browser/sync/internal_api/syncapi_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698