Shrink APIPermissionInfo from 32b to 24b with better packing

ExtensionsClient::DoInitialize is still reported as ~2% of renderer
startup time according to stack sampled metrics. This patch reduces the
size of APIPermissionInfo to reduce malloc.

Bug: 847237
Change-Id: Ib7d4a7156734ff84906925fa38174ece58925844
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1807717
Auto-Submit: Philip Rogers <pdr@chromium.org>
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Commit-Queue: Philip Rogers <pdr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697264}

extensions/common/permissions/api_permission.cc

@@ -104,11 +104,10 @@ const char* APIPermission::name() const {
 //
 
 APIPermissionInfo::APIPermissionInfo(const APIPermissionInfo::InitInfo& info)
-    : id_(info.id),
-      name_(info.name),
+    : name_(info.name),
+      id_(info.id),
       flags_(info.flags),
-      api_permission_constructor_(info.constructor) {
-}
+      api_permission_constructor_(info.constructor) {}
 
 APIPermissionInfo::~APIPermissionInfo() { }
 

extensions/common/permissions/api_permission.h

@@ -446,8 +446,8 @@ class APIPermissionInfo {
 
   explicit APIPermissionInfo(const InitInfo& info);
 
-  const APIPermission::ID id_;
   const char* const name_;
+  const APIPermission::ID id_;
   const int flags_;
   const APIPermissionConstructor api_permission_constructor_;
 };