Support V1 app

athena/content/web_activity.cc

Index: athena/content/web_activity.cc
diff --git a/athena/content/web_activity.cc b/athena/content/web_activity.cc
index 5223944177e637edfd3fe5340c32a92bbe0b01e4..4dca1c3aac557e2abf4ee8b16ba96bbc9de8ca40 100644
--- a/athena/content/web_activity.cc
+++ b/athena/content/web_activity.cc
@@ -222,8 +222,8 @@ class AthenaWebView : public views::WebView {
       case NEW_POPUP:
       case NEW_WINDOW: {
         ActivityManager::Get()->AddActivity(
-            ActivityFactory::Get()->CreateWebActivity(browser_context(),
-                                                      params.url));
+            ActivityFactory::Get()->CreateWebActivity(
+                browser_context(), base::string16(), params.url));
         break;
       }
       default:
@@ -325,8 +325,10 @@ class AthenaWebView : public views::WebView {
 };
 
 WebActivity::WebActivity(content::BrowserContext* browser_context,
+                         const base::string16& title,
                          const GURL& url)
     : browser_context_(browser_context),
+      title_(title),
       url_(url),
       web_view_(NULL),
       title_color_(kDefaultTitleColor),
@@ -424,6 +426,9 @@ SkColor WebActivity::GetRepresentativeColor() const {
 }
 
 base::string16 WebActivity::GetTitle() const {
+  if (!title_.empty())
+    return title_;
+  // TODO(oshima): Use title set by the web contents.

[sadrul, 2014/09/05 15:48:26]

We used to show the page-title, but switched to showing the domain because of
feedback from UX.

[oshima, 2014/09/05 16:45:27]

Thanks. We probably should confirm if this is still true for V1/hosted apps.

   return web_view_ ? base::UTF8ToUTF16(
                          web_view_->GetWebContents()->GetVisibleURL().host())
                    : base::string16();