Don't zoom in TextFinder if autosizing is disabled

Source/web/tests/TextFinderTest.cpp

 // 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 "config.h"
 #include "web/TextFinder.h"
 
 #include "bindings/core/v8/ExceptionStatePlaceholder.h"
 #include "core/dom/Document.h"
 #include "core/dom/NodeList.h"
 #include "core/dom/Range.h"
 #include "core/dom/shadow/ShadowRoot.h"
+#include "core/frame/FrameHost.h"
 #include "core/html/HTMLElement.h"
+#include "core/layout/TextAutosizer.h"
+#include "core/page/Page.h"
 #include "platform/testing/UnitTestHelpers.h"
 #include "public/platform/Platform.h"
 #include "public/web/WebDocument.h"
 #include "web/FindInPageCoordinates.h"
 #include "web/WebLocalFrameImpl.h"
 #include "web/tests/FrameTestHelpers.h"
 #include "wtf/OwnPtr.h"
 #include <gtest/gtest.h>
 
 using blink::testing::runPendingTasks;
 
 namespace blink {
 
 class TextFinderTest : public ::testing::Test {
 protected:
     void SetUp() override;
 
     Document& document() const;
     TextFinder& textFinder() const;
+    LocalFrame& frame() const;
 
     static WebFloatRect findInPageRect(Node* startContainer, int startOffset, Node* endContainer, int endOffset);
 
 private:
     FrameTestHelpers::WebViewHelper m_webViewHelper;
     RefPtrWillBePersistent<Document> m_document;
     TextFinder* m_textFinder;
 };
 
 void TextFinderTest::SetUp()
 {
     m_webViewHelper.initialize();
     WebLocalFrameImpl& frameImpl = *m_webViewHelper.webViewImpl()->mainFrameImpl();
     frameImpl.viewImpl()->resize(WebSize(640, 480));
     frameImpl.viewImpl()->layout();
     m_document = PassRefPtrWillBeRawPtr<Document>(frameImpl.document());
     m_textFinder = &frameImpl.ensureTextFinder();
 }
 
 Document& TextFinderTest::document() const
 {
     return *m_document;
 }
 
 TextFinder& TextFinderTest::textFinder() const
 {
     return *m_textFinder;
 }
 
+LocalFrame& TextFinderTest::frame() const
+{
+    return *m_webViewHelper.webViewImpl()->mainFrameImpl()->frame();
+}
+
 WebFloatRect TextFinderTest::findInPageRect(Node* startContainer, int startOffset, Node* endContainer, int endOffset)
 {
     RefPtrWillBeRawPtr<Range> range = Range::create(startContainer->document(), startContainer, startOffset, endContainer, endOffset);
     return WebFloatRect(findInPageRectFromRange(range.get()));
 }
 
 TEST_F(TextFinderTest, FindTextSimple)
 {
     document().body()->setInnerHTML("XXXXFindMeYYYYfindmeZZZZ", ASSERT_NO_EXCEPTION);
     Node* textNode = document().body()->firstChild();
@@ skipping 55 matching lines @@
     // Wrap to the first match (last occurence in the document).
     ASSERT_TRUE(textFinder().find(identifier, searchText, findOptions, wrapWithinFrame, selectionRect));
     activeMatch = textFinder().activeMatch();
     ASSERT_TRUE(activeMatch);
     EXPECT_EQ(textNode, activeMatch->startContainer());
     EXPECT_EQ(14, activeMatch->startOffset());
     EXPECT_EQ(textNode, activeMatch->endContainer());
     EXPECT_EQ(20, activeMatch->endOffset());
 }
 
+TEST_F(TextFinderTest, FindTextAutosizing)
+{
+    document().body()->setInnerHTML("XXXXFindMeYYYYfindmeZZZZ", ASSERT_NO_EXCEPTION);
+
+    int identifier = 0;
+    WebString searchText(String("FindMe"));
+    WebFindOptions findOptions; // Default.
+    bool wrapWithinFrame = true;
+    WebRect* selectionRect = nullptr;
+
+    // Set viewport scale to 20 in order to simulate zoom-in
+    PinchViewport& pinchViewport = frame().page()->frameHost().pinchViewport();

[philipj_slow, 2015/07/16 07:44:17]

Would document().page() do the same thing? If so you could drop the new frame()
helper.

+    pinchViewport.setScale(20);
+
+    // Enforce autosizing
+    document().settings()->setTextAutosizingEnabled(true);
+    document().settings()->setTextAutosizingWindowSizeOverride(IntSize(20, 20));
+    document().textAutosizer()->updatePageInfo();
+
+    // In case of autosizing, scale _should_ change
+    ASSERT_TRUE(textFinder().find(identifier, searchText, findOptions, wrapWithinFrame, selectionRect));
+    ASSERT_TRUE(textFinder().activeMatch());
+    ASSERT_EQ(1, pinchViewport.scale()); // in this case to 1
+
+    // Disable autosizing and reset scale to 20
+    pinchViewport.setScale(20);
+    document().settings()->setTextAutosizingEnabled(false);
+    document().textAutosizer()->updatePageInfo();
+
+    ASSERT_TRUE(textFinder().find(identifier, searchText, findOptions, wrapWithinFrame, selectionRect));
+    ASSERT_TRUE(textFinder().activeMatch());
+    ASSERT_EQ(20, pinchViewport.scale());
+}
+
 TEST_F(TextFinderTest, FindTextNotFound)
 {
     document().body()->setInnerHTML("XXXXFindMeYYYYfindmeZZZZ", ASSERT_NO_EXCEPTION);
 
     int identifier = 0;
     WebString searchText(String("Boo"));
     WebFindOptions findOptions; // Default.
     bool wrapWithinFrame = true;
     WebRect* selectionRect = nullptr;
 
@@ skipping 318 matching lines @@
     // There will be only one iteration before timeout, because increment
     // of the TimeProxyPlatform timer is greater than timeout threshold.
     textFinder().scopeStringMatches(identifier, searchPattern, findOptions, true);
     while (textFinder().scopingInProgress())
         runPendingTasks();
 
     EXPECT_EQ(4, textFinder().totalMatchCount());
 }
 
 } // namespace blink