Moving app_view to extensions.

extensions/browser/guest_view/app_view/app_view_guest_delegate.h

Index: extensions/browser/guest_view/app_view/app_view_guest_delegate.h
diff --git a/extensions/browser/guest_view/app_view/app_view_guest_delegate.h b/extensions/browser/guest_view/app_view/app_view_guest_delegate.h
new file mode 100644
index 0000000000000000000000000000000000000000..1b34feb742fb5fc59cd23505cba53aa3f465bfa4
--- /dev/null
+++ b/extensions/browser/guest_view/app_view/app_view_guest_delegate.h
@@ -0,0 +1,34 @@
+// Copyright 2014 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.
+
+#ifndef EXTENSIONS_BROWSER_GUEST_VIEW_APP_VIEW_APP_VIEW_GUEST_DELEGATE_H_
+#define EXTENSIONS_BROWSER_GUEST_VIEW_APP_VIEW_APP_VIEW_GUEST_DELEGATE_H_
+
+#include "content/public/common/context_menu_params.h"
+
+namespace content {
+class BrowserContext;
+class WebContents;
+}
+
+namespace extensions {
+class Extension;
+
+class AppViewGuestDelegate {

[Yoyo Zhou, 2014/08/19 21:55:08]

This should have some (simple) class documentation.

+ public:
+  AppViewGuestDelegate();
+  virtual ~AppViewGuestDelegate();
+
+  // Shows the context menu for the guest.
+  // Returns true if the context menu was handled.
+  virtual bool OnHandleContextMenu(content::WebContents* web_contents,

[not at google - send to devlin, 2014/08/19 20:19:02]

Seems like this should be called just HandleContextMenu? OnHandleContextMenu
implies to me that the context menu was already handled and this is some kind of
observer event.

+                                   const content::ContextMenuParams& params);
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(AppViewGuestDelegate);
+};
+
+}  // namespace extensions
+
+#endif  // EXTENSIONS_BROWSER_GUEST_VIEW_APP_VIEW_APP_VIEW_GUEST_DELEGATE_H_