Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(10)

Unified Diff: chrome/browser/ui/pdf/pdf_interactive_browsertest.cc

Issue 1128453003: Remove the OOP PDF flag (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/ui/pdf/pdf_browsertest_base.cc ('k') | chrome/chrome_tests.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/pdf/pdf_interactive_browsertest.cc
diff --git a/chrome/browser/ui/pdf/pdf_interactive_browsertest.cc b/chrome/browser/ui/pdf/pdf_interactive_browsertest.cc
deleted file mode 100644
index 2022320b60599acc950944ae7dc0381aa08e1d13..0000000000000000000000000000000000000000
--- a/chrome/browser/ui/pdf/pdf_interactive_browsertest.cc
+++ /dev/null
@@ -1,49 +0,0 @@
-// 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.
-
-#include "base/strings/utf_string_conversions.h"
-#include "chrome/browser/ui/browser.h"
-#include "chrome/browser/ui/pdf/pdf_browsertest_base.h"
-#include "chrome/browser/ui/tabs/tab_strip_model.h"
-#include "chrome/test/base/ui_test_utils.h"
-#include "content/public/browser/web_contents.h"
-#include "ui/base/clipboard/clipboard.h"
-
-// Note: All tests in here require the internal PDF plugin, so they're disabled
-// in non-official builds. We still compile them though, to prevent bitrot.
-//
-// These tests are interactive because they access the clipboard.
-
-namespace {
-
-const char kSearchKeyword[] = "adipiscing";
-
-#if defined(GOOGLE_CHROME_BUILD) && (defined(OS_WIN) || defined(OS_LINUX))
-#define MAYBE_FindAndCopy DISABLED_FindAndCopy
-#else
-// TODO(thestig): http://crbug.com/79837, http://crbug.com/329912
-#define MAYBE_FindAndCopy DISABLED_FindAndCopy
-#endif
-IN_PROC_BROWSER_TEST_F(PDFBrowserTest, MAYBE_FindAndCopy) {
- ASSERT_NO_FATAL_FAILURE(Load());
- // Verifies that find in page works.
- ASSERT_EQ(3, ui_test_utils::FindInPage(
- browser()->tab_strip_model()->GetActiveWebContents(),
- base::ASCIIToUTF16(kSearchKeyword),
- true, false, NULL, NULL));
-
- // Verify that copying selected text works.
- ui::Clipboard* clipboard = ui::Clipboard::GetForCurrentThread();
- // Reset the clipboard first.
- clipboard->Clear(ui::CLIPBOARD_TYPE_COPY_PASTE);
-
- browser()->tab_strip_model()->GetActiveWebContents()->Copy();
- ASSERT_NO_FATAL_FAILURE(WaitForResponse());
-
- std::string text;
- clipboard->ReadAsciiText(ui::CLIPBOARD_TYPE_COPY_PASTE, &text);
- EXPECT_EQ(kSearchKeyword, text);
-}
-
-} // namespace
« no previous file with comments | « chrome/browser/ui/pdf/pdf_browsertest_base.cc ('k') | chrome/chrome_tests.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698