Log mixed script shield events.

chrome/browser/content_settings/tab_specific_content_settings.h

 // 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.
 
 #ifndef CHROME_BROWSER_CONTENT_SETTINGS_TAB_SPECIFIC_CONTENT_SETTINGS_H_
 #define CHROME_BROWSER_CONTENT_SETTINGS_TAB_SPECIFIC_CONTENT_SETTINGS_H_
 
 #include <string>
 
 #include "base/basictypes.h"
@@ skipping 37 matching lines @@
   // Fields describing the current mic/camera state. If a page has attempted to
   // access a device, the XXX_ACCESSED bit will be set. If access was blocked,
   // XXX_BLOCKED will be set.
   typedef uint32_t MicrophoneCameraState;
   static const MicrophoneCameraState MICROPHONE_CAMERA_NOT_ACCESSED = 0;
   static const MicrophoneCameraState MICROPHONE_ACCESSED = 1 << 0;
   static const MicrophoneCameraState MICROPHONE_BLOCKED = 1 << 1;
   static const MicrophoneCameraState CAMERA_ACCESSED = 1 << 2;
   static const MicrophoneCameraState CAMERA_BLOCKED = 1 << 3;
 
+  // UMA statistics for the mixed content shield
+  enum MixedScriptAction {
+    MIXEDSCRIPT_DISPLAYED_SHIELD = 0,
+    MIXEDSCRIPT_DISPLAYED_BUBBLE,
+    MIXEDSCRIPT_CLICKED_ALLOW,
+    MIXEDSCRIPT_CLICKED_LEARN_MORE,
+    MIXEDSCRIPT_NUM_ACTIONS

[Ilya Sherman, 2014/10/16 22:20:35]

FYI, the typical style is, if the enum name is FooBar, to have the enum values
be named FOO_BAR_APPLE, FOO_BAR_BANANA, etc.  So, in this case, I'd expect the
enum value names to start with the prefix "MIXED_SCRIPT_ACTION_".

(In theory, C++11 enum classes could provide actually namespacing, so that we
wouldn't need this ugly naming convention.  IIRC, that feature is still under
discussion on chromium-dev.)

[lgarron, 2014/10/16 22:24:30]

Man, apparently I can't get nuttin' right, even if I consult the style guide.
:-P

I consolidated MIXEDSCRIPT into one word because that's how
CONTENT_TYPE_MIXEDSCRIPT does it, but I'll change it over now.

+  };
+
+  static void RecordMixedScriptAction(MixedScriptAction action);
+
   // Classes that want to be notified about site data events must implement
   // this abstract class and add themselves as observer to the
   // |TabSpecificContentSettings|.
   class SiteDataObserver {
    public:
     explicit SiteDataObserver(
         TabSpecificContentSettings* tab_specific_content_settings);
     virtual ~SiteDataObserver();
 
     // Called whenever site data is accessed.
@@ skipping 377 matching lines @@
   std::string media_stream_requested_audio_device_;
   std::string media_stream_requested_video_device_;
 
   // Observer to watch for content settings changed.
   ScopedObserver<HostContentSettingsMap, content_settings::Observer> observer_;
 
   DISALLOW_COPY_AND_ASSIGN(TabSpecificContentSettings);
 };
 
 #endif  // CHROME_BROWSER_CONTENT_SETTINGS_TAB_SPECIFIC_CONTENT_SETTINGS_H_