Remove app_shell chrome.shell API and ShellAppWindow

The chrome.app.window API has been moved to src/extensions, so app_shell
no longer needs its custom chrome.shell API to open windows.

* Remove the API implementation -- mostly a manual revert of https://codereview.chromium.org/254473011/
* Remove app_shell_resources.grd since there are no more app_shell resources
* Remove athena's support for ShellAppWindow
* Fix example apps to remove the chrome.shell workaround.

No presubmit because it has a false-positive warning about the enum rename in extension_function_histogram_value.h

No try because no presubmit doesn't seem to work by itself.

BUG=413164
TEST=extensions_unittests, app_shell_unittests, app_shell_browsertests
TBR=isherman@chromium.org for deprecating an extension function histogram
NOPRESUBMIT=true
NOTRY=true

Review URL: https://codereview.chromium.org/571643003

Cr-Commit-Position: refs/heads/master@{#294852}

4 files changed, 3 insertions, 23 deletions

diff --git a/athena/content/shell/DEPS b/athena/content/shell/DEPS
index 4bc63bc..3e1cb59 100644
--- a/athena/content/shell/DEPS
+++ b/athena/content/shell/DEPS
@@ -1,4 +1,3 @@
 include_rules = [
   "+extensions/browser/app_window",
-  "+extensions/shell/browser/shell_app_window.h",
 ]
diff --git a/athena/content/shell/content_activity_factory.cc b/athena/content/shell/content_activity_factory.cc
index edcc76f..76ee114 100644
--- a/athena/content/shell/content_activity_factory.cc
+++ b/athena/content/shell/content_activity_factory.cc
@@ -9,12 +9,6 @@
 namespace athena {
 
 Activity* ContentActivityFactory::CreateAppActivity(
-    extensions::ShellAppWindow* app_window,
-    const std::string& app_id) {
-  return new ShellAppActivity(app_window, app_id);
-}
-
-Activity* ContentActivityFactory::CreateAppActivity(
     extensions::AppWindow* app_window,
     views::WebView* web_view) {
   return new ShellAppActivity(app_window);
diff --git a/athena/content/shell/shell_app_activity.cc b/athena/content/shell/shell_app_activity.cc
index 50eb45b..cb008e0 100644
--- a/athena/content/shell/shell_app_activity.cc
+++ b/athena/content/shell/shell_app_activity.cc
@@ -7,23 +7,17 @@
 #include "content/public/browser/web_contents.h"
 #include "extensions/browser/app_window/app_window.h"
 #include "extensions/browser/app_window/native_app_window.h"
-#include "extensions/shell/browser/shell_app_window.h"
 #include "ui/views/controls/webview/webview.h"
 
 namespace athena {
 
 ShellAppActivity::ShellAppActivity(extensions::AppWindow* app_window)
     : AppActivity(app_window->extension_id()), app_window_(app_window) {
-}
-
-ShellAppActivity::ShellAppActivity(extensions::ShellAppWindow* app_window,
-                                   const std::string& app_id)
-    : AppActivity(app_id), app_window_(NULL), shell_app_window_(app_window) {
+  DCHECK(app_window_);
 }
 
 ShellAppActivity::~ShellAppActivity() {
-  if (app_window_)
-    app_window_->GetBaseWindow()->Close();  // Deletes |app_window_|.
+  app_window_->GetBaseWindow()->Close();  // Deletes |app_window_|.
 }
 
 views::Widget* ShellAppActivity::CreateWidget() {
@@ -31,9 +25,7 @@ views::Widget* ShellAppActivity::CreateWidget() {
 }
 
 views::WebView* ShellAppActivity::GetWebView() {
-  content::WebContents* web_contents =
-      app_window_ ? app_window_->web_contents() :
-      shell_app_window_->GetAssociatedWebContents();
+  content::WebContents* web_contents = app_window_->web_contents();
   views::WebView* web_view =
       new views::WebView(web_contents->GetBrowserContext());
   web_view->SetWebContents(web_contents);
diff --git a/athena/content/shell/shell_app_activity.h b/athena/content/shell/shell_app_activity.h
index ad3467b..a6b7dec 100644
--- a/athena/content/shell/shell_app_activity.h
+++ b/athena/content/shell/shell_app_activity.h
@@ -11,7 +11,6 @@
 
 namespace extensions {
 class AppWindow;
-class ShellAppWindow;
 }
 
 namespace athena {
@@ -19,9 +18,6 @@ namespace athena {
 class ShellAppActivity : public AppActivity {
  public:
   explicit ShellAppActivity(extensions::AppWindow* app_window);
-  // TODO(hashimoto) Remove this.
-  ShellAppActivity(extensions::ShellAppWindow* app_window,
-                   const std::string& app_id);
   virtual ~ShellAppActivity();
 
  private:
@@ -32,7 +28,6 @@ class ShellAppActivity : public AppActivity {
   virtual views::WebView* GetWebView() OVERRIDE;
 
   extensions::AppWindow* app_window_;
-  scoped_ptr<extensions::ShellAppWindow> shell_app_window_;
 
   DISALLOW_COPY_AND_ASSIGN(ShellAppActivity);
 };