Experimental app list: Custom launcher pages have app APIs and style.

chrome/browser/ui/app_list/app_list_view_delegate.cc

 // Copyright (c) 2012 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/ui/app_list/app_list_view_delegate.h"
 
 #include <vector>
 
+#include "apps/custom_launcher_page_contents.h"
 #include "base/callback.h"
 #include "base/command_line.h"
 #include "base/files/file_path.h"
 #include "base/metrics/user_metrics.h"
 #include "base/stl_util.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/profiles/profile_info_cache.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/search/hotword_service.h"
@@ skipping 136 matching lines @@
   // arbitrary extensions may be able to specify their own custom pages.
   base::CommandLine* command_line = base::CommandLine::ForCurrentProcess();
   if (app_list::switches::IsExperimentalAppListEnabled() &&
       command_line->HasSwitch(switches::kCustomLauncherPage)) {
     GURL custom_launcher_page_url(
         command_line->GetSwitchValueASCII(switches::kCustomLauncherPage));
     if (!custom_launcher_page_url.SchemeIs(extensions::kExtensionScheme)) {
       LOG(ERROR) << "Invalid custom launcher page URL: "
                  << custom_launcher_page_url.possibly_invalid_spec();
     } else {
-      content::WebContents::CreateParams params(profile_);
-      custom_page_web_contents_.reset(content::WebContents::Create(params));
-      custom_page_web_contents_->GetController().LoadURL(
-          custom_launcher_page_url,
-          content::Referrer(),
-          content::PAGE_TRANSITION_AUTO_TOPLEVEL,
-          std::string());
+      custom_page_contents_.reset(new apps::CustomLauncherPageContents());
+      custom_page_contents_->Initialize(profile, custom_launcher_page_url);

[tapted, 2014/07/10 04:29:34]

Should this be in OnProfileChanged() to properly handle profile switching, and
deleting a profile from chrome://settings?

Alternatively, I think you could you just move this into
AppListViewDelegate::CreateCustomPageWebView, and reset it when that's called
via AppListMainView->ModelChanged() which will create a new ContentsView

[Matt Giuca, 2014/07/10 07:37:03]

Good catch! This is actually a pre-existing bug. So I have filed it and will
address it in a follow-up:
https://code.google.com/p/chromium/issues/detail?id=392763

+      custom_page_contents_->LoadContents();
     }
   }
 }
 
 AppListViewDelegate::~AppListViewDelegate() {
   app_list::StartPageService* service =
       app_list::StartPageService::Get(profile_);
   if (service)
     service->RemoveObserver(this);
   g_browser_process->
@@ skipping 292 matching lines @@
 
   views::WebView* web_view = new views::WebView(
       web_contents->GetBrowserContext());
   web_view->SetPreferredSize(size);
   web_view->SetWebContents(web_contents);
   return web_view;
 }
 
 views::View* AppListViewDelegate::CreateCustomPageWebView(
     const gfx::Size& size) {
-  if (!custom_page_web_contents_)
+  if (!custom_page_contents_)
     return NULL;
 
-  views::WebView* web_view = new views::WebView(
-      custom_page_web_contents_->GetBrowserContext());
+  content::WebContents* web_contents = custom_page_contents_->GetWebContents();
+  views::WebView* web_view =
+      new views::WebView(web_contents->GetBrowserContext());

[tapted, 2014/07/10 04:29:34]

is web_contents->GetBrowserContext() equivalent to this->profile_? You could use
that and 
  views::WebView* web_view = new views::WebView(profile_);
  web_view->SetWebContents(custom_page_contents_->web_contents());

[Matt Giuca, 2014/07/10 07:37:03]

Hmm, yeah they SHOULD be equivalent (but actually is not, due to the
aforementioned bug, Issue 392763).

I think the right thing to do is still to use GetBrowserContext() since you are
creating a WebView for a WebContents and the main source of truth about which
profile is the GetBrowserContext().

But it is good to add a DCHECK to ensure these are the same. I've done that in
Chris's code just above (which I copied this from). However, I can't add the
DCHECK to my code right now; added a TODO.

   web_view->SetPreferredSize(size);
-  web_view->SetWebContents(custom_page_web_contents_.get());
+  web_view->SetWebContents(web_contents);
   return web_view;
 }
 #endif
 
 bool AppListViewDelegate::IsSpeechRecognitionEnabled() {
   app_list::StartPageService* service =
       app_list::StartPageService::Get(profile_);
   return service && service->GetSpeechRecognitionContents();
 }
 
@@ skipping 32 matching lines @@
 
 void AppListViewDelegate::AddObserver(
     app_list::AppListViewDelegateObserver* observer) {
   observers_.AddObserver(observer);
 }
 
 void AppListViewDelegate::RemoveObserver(
     app_list::AppListViewDelegateObserver* observer) {
   observers_.RemoveObserver(observer);
 }