Offer user to send feedback from profile error dialog

components/webdata/common/web_database_backend.cc

 // Copyright 2013 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 "components/webdata/common/web_database_backend.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/location.h"
@@ skipping 17 matching lines @@
 }
 
 void WebDatabaseBackend::AddTable(std::unique_ptr<WebDatabaseTable> table) {
   DCHECK(!db_.get());
   tables_.push_back(table.release());
 }
 
 void WebDatabaseBackend::InitDatabase() {
   LoadDatabaseIfNecessary();
   if (delegate_) {
-    delegate_->DBLoaded(init_status_);
+    delegate_->DBLoaded(init_status_, db_diagnostics_);
   }
 }
 
 sql::InitStatus WebDatabaseBackend::LoadDatabaseIfNecessary() {
   if (init_complete_ || db_path_.empty()) {
     return init_status_;
   }
   init_complete_ = true;
   db_.reset(new WebDatabase());
 
-  for (ScopedVector<WebDatabaseTable>::iterator it = tables_.begin();
-       it != tables_.end(); ++it) {
-    db_->AddTable(*it);
-  }
+  for (const auto& table : tables_)
+    db_->AddTable(table);
 
   init_status_ = db_->Init(db_path_);
   if (init_status_ != sql::INIT_OK) {
     LOG(ERROR) << "Cannot initialize the web database: " << init_status_;
+    db_diagnostics_ = db_->GetDiagnosticMap();
+    db_diagnostics_["Corrupted file"] = db_path_.AsUTF8Unsafe();

[michaeln, 2016/07/09 03:00:38]

we don't want to send the full path as part of a bug report

[Scott Hess - ex-Googler, 2016/07/10 05:16:52]

I think I had earlier comments about this - we should only include what suffix
of the path which is sufficient to distinguish databases.  I can't think of any
reason why the PII in the front of the path would be helpful as a diagnostic.

[afakhry, 2016/07/11 16:47:45]

Done.

[afakhry, 2016/07/11 16:47:45]

Yes, I needed to think about a way to distinguish between the user vs. default
profiles, and I forgot to do this in the previous patch. Now done.

     db_.reset(NULL);
     return init_status_;
   }
 
   db_->BeginTransaction();
   return init_status_;
 }
 
 void WebDatabaseBackend::ShutdownDatabase() {
   if (db_ && init_status_ == sql::INIT_OK)
@@ skipping 45 matching lines @@
 WebDatabaseBackend::~WebDatabaseBackend() {
   ShutdownDatabase();
 }
 
 void WebDatabaseBackend::Commit() {
   DCHECK(db_);
   DCHECK_EQ(sql::INIT_OK, init_status_);
   db_->CommitTransaction();
   db_->BeginTransaction();
 }