[WIP] Use Optional<SkColor> instead of int64 for colors in Manifest.

content/browser/manifest/manifest_manager_host.cc

 // Copyright 2014 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.
 
 #include "content/browser/manifest/manifest_manager_host.h"
 
 #include <stdint.h>
 
 #include "base/stl_util.h"
 #include "content/common/manifest_manager_messages.h"
@@ skipping 123 matching lines @@
             0, Manifest::kMaxIPCStringLength),
         manifest.gcm_sender_id.is_null());
   for (auto& related_application : manifest.related_applications) {
     if (!related_application.url.is_valid())
       related_application.url = GURL();
     related_application.id =
         base::NullableString16(related_application.id.string().substr(
                                    0, Manifest::kMaxIPCStringLength),
                                related_application.id.is_null());
   }
-  // theme_color and background_color are 32 bit unsigned integers with 64 bit
-  // integers simply being used to encode the occurence of an error. Therefore,
-  // any value outside the range of a 32 bit integer is invalid.
-  if (manifest.theme_color < std::numeric_limits<int32_t>::min() ||
-      manifest.theme_color > std::numeric_limits<int32_t>::max())
-    manifest.theme_color = Manifest::kInvalidOrMissingColor;
-  if (manifest.background_color < std::numeric_limits<int32_t>::min() ||
-      manifest.background_color > std::numeric_limits<int32_t>::max())
-    manifest.background_color = Manifest::kInvalidOrMissingColor;
 
   callback->Run(manifest_url, manifest);
   callbacks->Remove(request_id);
   if (callbacks->IsEmpty()) {
     delete callbacks;
     pending_get_callbacks_.erase(render_frame_host);
   }
 }
 
 } // namespace content