Use webview in web activity

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 12069d0bcd615b6c1c8283f0f4ab9a6cebe28126..fa7b6da24114278939c96cc0898e75c4d60b4c7a 100644
--- a/athena/content/web_activity.cc
+++ b/athena/content/web_activity.cc
@@ -7,11 +7,13 @@
 #include "athena/activity/public/activity_manager.h"
 #include "base/strings/utf_string_conversions.h"
 #include "content/public/browser/web_contents.h"
+#include "ui/views/controls/webview/webview.h"
 
 namespace athena {
 
-WebActivity::WebActivity(content::WebContents* contents)
-    : content::WebContentsObserver(contents) {
+WebActivity::WebActivity(content::BrowserContext* browser_context,
+                         const GURL& url)
+    : browser_context_(browser_context), url_(url), web_view_(NULL) {
 }
 
 WebActivity::~WebActivity() {
@@ -27,11 +29,16 @@ SkColor WebActivity::GetRepresentativeColor() {
 }
 
 std::string WebActivity::GetTitle() {
-  return base::UTF16ToUTF8(web_contents()->GetTitle());
+  return base::UTF16ToUTF8(web_view_->GetWebContents()->GetTitle());

[Jun Mukai, 2014/06/06 20:17:29]

Don't we need to check web_view_?  It might be called before GetContentsView().
Or just static_cast<views::WebView*>(GetContentsView())->...

[oshima, 2014/06/06 20:55:51]

This never called before GetContentsView(), and the view created by
GetContentsView() is supposed to be added to a parent. So if someone called
GetTitle, then delete a web activity, it will cause a memory leak.

Even if we ever changed the order, we can catch such change easily, so I wont
worry about.

[Jun Mukai, 2014/06/06 20:57:03]

fair enough. thanks.

 }
 
-aura::Window* WebActivity::GetNativeWindow() {
-  return web_contents()->GetNativeView();
+views::View* WebActivity::GetContentsView() {
+  if (!web_view_) {
+    web_view_ = new views::WebView(browser_context_);
+    web_view_->LoadInitialURL(url_);
+    Observe(web_view_->GetWebContents());
+  }
+  return web_view_;
 }
 
 void WebActivity::TitleWasSet(content::NavigationEntry* entry,