Delete most web intents code.

chrome/browser/extensions/api/app_runtime/app_runtime_api.h

 // 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.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_API_APP_RUNTIME_APP_RUNTIME_API_H_
 #define CHROME_BROWSER_EXTENSIONS_API_APP_RUNTIME_APP_RUNTIME_API_H_
 
 #include "base/string16.h"
 #include "chrome/browser/extensions/extension_function.h"
 
 class Profile;
 
 namespace content {
 class WebContents;
-class WebIntentsDispatcher;
 }
 
 namespace extensions {
 
 class Extension;
 
 class AppEventRouter {
  public:
   // Dispatches the onLaunched event to the given app, providing no launch
   // data.
   static void DispatchOnLaunchedEvent(Profile* profile,
                                       const Extension* extension);
   static void DispatchOnRestartedEvent(Profile* profile,
                                        const Extension* extension);
 
   // Dispatches the onLaunched event to the given app, providing launch data of

[benwells, 2013/02/07 06:00:03]

Please add a TODO here on me to update this comment, it is out of date.

[Nico, 2013/02/07 06:20:23]

Done.

   // the form:
   // {
   //   "intent" : {
-  //     "action" : |action|,
   //     "type" : "chrome-extension://fileentry",
   //     "data" : a FileEntry,
   //     "postResults" : a null function,
   //     "postFailure" : a null function
   //   }
   // }
 
   // launchData.intent.data and launchData.intent.postResults are created in a
   // custom dispatch event in javascript. The FileEntry is created from
   // |file_system_id| and |base_name|.
   static void DispatchOnLaunchedEventWithFileEntry(
       Profile* profile,
       const Extension* extension,
-      const string16& action,
       const std::string& handler_id,
       const std::string& mime_type,
       const std::string& file_system_id,
       const std::string& base_name);
-
-  // Dispatches the onLaunched event to the app implemented by |extension|
-  // running in |profile|. The event parameter launchData will have a field
-  // called intent, populated by |web_intent_data|.
-  static void DispatchOnLaunchedEventWithWebIntent(
-      Profile* profile,
-      const Extension* extension,
-      content::WebIntentsDispatcher* intents_dispatcher,
-      content::WebContents* source);
 };
 
 class AppRuntimePostIntentResponseFunction : public SyncExtensionFunction {

[Nico, 2013/02/07 05:25:36]

This is still here because as far as I understand it's needed to silently ignore
"intent" sections in the manifest. Is this correct?

[benwells, 2013/02/07 06:00:03]

I don't think so, it should be removed.

[Nico, 2013/02/07 06:20:23]

If I delete the class, I get this error:

  gen/chrome/common/extensions/api/generated_api.h:308:32: error: use of
undeclared identifier 'AppRuntimePostIntentResponseFunction'

Should I delete postIntentResponse() from
chrome/common/extensions/api/app_runtime.idl ? Does this not break apps apis?

[benwells, 2013/02/07 06:44:53]

Yes you should delete that and the other intent stuff from the IDL. These apis
are new and platform apps are not yet publicly released, so we still have room
to remove these APIs.

[Nico, 2013/02/08 22:42:55]

Done.

  public:
   DECLARE_EXTENSION_FUNCTION("app.runtime.postIntentResponse",
                              APP_RUNTIME_POSTINTENTRESPONSE)
 
  protected:
   virtual ~AppRuntimePostIntentResponseFunction() {}
   virtual bool RunImpl() OVERRIDE;
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_APP_RUNTIME_APP_RUNTIME_API_H_