| OLD | NEW |
| 1 // Copyright 2016 The Chromium Authors. All rights reserved. | 1 // Copyright 2016 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #include "chrome/renderer/chrome_render_frame_observer.h" | 5 #include "chrome/renderer/chrome_render_frame_observer.h" |
| 6 | 6 |
| 7 #include <tuple> | 7 #include <tuple> |
| 8 | 8 |
| 9 #include "base/test/histogram_tester.h" | 9 #include "base/test/histogram_tester.h" |
| 10 #include "chrome/test/base/chrome_render_view_test.h" | 10 #include "chrome/test/base/chrome_render_view_test.h" |
| (...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 50 } // namespace | 50 } // namespace |
| 51 | 51 |
| 52 // Constants for UMA statistic collection. | 52 // Constants for UMA statistic collection. |
| 53 static const char kTranslateCaptureText[] = "Translate.CaptureText"; | 53 static const char kTranslateCaptureText[] = "Translate.CaptureText"; |
| 54 | 54 |
| 55 class ChromeRenderFrameObserverTest : public ChromeRenderViewTest { | 55 class ChromeRenderFrameObserverTest : public ChromeRenderViewTest { |
| 56 protected: | 56 protected: |
| 57 void SetUp() override { | 57 void SetUp() override { |
| 58 ChromeRenderViewTest::SetUp(); | 58 ChromeRenderViewTest::SetUp(); |
| 59 | 59 |
| 60 shell::InterfaceProvider* remote_interfaces = | 60 service_manager::InterfaceProvider* remote_interfaces = |
| 61 view_->GetMainRenderFrame()->GetRemoteInterfaces(); | 61 view_->GetMainRenderFrame()->GetRemoteInterfaces(); |
| 62 shell::InterfaceProvider::TestApi test_api(remote_interfaces); | 62 service_manager::InterfaceProvider::TestApi test_api(remote_interfaces); |
| 63 test_api.SetBinderForName( | 63 test_api.SetBinderForName( |
| 64 translate::mojom::ContentTranslateDriver::Name_, | 64 translate::mojom::ContentTranslateDriver::Name_, |
| 65 base::Bind(&FakeContentTranslateDriver::BindHandle, | 65 base::Bind(&FakeContentTranslateDriver::BindHandle, |
| 66 base::Unretained(&fake_translate_driver_))); | 66 base::Unretained(&fake_translate_driver_))); |
| 67 } | 67 } |
| 68 | 68 |
| 69 FakeContentTranslateDriver fake_translate_driver_; | 69 FakeContentTranslateDriver fake_translate_driver_; |
| 70 }; | 70 }; |
| 71 | 71 |
| 72 TEST_F(ChromeRenderFrameObserverTest, SkipCapturingSubFrames) { | 72 TEST_F(ChromeRenderFrameObserverTest, SkipCapturingSubFrames) { |
| 73 base::HistogramTester histogram_tester; | 73 base::HistogramTester histogram_tester; |
| 74 LoadHTML( | 74 LoadHTML( |
| 75 "<!DOCTYPE html><body>" | 75 "<!DOCTYPE html><body>" |
| 76 "This is a main document" | 76 "This is a main document" |
| 77 "<iframe srcdoc=\"This a document in an iframe.\">" | 77 "<iframe srcdoc=\"This a document in an iframe.\">" |
| 78 "</body>"); | 78 "</body>"); |
| 79 view_->GetWebView()->updateAllLifecyclePhases(); | 79 view_->GetWebView()->updateAllLifecyclePhases(); |
| 80 | 80 |
| 81 base::RunLoop().RunUntilIdle(); | 81 base::RunLoop().RunUntilIdle(); |
| 82 ASSERT_TRUE(fake_translate_driver_.called_new_page_); | 82 ASSERT_TRUE(fake_translate_driver_.called_new_page_); |
| 83 EXPECT_TRUE(fake_translate_driver_.page_needs_translation_) | 83 EXPECT_TRUE(fake_translate_driver_.page_needs_translation_) |
| 84 << "Page should be translatable."; | 84 << "Page should be translatable."; |
| 85 // Should have 2 samples: one for preliminary capture, one for final capture. | 85 // Should have 2 samples: one for preliminary capture, one for final capture. |
| 86 // If there are more, then subframes are being captured more than once. | 86 // If there are more, then subframes are being captured more than once. |
| 87 histogram_tester.ExpectTotalCount(kTranslateCaptureText, 2); | 87 histogram_tester.ExpectTotalCount(kTranslateCaptureText, 2); |
| 88 } | 88 } |
| OLD | NEW |