Remove unintended options_page support from platform apps.

chrome/common/extensions/manifest_tests/extension_manifests_options_unittest.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/common/extensions/extension_manifest_constants.h"
 #include "chrome/common/extensions/manifest_tests/extension_manifest_test.h"
 #include "chrome/common/extensions/manifest_url_handler.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace errors = extension_manifest_errors;
@@ skipping 14 matching lines @@
       "http",
       extensions::ManifestURL::GetOptionsPage(extension).scheme().c_str());
   EXPECT_STREQ(
       "example.com",
       extensions::ManifestURL::GetOptionsPage(extension).host().c_str());
   EXPECT_STREQ(
       "options.html",
       extensions::ManifestURL::
           GetOptionsPage(extension).ExtractFileName().c_str());
 
+  extension = LoadAndExpectSuccess("platform_app_with_options_page.json");

[Yoyo Zhou, 2013/04/16 20:11:40]

Maybe add a comment here: // Platforms apps do not support options pages.

+  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",
              errors::kInvalidOptionsPageInHostedApp),
 
     // Forbid options page with non-(http|https) scheme in hosted app.
     Testcase("hosted_app_file_options.json",
              errors::kInvalidOptionsPageInHostedApp),
 
     // Forbid absolute URL for options page in packaged apps.
     Testcase("packaged_app_absolute_options.json",
              errors::kInvalidOptionsPageExpectUrlInPackage)
   };
   RunTestcases(testcases, arraysize(testcases),
                EXPECT_TYPE_ERROR);
 }