webapps: utilize manifest theme colors

chrome/browser/android/shortcut_data_fetcher.cc

 // Copyright 2015 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 "chrome/browser/android/shortcut_data_fetcher.h"
 
 #include "base/basictypes.h"
 #include "base/location.h"
 #include "base/strings/string16.h"
 #include "base/task/cancelable_task_tracker.h"
@@ skipping 55 matching lines @@
       web_app_info.description.substr(0, chrome::kMaxMetaTagAttributeLength);
 
   shortcut_info_.title = web_app_info.title.empty() ? web_contents()->GetTitle()
                                                     : web_app_info.title;
 
   if (web_app_info.mobile_capable == WebApplicationInfo::MOBILE_CAPABLE ||
       web_app_info.mobile_capable == WebApplicationInfo::MOBILE_CAPABLE_APPLE) {
     shortcut_info_.display = content::Manifest::DISPLAY_MODE_STANDALONE;
   }
 
+  // Set the theme color to undefined
+  shortcut_info_.theme_color = content::Manifest::kInvalidOrMissingThemeColor;

[mlamouri (slow - plz ping), 2015/07/20 13:03:30]

It doesn't sound right to do that here. Shouldn't you do that in ShortcutInfo's
constructor?

[Lalit Maganti, 2015/07/20 13:15:19]

Moved.

+
   // Record what type of shortcut was added by the user.
   switch (web_app_info.mobile_capable) {
     case WebApplicationInfo::MOBILE_CAPABLE:
       content::RecordAction(
           base::UserMetricsAction("webapps.AddShortcut.AppShortcut"));
       break;
     case WebApplicationInfo::MOBILE_CAPABLE_APPLE:
       content::RecordAction(
           base::UserMetricsAction("webapps.AddShortcut.AppShortcutApple"));
       break;
@@ skipping 160 matching lines @@
 void ShortcutDataFetcher::NotifyObserver(const SkBitmap& bitmap) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
   if (!web_contents() || !weak_observer_ || is_icon_saved_)
     return;
 
   is_icon_saved_ = true;
   shortcut_icon_ = bitmap;
   is_ready_ = true;
   weak_observer_->OnDataAvailable(shortcut_info_, shortcut_icon_);
 }