Refactor: unsigned short/int -> uint16_t in third_party/blink/renderer/core/loader/appcache

- unsigned short/int -> uint16_t.
- No logic changes.
- Reference: https://google.github.io/styleguide/cppguide.html#Integer_Types

Bug: 929986
Change-Id: I072bdca3916a24def0ceda5915b4b45afaf1dfad
Reviewed-on: https://chromium-review.googlesource.com/c/1487760Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Miyoung Shin <myid.shin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#635431}

third_party/blink/renderer/core/events/application_cache_error_event.cc

@@ -44,7 +44,7 @@ static const String& ErrorReasonToString(mojom::AppCacheErrorReason reason) {
 ApplicationCacheErrorEvent::ApplicationCacheErrorEvent(
     mojom::AppCacheErrorReason reason,
     const String& url,
-    int status,
+    uint16_t status,
     const String& message)
     : Event(event_type_names::kError, Bubbles::kNo, Cancelable::kNo),
       reason_(ErrorReasonToString(reason)),

third_party/blink/renderer/core/events/application_cache_error_event.h

@@ -20,7 +20,7 @@ class ApplicationCacheErrorEvent final : public Event {
  public:
   ApplicationCacheErrorEvent(mojom::AppCacheErrorReason,
                              const String& url,
-                             int status,
+                             uint16_t status,
                              const String& message);
   ApplicationCacheErrorEvent(const AtomicString& event_type,
                              const ApplicationCacheErrorEventInit* initializer);
@@ -28,7 +28,7 @@ class ApplicationCacheErrorEvent final : public Event {
 
   static ApplicationCacheErrorEvent* Create(mojom::AppCacheErrorReason reason,
                                             const String& url,
-                                            int status,
+                                            uint16_t status,
                                             const String& message) {
     return MakeGarbageCollected<ApplicationCacheErrorEvent>(reason, url, status,
                                                             message);
@@ -43,7 +43,7 @@ class ApplicationCacheErrorEvent final : public Event {
 
   const String& reason() const { return reason_; }
   const String& url() const { return url_; }
-  int status() const { return status_; }
+  uint16_t status() const { return status_; }
   const String& message() const { return message_; }
 
   const AtomicString& InterfaceName() const override {
@@ -55,7 +55,7 @@ class ApplicationCacheErrorEvent final : public Event {
  private:
   String reason_;
   String url_;
-  int status_;
+  uint16_t status_;
   String message_;
 };
 

third_party/blink/renderer/core/loader/appcache/application_cache.cc

@@ -56,34 +56,34 @@ ApplicationCacheHost* ApplicationCache::GetApplicationCacheHost() const {
   return GetFrame()->Loader().GetDocumentLoader()->GetApplicationCacheHost();
 }
 
-unsigned short ApplicationCache::status() const {
+uint16_t ApplicationCache::status() const {
   // Application Cache status numeric values are specified in the HTML5 spec.
-  static_assert(static_cast<unsigned short>(
+  static_assert(static_cast<uint16_t>(
                     mojom::AppCacheStatus::APPCACHE_STATUS_UNCACHED) == 0,
                 "");
-  static_assert(static_cast<unsigned short>(
-                    mojom::AppCacheStatus::APPCACHE_STATUS_IDLE) == 1,
-                "");
-  static_assert(static_cast<unsigned short>(
+  static_assert(
+      static_cast<uint16_t>(mojom::AppCacheStatus::APPCACHE_STATUS_IDLE) == 1,
+      "");
+  static_assert(static_cast<uint16_t>(
                     mojom::AppCacheStatus::APPCACHE_STATUS_CHECKING) == 2,
                 "");
-  static_assert(static_cast<unsigned short>(
+  static_assert(static_cast<uint16_t>(
                     mojom::AppCacheStatus::APPCACHE_STATUS_DOWNLOADING) == 3,
                 "");
-  static_assert(static_cast<unsigned short>(
+  static_assert(static_cast<uint16_t>(
                     mojom::AppCacheStatus::APPCACHE_STATUS_UPDATE_READY) == 4,
                 "");
-  static_assert(static_cast<unsigned short>(
+  static_assert(static_cast<uint16_t>(
                     mojom::AppCacheStatus::APPCACHE_STATUS_OBSOLETE) == 5,
                 "");
 
   RecordAPIUseType();
   ApplicationCacheHost* cache_host = GetApplicationCacheHost();
   if (!cache_host) {
-    return static_cast<unsigned short>(
+    return static_cast<uint16_t>(
         mojom::AppCacheStatus::APPCACHE_STATUS_UNCACHED);
   }
-  return static_cast<unsigned short>(cache_host->GetStatus());
+  return static_cast<uint16_t>(cache_host->GetStatus());
 }
 
 void ApplicationCache::update(ExceptionState& exception_state) {

third_party/blink/renderer/core/loader/appcache/application_cache.h

@@ -51,7 +51,7 @@ class ApplicationCache final : public EventTargetWithInlineData,
   explicit ApplicationCache(LocalFrame*);
   ~ApplicationCache() override = default;
 
-  unsigned short status() const;
+  uint16_t status() const;
   void update(ExceptionState&);
   void swapCache(ExceptionState&);
   void abort();