Class/struct layout optimization for blink Resource related classes

third_party/WebKit/Source/platform/loader/fetch/ResourceLoaderOptions.h

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 21 matching lines @@
 #define ResourceLoaderOptions_h
 
 #include "platform/CrossThreadCopier.h"
 #include "platform/loader/fetch/FetchInitiatorInfo.h"
 #include "platform/loader/fetch/IntegrityMetadata.h"
 #include "platform/weborigin/SecurityOrigin.h"
 #include "platform/wtf/Allocator.h"
 
 namespace blink {
 
-enum DataBufferingPolicy { kBufferData, kDoNotBufferData };
+enum DataBufferingPolicy : uint8_t { kBufferData, kDoNotBufferData };
 
-enum ContentSecurityPolicyDisposition {
+enum ContentSecurityPolicyDisposition : uint8_t {
   kCheckContentSecurityPolicy,
   kDoNotCheckContentSecurityPolicy
 };
 
-enum RequestInitiatorContext {
+enum RequestInitiatorContext : uint8_t {
   kDocumentContext,
   kWorkerContext,
 };
 
-enum StoredCredentials {
+enum StoredCredentials : uint8_t {
   kAllowStoredCredentials,
   kDoNotAllowStoredCredentials
 };
 
 // APIs like XMLHttpRequest and EventSource let the user decide whether to send
 // credentials, but they're always sent for same-origin requests. Additional
 // information is needed to handle cross-origin redirects correctly.
-enum CredentialRequest {
+enum CredentialRequest : uint8_t {
   kClientRequestedCredentials,
   kClientDidNotRequestCredentials
 };
 
-enum SynchronousPolicy { kRequestSynchronously, kRequestAsynchronously };
+enum SynchronousPolicy : uint8_t {
+  kRequestSynchronously,
+  kRequestAsynchronously
+};
 
 // A resource fetch can be marked as being CORS enabled. The loader must perform
 // an access check upon seeing the response.
-enum CORSEnabled { kNotCORSEnabled, kIsCORSEnabled };
+enum CORSEnabled : uint8_t { kNotCORSEnabled, kIsCORSEnabled };
 
 // Was the request generated from a "parser-inserted" element?
 // https://html.spec.whatwg.org/multipage/scripting.html#parser-inserted
-enum ParserDisposition { kParserInserted, kNotParserInserted };
+enum ParserDisposition : uint8_t { kParserInserted, kNotParserInserted };
 
-enum CacheAwareLoadingEnabled {
+enum CacheAwareLoadingEnabled : uint8_t {
   kNotCacheAwareLoadingEnabled,
   kIsCacheAwareLoadingEnabled
 };
 
 struct ResourceLoaderOptions {
   USING_FAST_MALLOC(ResourceLoaderOptions);
 
  public:
   ResourceLoaderOptions(StoredCredentials allow_credentials,
                         CredentialRequest credentials_requested)
@@ skipping 17 matching lines @@
     // FIXME: check contentSecurityPolicyOption.
     // initiatorInfo is purely informational and should be benign for re-use.
     // requestInitiatorContext is benign (indicates document vs. worker)
     if (synchronous_policy != other.synchronous_policy)
       return false;
     return cors_enabled == other.cors_enabled;
     // securityOrigin has more complicated checks which callers are responsible
     // for.
   }
 
+  FetchInitiatorInfo initiator_info;
+
   // When adding members, CrossThreadResourceLoaderOptionsData should be
   // updated.
   DataBufferingPolicy data_buffering_policy;
 
   // Whether HTTP credentials and cookies are sent with the request.
   StoredCredentials allow_credentials;
 
   // Whether the client (e.g. XHR) wanted credentials in the first place.
   CredentialRequest credentials_requested;
 
   ContentSecurityPolicyDisposition content_security_policy_option;
-  FetchInitiatorInfo initiator_info;
   RequestInitiatorContext request_initiator_context;
   SynchronousPolicy synchronous_policy;
 
   // If the resource is loaded out-of-origin, whether or not to use CORS.
   CORSEnabled cors_enabled;
 
   RefPtr<SecurityOrigin> security_origin;
   String content_security_policy_nonce;
   IntegrityMetadataSet integrity_metadata;
   ParserDisposition parser_disposition;
@@ skipping 56 matching lines @@
 struct CrossThreadCopier<ResourceLoaderOptions> {
   using Type = CrossThreadResourceLoaderOptionsData;
   static Type Copy(const ResourceLoaderOptions& options) {
     return CrossThreadResourceLoaderOptionsData(options);
   }
 };
 
 }  // namespace blink
 
 #endif  // ResourceLoaderOptions_h