Integrate SMS service with Desktop iOS promotion

chrome/browser/ui/desktop_ios_promotion/desktop_ios_promotion_util.h

 // Copyright 2017 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.
 
 #ifndef CHROME_BROWSER_UI_DESKTOP_IOS_PROMOTION_DESKTOP_IOS_PROMOTION_UTIL_H_
 #define CHROME_BROWSER_UI_DESKTOP_IOS_PROMOTION_DESKTOP_IOS_PROMOTION_UTIL_H_
 
 #include "base/macros.h"
 #include "chrome/common/pref_names.h"
 #include "ui/base/l10n/l10n_util.h"
 
 namespace syncer {
 class SyncService;
 }
 
 namespace user_prefs {
 class PrefRegistrySyncable;
 }
 
 class PrefRegistrySimple;
 class PrefService;
 
 namespace desktop_ios_promotion {
 
 // Represents the reason that the Desktop to iOS promotion was dismissed for.
+// These values are written to logs. New values can be added, but existing
+// values must never be reordered or deleted and reused.

[Mark P, 2017/02/16 23:18:57]

Can you confirm that nothing caused these values to be written to logs or stored
anywhere before this changelist?

[mrefaat, 2017/02/17 00:20:59]

Yes, these values have not been written to logs before - the feature is disabled
and loging is being added just now.

 enum class PromotionDismissalReason {
   FOCUS_LOST = 0,
-  TAB_SWITCHED,
   NO_THANKS,

[Mark P, 2017/02/16 23:18:57]

Following histogram enum best practices, please explicitly set = 1, = 2, etc.

(
https://chromium.googlesource.com/chromium/src.git/+/HEAD/tools/metrics/histo...
)

[mrefaat, 2017/02/17 00:20:59]

Done.

   CLOSE_BUTTON,
   SEND_SMS,
-  LEARN_MORE_CLICKED,
 };
 
+enum class EntryPointLocalPrefType { IMPRESSIONS = 0, DISMISSED = 1 };
+
 // The place where the promotion appeared.
+// These values are written to logs. New values can be added, but existing
+// values must never be reordered or deleted and reused.
 enum class PromotionEntryPoint {
-  SAVE_PASSWORD_BUBBLE = 0,
-  BOOKMARKS_BUBBLE,
-  BOOKMARKS_FOOTNOTE,
-  HISTORY_PAGE
+  SAVE_PASSWORD_BUBBLE = 1,
+  BOOKMARKS_BUBBLE = 2,
+  BOOKMARKS_FOOTNOTE = 3,
+  HISTORY_PAGE = 4,
+  // Used for histograms logging.
+  ENTRY_POINT_MAX_VALUE = 5
 };
 
+// Used with PromotionEntryPoint values as an index on kEntryPointLocalPrefs.
+constexpr int kPromotionEntrypointIndexOffset = -1;

[Mark P, 2017/02/16 23:18:57]

Rather than do this hack, why don't you just put an empty entry in
kEntryPointLocalPrefs?  Then you don't have to remember at *every single use* to
apply the offset.

[mrefaat, 2017/02/17 00:20:59]

I was going to do that but was afraid of confusion or mistakenly using the first
entry ( as no exception will happen so it may skip testing).
But i will change it now.

+
 // Entry points local prefs, each entry point has a preference for impressions
 // count and a preference for whether user dismissed it or not.
 // Do not change the order of this array, as it's indexed using
 // desktop_ios_promotion::PromotionEntryPoint.
 const char* const kEntryPointLocalPrefs[4][2] = {
     {prefs::kNumberSavePasswordsBubbleIOSPromoShown,
      prefs::kSavePasswordsBubbleIOSPromoDismissed},
     {prefs::kNumberBookmarksBubbleIOSPromoShown,
      prefs::kBookmarksBubbleIOSPromoDismissed},
     {prefs::kNumberBookmarksFootNoteIOSPromoShown,
      prefs::kBookmarksFootNoteIOSPromoDismissed},
     {prefs::kNumberHistoryPageIOSPromoShown,
      prefs::kHistoryPageIOSPromoDismissed}};
 
 bool IsEligibleForIOSPromotion(PrefService* prefs,
                                const syncer::SyncService* sync_service,
-                               desktop_ios_promotion::PromotionEntryPoint);
+                               PromotionEntryPoint entry_point);
 
 // Returns the Bubble text based on the promotion entry point.
-base::string16 GetPromoBubbleText(
-    desktop_ios_promotion::PromotionEntryPoint entry_point);
+base::string16 GetPromoBubbleText(PromotionEntryPoint entry_point);
 
 // Register all Priority Sync preferences.
 void RegisterProfilePrefs(user_prefs::PrefRegistrySyncable* registry);
 
 // Register all local only Preferences.
 void RegisterLocalPrefs(PrefRegistrySimple* registry);
 
+// Log promotion dismissal reason for entry points.
+void LogDismissalReason(PromotionDismissalReason reason,
+                        PromotionEntryPoint entry_point);
+
 }  // namespace desktop_ios_promotion
 
 #endif  // CHROME_BROWSER_UI_DESKTOP_IOS_PROMOTION_DESKTOP_IOS_PROMOTION_UTIL_H_