Permission Report: Add new fields.

chrome/common/safe_browsing/permission_report.proto

 // Copyright 2016 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.
 
 // Protocol buffer for permission reports sent to the Safe Browsing client-side
 // detection (CSD) frontends. This should stay in sync with the Safe Browsing 
 // server-side protocol buffer.
 
 syntax = "proto2";
 
@@ skipping 18 matching lines @@
 
   // The action the user took. Required.
   optional Action action = 5;
 
   // The UI used to complete the action.
   optional SourceUI source_ui = 6;
 
   // The relevant field trials enabled for this report.
   repeated FieldTrial field_trials = 7;
 
+  // The number of dismissal on a prompt for this origin prior to this report.

[dominickn, 2016/08/15 04:37:11]

Nit: dismissal -> dismissals

Nit: "for this origin" -> "for this permission for this origin"

Same for num_prior_ignores.

[kcarattini, 2016/08/15 07:02:25]

Done.

+  optional int32 num_prior_dismissals = 8;
+
+  // The number of ignores of a prompt for this origin prior to this report.
+  optional int32 num_prior_ignores = 9;
+
+  // The persistance decision on a prompt. Only some experimental prompts will

[dominickn, 2016/08/15 04:37:11]

Nit: "persistance" -> "persistence"

[kcarattini, 2016/08/15 07:02:25]

Done.

+  // have this field set.
+  optional PersistDecision persisted = 10;
+
   // Platform
   enum PlatformType {
     PLATFORM_TYPE_UNSPECIFIED = 0;
     DESKTOP_PLATFORM = 1;
     ANDROID_PLATFORM = 2;
   }
 
   // Whether the action occurred after a user gesture.
   enum GestureType {
     GESTURE_TYPE_UNSPECIFIED = 0;
     GESTURE = 1;
     NO_GESTURE = 2;
   }
 
-  // User Permission Actions. This enum is intentionally different with 
+  // User Permission Actions. This enum is intentionally different from
   // the one in src/chrome/browser/permissions/permission_uma_util.h
   enum Action {
     ACTION_UNSPECIFIED = 0;
     GRANTED = 1;
     DENIED = 2;
     DISMISSED = 3;
     IGNORED = 4;
     REVOKED = 5;
   }
 
@@ skipping 14 matching lines @@
     PUSH_MESSAGING = 2;
     NOTIFICATIONS = 3;
     GEOLOCATION = 4;
     PROTECTED_MEDIA_IDENTIFIER = 5;
     MIDI = 6;
     DURABLE_STORAGE = 7;
     AUDIO_CAPTURE = 8;
     VIDEO_CAPTURE = 9;
   }
 
+  // Whether the decision on a prompt was persisted.
+  enum PersistDecision {
+    PERSIST_DECISION_UNSPECIFIED = 0;
+    PERSISTED = 1;
+    NOT_PERSISTED = 2;
+  }
+
   // Description of a field trial or experiment that the user is currently
   // enrolled in. All metrics reported in this upload can potentially be
   // influenced by the field trial.
   message FieldTrial {
     // The name of the field trial, as a 32-bit identifier. Currently, the
     // identifier is a hash of the field trial's name.
     optional fixed32 name_id = 1;
 
     // The user's group within the field trial, as a 32-bit identifier.
     // Currently, the identifier is a hash of the group's name.
     optional fixed32 group_id = 2;
   }
 }