Stop logging users' history urls when url db is not available

chrome/browser/android/history_report/data_provider.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/android/history_report/data_provider.h"
 #include "base/bind.h"
 #include "base/containers/hash_tables.h"
 #include "base/logging.h"
 #include "base/synchronization/waitable_event.h"
 #include "chrome/browser/android/history_report/delta_file_commons.h"
 #include "chrome/browser/android/history_report/delta_file_service.h"
 #include "chrome/browser/android/history_report/get_all_urls_from_history_task.h"
 #include "chrome/browser/android/history_report/historic_visits_migration_task.h"
 #include "chrome/browser/android/history_report/usage_reports_buffer_service.h"
 #include "chrome/browser/history/history_service_factory.h"
 #include "chrome/browser/profiles/profile.h"
 #include "components/bookmarks/browser/bookmark_model.h"
 #include "components/history/core/browser/history_db_task.h"
 #include "components/history/core/browser/history_service.h"
 #include "content/public/browser/browser_thread.h"
 
 using bookmarks::BookmarkModel;
 
 namespace {
+static bool g_is_debug = false;
+
 typedef base::hash_map<std::string, BookmarkModel::URLAndTitle*> BookmarkMap;
 
 struct Context {
   history::HistoryService* history_service;
   base::CancelableTaskTracker* history_task_tracker;
   base::WaitableEvent finished;
 
   Context(history::HistoryService* hservice,
           base::CancelableTaskTracker* tracker)
       : history_service(hservice),
         history_task_tracker(tracker),
         finished(false, false) {}
 };
 
 void UpdateUrl(Context* context,
                size_t position,
                std::vector<history_report::DeltaFileEntryWithData>* urls,
                bool success,
                const history::URLRow& url,
                const history::VisitVector& visits) {
   history_report::DeltaFileEntryWithData* entry = &((*urls)[position]);
   if (success) {
     entry->SetData(url);
-  } else {
-    LOG(WARNING) << "No data for url " << entry->Url();
+  } else if (g_is_debug){
+    LOG(WARNING) << "DB not initialized or no data for url " << entry->Url();
   }
   if (position + 1 == urls->size()) {
     context->finished.Signal();
   } else {
     context->history_service->QueryURL(GURL((*urls)[position + 1].Url()),
                                        false,
                                        base::Bind(&UpdateUrl,
                                                   base::Unretained(context),
                                                   position + 1,
                                                   base::Unretained(urls)),
@@ skipping 124 matching lines @@
   bookmark_model_->BlockTillLoaded();
   bookmark_model_->GetBookmarks(&bookmarks);
   urls.reserve(urls.size() + bookmarks.size());
   for (size_t i = 0; i < bookmarks.size(); i++) {
     urls.push_back(bookmarks[i].url.spec());
   }
   delta_file_service_->Recreate(urls);
 }
 
 }  // namespace history_report