diff options
author | robertshield@chromium.org <robertshield@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-01-05 19:16:23 +0000 |
---|---|---|
committer | robertshield@chromium.org <robertshield@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-01-05 19:16:23 +0000 |
commit | ea9ed97d074fa3ae84486f952eb66ea1e95ce37e (patch) | |
tree | d4603d01863ded8450e4d3109bf56d406e2c1d1a /chrome_frame/html_utils_unittest.cc | |
parent | 880b6f26f7cda4ddefbf2c269a2ec86dce71c2ff (diff) | |
download | chromium_src-ea9ed97d074fa3ae84486f952eb66ea1e95ce37e.zip chromium_src-ea9ed97d074fa3ae84486f952eb66ea1e95ce37e.tar.gz chromium_src-ea9ed97d074fa3ae84486f952eb66ea1e95ce37e.tar.bz2 |
Some Chrome Frame cleanup:
1) Rearrange the chrome_frame.gyp file to:
a) Split out most of CF into a static lib to make writing unit tests easier (not having to re-include source files).
b) Remove most of the un-needed xulrunner-sdk includes.
2) Move all OBJECT_ENTRY_AUTO macros to chrome_tab.cc since they don't work without extra work when residing in a lib.
3) Rename npchrome_tab.dll to npchrome_frame.dll.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/523040
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35543 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome_frame/html_utils_unittest.cc')
-rw-r--r-- | chrome_frame/html_utils_unittest.cc | 61 |
1 files changed, 0 insertions, 61 deletions
diff --git a/chrome_frame/html_utils_unittest.cc b/chrome_frame/html_utils_unittest.cc deleted file mode 100644 index 73b7a4a..0000000 --- a/chrome_frame/html_utils_unittest.cc +++ /dev/null @@ -1,61 +0,0 @@ -// Copyright (c) 2009 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 "base/logging.h" -#include "chrome_frame/html_utils.h" -#include "testing/gtest/include/gtest/gtest.h" - -namespace { - -TEST(HttpUtils, HasFrameBustingHeader) { - // Simple negative cases. - ASSERT_FALSE(http_utils::HasFrameBustingHeader("")); - ASSERT_FALSE(http_utils::HasFrameBustingHeader("Content-Type: text/plain")); - // Explicit negative cases, test that we ignore case. - ASSERT_FALSE(http_utils::HasFrameBustingHeader("X-Frame-Options: ALLOWALL")); - ASSERT_FALSE(http_utils::HasFrameBustingHeader("X-Frame-Options: allowall")); - ASSERT_FALSE(http_utils::HasFrameBustingHeader("X-Frame-Options: ALLowalL")); - // Added space, ensure stripped out - ASSERT_FALSE(http_utils::HasFrameBustingHeader( - "X-Frame-Options: ALLOWALL ")); - // Added space with linefeed, ensure still stripped out - ASSERT_FALSE(http_utils::HasFrameBustingHeader( - "X-Frame-Options: ALLOWALL \r\n")); - // Multiple identical headers, all of them allowing framing. - ASSERT_FALSE(http_utils::HasFrameBustingHeader( - "X-Frame-Options: ALLOWALL\r\n" - "X-Frame-Options: ALLOWALL\r\n" - "X-Frame-Options: ALLOWALL")); - // Interleave with other headers. - ASSERT_FALSE(http_utils::HasFrameBustingHeader( - "Content-Type: text/plain\r\n" - "X-Frame-Options: ALLOWALL\r\n" - "Content-Length: 42")); - - // Simple positive cases. - ASSERT_TRUE(http_utils::HasFrameBustingHeader("X-Frame-Options: deny")); - ASSERT_TRUE(http_utils::HasFrameBustingHeader( - "X-Frame-Options: SAMEorigin")); - - // Allowall entries do not override the denying entries, are - // order-independent, and the deny entries can interleave with - // other headers. - ASSERT_TRUE(http_utils::HasFrameBustingHeader( - "Content-Length: 42\r\n" - "X-Frame-Options: ALLOWall\r\n" - "X-Frame-Options: deny\r\n")); - ASSERT_TRUE(http_utils::HasFrameBustingHeader( - "X-Frame-Options: ALLOWall\r\n" - "Content-Length: 42\r\n" - "X-Frame-Options: SAMEORIGIN\r\n")); - ASSERT_TRUE(http_utils::HasFrameBustingHeader( - "X-Frame-Options: deny\r\n" - "X-Frame-Options: ALLOWall\r\n" - "Content-Length: 42\r\n")); - ASSERT_TRUE(http_utils::HasFrameBustingHeader( - "X-Frame-Options: SAMEORIGIN\r\n" - "X-Frame-Options: ALLOWall\r\n")); -} - -} // namespace |