Refactor safe-browsing build-config definitions.

chrome/browser/profiles/profile.cc

Index: chrome/browser/profiles/profile.cc
diff --git a/chrome/browser/profiles/profile.cc b/chrome/browser/profiles/profile.cc
index daa082820030a8c8707c1fddfe2cc7cbf18e1738..e42e30d109b041af955da3afdef62962332880c9 100644
--- a/chrome/browser/profiles/profile.cc
+++ b/chrome/browser/profiles/profile.cc
@@ -29,6 +29,7 @@
 #include "chromeos/chromeos_switches.h"
 #endif
 
+// TODO(shess): Should possibly be SAFE_BROWSING_SERVICE.
 #if defined(OS_ANDROID) && defined(FULL_SAFE_BROWSING)
 #include "chrome/browser/safe_browsing/safe_browsing_service.h"
 #endif
@@ -83,6 +84,8 @@ void Profile::RegisterProfilePrefs(user_prefs::PrefRegistrySyncable* registry) {
       prefs::kSessionExitType,
       std::string(),
       user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF);
+  // TODO(shess): This seems wrongish.  Perhaps the API should be in the
+  // service, with the #if OS_ANDROID inside that implementation.
 #if defined(OS_ANDROID) && defined(FULL_SAFE_BROWSING)
   // During Finch trail, safe browsing should be turned off
   // by default, and not sync'ed with desktop.