Making the extension permissions dialog scrollable, when needed (adding expandable sections for thi…

chrome/common/extensions/permissions/permission_message.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_COMMON_EXTENSIONS_PERMISSIONS_PERMISSION_MESSAGE_H_
 #define CHROME_COMMON_EXTENSIONS_PERMISSIONS_PERMISSION_MESSAGE_H_
 
 #include <set>
 #include <string>
 #include <vector>
@@ skipping 57 matching lines @@
     kEnumBoundary,
   };
 
   // Creates the corresponding permission message for a list of hosts. This is
   // simply a convenience method around the constructor, since the messages
   // change depending on what hosts are present.
   static PermissionMessage CreateFromHostList(
       const std::set<std::string>& hosts);
 
   // Creates the corresponding permission message.
-  PermissionMessage(ID id, const string16& message);
+  PermissionMessage(ID id, const string16& message, const string16& details);

[Yoyo Zhou, 2013/07/11 17:39:48]

nit: you could add a form of the constructor with no details argument.

   ~PermissionMessage();
 
   // Gets the id of the permission message, which can be used in UMA
   // histograms.
   ID id() const { return id_; }
 
   // Gets a localized message describing this permission. Please note that
   // the message will be empty for message types TYPE_NONE and TYPE_UNKNOWN.
   const string16& message() const { return message_; }
 
+  // Gets a localized message describing the details for this permission. Please
+  // note that the message will be empty for message types TYPE_NONE and
+  // TYPE_UNKNOWN.
+  const string16& details() const { return details_; }
+
   // Comparator to work with std::set.
   bool operator<(const PermissionMessage& that) const {
     return id_ < that.id_;
   }
 
  private:
   ID id_;
   string16 message_;
+  string16 details_;
 };
 
 typedef std::vector<PermissionMessage> PermissionMessages;
 
 }  // namespace extensions
 
 #endif  // CHROME_COMMON_EXTENSIONS_PERMISSIONS_PERMISSION_MESSAGE_H_