Remove unintended options_page support from platform apps.

Platform apps do not use an Options page. This change makes OptionsPageHandler
a no-op when parsing a platform app manifest.

BUG=229584

Review URL: https://chromiumcodereview.appspot.com/14130005

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@194801 0039d316-1c4b-4281-b951-d872f2087c98

chrome/common/extensions/api/_manifest_features.json

@@ -237,7 +237,7 @@
   "options_page": {
     "channel": "stable",
     "extension_types": [
-      "extension", "packaged_app", "hosted_app", "platform_app"
+      "extension", "packaged_app", "hosted_app"
     ]
   },
   "oauth2": {

chrome/common/extensions/manifest_tests/extension_manifests_options_unittest.cc

@@ -32,6 +32,9 @@ TEST_F(OptionsPageManifestTest, OptionsPageInApps) {
       extensions::ManifestURL::
           GetOptionsPage(extension).ExtractFileName().c_str());
 
+  extension = LoadAndExpectSuccess("platform_app_with_options_page.json");
+  EXPECT_TRUE(extensions::ManifestURL::GetOptionsPage(extension).is_empty());
+
   Testcase testcases[] = {
     // Forbid options page with relative URL in hosted apps.
     Testcase("hosted_app_relative_options.json",

chrome/test/data/extensions/manifest_tests/platform_app_with_options_page.json

+{
+  "name": "test",
+  "description": "I specify an options_page but it should be ignored.",
+  "version": "1",
+  "app": {
+    "background": {
+      "scripts": ["background.js"]
+    }
+  },
+  "options_page": "options.html"
+}