So it looks like you only need "struct SafeBrowsingProtocolConfig" from chrome/browser/safe_browsing/protocol_manager_helper.h

This can be moved into "components/safe_browsing_db/util.h" which is already included in chrome/browser/safe_browsing/ping_manager.h

Wait, I take that back partially -- it does need the other methods defined in that file.

However, that file can just be removed and its contents placed in components/safe_browsing_db/util.*

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d06419a9457679d7daf3e513c3f72431233ca708

commit d06419a9457679d7daf3e513c3f72431233ca708
Author: jialiul <jialiul@chromium.org>
Date: Mon Feb 27 19:40:11 2017

Componentize ping_manager

Consolidate functions in ProtocolManagerHelper and put them inside
components/safe_browsing_db/util.*.
Separate BasePingManager and SafeBrowsingPingManager, where:
BasePingManager handles basic safe browsing pings such as HitReport,
ThreatDetails report etc.
Leave NotificationImageReport and PermissionReport to
SafeBrowsingPingManager due to their chrome/browser/ dependencies.
BUG= 684805 

Review-Url: https://codereview.chromium.org/2650973005
Cr-Commit-Position: refs/heads/master@{#453300}

[modify] https://crrev.com/d06419a9457679d7daf3e513c3f72431233ca708/chrome/browser/BUILD.gn
[modify] https://crrev.com/d06419a9457679d7daf3e513c3f72431233ca708/chrome/browser/extensions/blacklist_state_fetcher.cc
[modify] https://crrev.com/d06419a9457679d7daf3e513c3f72431233ca708/chrome/browser/extensions/blacklist_state_fetcher.h
[modify] https://crrev.com/d06419a9457679d7daf3e513c3f72431233ca708/chrome/browser/safe_browsing/ping_manager.cc
[modify] https://crrev.com/d06419a9457679d7daf3e513c3f72431233ca708/chrome/browser/safe_browsing/ping_manager.h
[modify] https://crrev.com/d06419a9457679d7daf3e513c3f72431233ca708/chrome/browser/safe_browsing/protocol_manager.cc
[modify] https://crrev.com/d06419a9457679d7daf3e513c3f72431233ca708/chrome/browser/safe_browsing/protocol_manager.h
[delete] https://crrev.com/cb3da9c98f24a2fe73a40d490548e1a9acf1df1d/chrome/browser/safe_browsing/protocol_manager_helper.cc
[delete] https://crrev.com/cb3da9c98f24a2fe73a40d490548e1a9acf1df1d/chrome/browser/safe_browsing/protocol_manager_helper.h
[modify] https://crrev.com/d06419a9457679d7daf3e513c3f72431233ca708/chrome/browser/safe_browsing/safe_browsing_service.cc
[modify] https://crrev.com/d06419a9457679d7daf3e513c3f72431233ca708/chrome/browser/safe_browsing/test_safe_browsing_service.h
[modify] https://crrev.com/d06419a9457679d7daf3e513c3f72431233ca708/chrome/test/BUILD.gn
[modify] https://crrev.com/d06419a9457679d7daf3e513c3f72431233ca708/components/safe_browsing/BUILD.gn
[modify] https://crrev.com/d06419a9457679d7daf3e513c3f72431233ca708/components/safe_browsing/DEPS
[add] https://crrev.com/d06419a9457679d7daf3e513c3f72431233ca708/components/safe_browsing/base_ping_manager.cc
[add] https://crrev.com/d06419a9457679d7daf3e513c3f72431233ca708/components/safe_browsing/base_ping_manager.h
[rename] https://crrev.com/d06419a9457679d7daf3e513c3f72431233ca708/components/safe_browsing/base_ping_manager_unittest.cc
[modify] https://crrev.com/d06419a9457679d7daf3e513c3f72431233ca708/components/safe_browsing_db/BUILD.gn
[modify] https://crrev.com/d06419a9457679d7daf3e513c3f72431233ca708/components/safe_browsing_db/DEPS
[modify] https://crrev.com/d06419a9457679d7daf3e513c3f72431233ca708/components/safe_browsing_db/util.cc
[modify] https://crrev.com/d06419a9457679d7daf3e513c3f72431233ca708/components/safe_browsing_db/util.h