[Extensions] Add logic for when to show the toolbar redesign bubble.

chrome/common/pref_names.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/common/pref_names.h"
 
 #include "base/basictypes.h"
 #include "chrome/common/pref_font_webkit_names.h"
 
 namespace prefs {
@@ skipping 1200 matching lines @@
 // Whether close proximity between the remote and the local device is required
 // in order to use Easy Unlock.
 const char kEasyUnlockProximityRequired[] = "easy_unlock.proximity_required";
 
 // Whether to show the Easy Unlock first run tutorial.
 const char kEasyUnlockShowTutorial[] = "easy_unlock.show_tutorial";
 
 // A cache of zero suggest results using JSON serialized into a string.
 const char kZeroSuggestCachedResults[] = "zerosuggest.cachedresults";
 
+#if defined(ENABLE_EXTENSIONS) && !defined(OS_ANDROID) && !defined(OS_IOS)
 // These device IDs are used by the copresence component, to uniquely identify
 // this device to the server. For privacy, authenticated and unauthenticated
 // calls are made using different device IDs.
-#if defined(ENABLE_EXTENSIONS) && !defined(OS_ANDROID) && !defined(OS_IOS)
 const char kCopresenceAuthenticatedDeviceId[] =
     "apps.copresence.auth_device_id";
 const char kCopresenceAnonymousDeviceId[] = "apps.copresence.unauth_device_id";
+
+// Used to indicate whether or not the toolbar redesign bubble has been shown
+// and acknowledged, and the last time the bubble was shown.
+const char kToolbarIconSurfacingBubbleAcknowledged[] =
+    "toolbar_icon_surfacing_bubble_acknowledged";
+extern const char kToolbarIconSurfacingBubbleLastShowTime[] =

[battre, 2015/03/19 09:28:45]

extern?

[Devlin, 2015/03/19 16:28:09]

Whoops! Copy-paste error.  Looks like I wasn't the only one - fixed the others,
too :)

+    "toolbar_icon_surfacing_bubble_show_time";
 #endif
 
 // Whether WebRTC should bind to individual NICs to explore all possible routing
 // options. Default is true.
 #if defined(ENABLE_WEBRTC)
 const char kWebRTCMultipleRoutesEnabled[] = "webrtc.multiple_routes_enabled";
 #endif
 
 // *************** LOCAL STATE ***************
 // These are attached to the machine/installation
@@ skipping 1044 matching lines @@
 // (name and a list of clients that registered the whitelist).
 const char kRegisteredSupervisedUserWhitelists[] =
     "supervised_users.whitelists";
 
 #if defined(ENABLE_EXTENSIONS)
 // Policy that indicates how to handle animated images.
 const char kAnimationPolicy[] = "settings.a11y.animation_policy";
 #endif
 
 }  // namespace prefs