Get more useful LOG(FATAL) messages from ExtensionSyncData to help diagnose crashes.

chrome/browser/extensions/extension_sync_data.cc

 // Copyright (c) 2012 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/extensions/extension_sync_data.h"
 
 #include "base/logging.h"
+#include "base/strings/stringprintf.h"
 #include "chrome/browser/extensions/app_sync_data.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "components/crx_file/id_util.h"
 #include "extensions/common/extension.h"
 #include "extensions/common/manifest_url_handlers.h"
 #include "sync/api/sync_data.h"
 #include "sync/protocol/extension_specifics.pb.h"
 #include "sync/protocol/sync.pb.h"
 
 namespace extensions {
 
+namespace {
+
+std::string GetExtensionSpecificsLogMessage(
+    const sync_pb::ExtensionSpecifics& specifics) {
+  return base::StringPrintf("id: %s\nversion: %s\nupdate_url: %s",
+                            specifics.id().c_str(),
+                            specifics.version().c_str(),
+                            specifics.update_url().c_str());
+}
+
+}  // namespace
+
 ExtensionSyncData::ExtensionSyncData()
     : uninstalled_(false),
       enabled_(false),
       incognito_enabled_(false),
       remote_install_(false),
       all_urls_enabled_(BOOLEAN_UNSET),
       installed_by_custodian_(false) {
 }
 
 ExtensionSyncData::ExtensionSyncData(const syncer::SyncData& sync_data)
@@ skipping 60 matching lines @@
   specifics->set_remote_install(remote_install_);
   if (all_urls_enabled_ != BOOLEAN_UNSET)
     specifics->set_all_urls_enabled(all_urls_enabled_ == BOOLEAN_TRUE);
   specifics->set_installed_by_custodian(installed_by_custodian_);
   specifics->set_name(name_);
 }
 
 void ExtensionSyncData::PopulateFromExtensionSpecifics(
     const sync_pb::ExtensionSpecifics& specifics) {
   if (!crx_file::id_util::IdIsValid(specifics.id())) {
-    LOG(FATAL) << "Attempt to sync bad ExtensionSpecifics.";
+    LOG(FATAL) << "Attempt to sync bad ExtensionSpecifics (bad ID):\n"
+               << GetExtensionSpecificsLogMessage(specifics);
   }
 
   Version specifics_version(specifics.version());
-  if (!specifics_version.IsValid())
-    LOG(FATAL) << "Attempt to sync bad ExtensionSpecifics.";
+  if (!specifics_version.IsValid()) {
+    LOG(FATAL) << "Attempt to sync bad ExtensionSpecifics (bad version):\n"
+               << GetExtensionSpecificsLogMessage(specifics);
+  }
 
   // The update URL must be either empty or valid.
   GURL specifics_update_url(specifics.update_url());
   if (!specifics_update_url.is_empty() && !specifics_update_url.is_valid()) {
-    LOG(FATAL) << "Attempt to sync bad ExtensionSpecifics.";
+    LOG(FATAL) << "Attempt to sync bad ExtensionSpecifics (bad update URL):\n"
+               << GetExtensionSpecificsLogMessage(specifics);
   }
 
   id_ = specifics.id();
   update_url_ = specifics_update_url;
   version_ = specifics_version;
   enabled_ = specifics.enabled();
   incognito_enabled_ = specifics.incognito_enabled();
   if (specifics.has_all_urls_enabled()) {
     all_urls_enabled_ =
         specifics.all_urls_enabled() ? BOOLEAN_TRUE : BOOLEAN_FALSE;
@@ skipping 16 matching lines @@
   const sync_pb::EntitySpecifics& entity_specifics = sync_data.GetSpecifics();
 
   if (entity_specifics.has_extension()) {
     PopulateFromExtensionSpecifics(entity_specifics.extension());
   } else {
     LOG(FATAL) << "Attempt to sync bad EntitySpecifics.";
   }
 }
 
 }  // namespace extensions