Multi-Process Find-in-Page.

third_party/WebKit/Source/web/WebLocalFrameImpl.cpp

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 604 matching lines @@
     m_sharedWorkerRepositoryClient = SharedWorkerRepositoryClientImpl::create(client);
 }
 
 ScrollableArea* WebLocalFrameImpl::layoutViewportScrollableArea() const
 {
     if (FrameView* view = frameView())
         return view->layoutViewportScrollableArea();
     return nullptr;
 }
 
+bool WebLocalFrameImpl::isFocused() const
+{
+    return this == WebFrame::fromFrame(viewImpl()->page()->focusController().focusedFrame());
+}
+
 WebSize WebLocalFrameImpl::scrollOffset() const
 {
     if (ScrollableArea* scrollableArea = layoutViewportScrollableArea())
         return toIntSize(scrollableArea->scrollPosition());
     return WebSize();
 }
 
 void WebLocalFrameImpl::setScrollOffset(const WebSize& offset)
 {
     if (ScrollableArea* scrollableArea = layoutViewportScrollableArea())
@@ skipping 911 matching lines @@
     // Note a synchronous navigation (about:blank) would have already processed
     // onload, so it is possible for the child frame to have already been
     // detached by script in the page.
     if (!webframeChild->parent())
         return nullptr;
     return webframeChild->frame();
 }
 
 void WebLocalFrameImpl::didChangeContentsSize(const IntSize& size)
 {
-    // This is only possible on the main frame.
-    if (m_textFinder && m_textFinder->totalMatchCount() > 0) {
-        DCHECK(!parent());
+    if (m_textFinder && m_textFinder->totalMatchCount() > 0)
         m_textFinder->increaseMarkerVersion();
-    }
 }
 
 void WebLocalFrameImpl::createFrameView()
 {
     TRACE_EVENT0("blink", "WebLocalFrameImpl::createFrameView");
 
     DCHECK(frame()); // If frame() doesn't exist, we probably didn't init properly.
 
     WebViewImpl* webView = viewImpl();
 
@@ skipping 47 matching lines @@
     return static_cast<WebDataSourceImpl*>(dataSource());
 }
 
 WebDataSourceImpl* WebLocalFrameImpl::provisionalDataSourceImpl() const
 {
     return static_cast<WebDataSourceImpl*>(provisionalDataSource());
 }
 
 void WebLocalFrameImpl::setFindEndstateFocusAndSelection()
 {
-    WebLocalFrameImpl* mainFrameImpl = viewImpl()->mainFrameImpl();
-
-    // Main frame should already have a textFinder at this point of time.
-    DCHECK(mainFrameImpl->textFinder());
-
-    if (this != mainFrameImpl->textFinder()->activeMatchFrame())
+    if (!m_textFinder || !m_textFinder->activeMatchFrame())
         return;
 
     if (Range* activeMatch = m_textFinder->activeMatch()) {
         // If the user has set the selection since the match was found, we
         // don't focus anything.
         VisibleSelection selection(frame()->selection().selection());
         if (!selection.isNone())
             return;
 
         // Need to clean out style and layout state before querying Element::isFocusable().
@@ skipping 325 matching lines @@
     UseCounter::count(frame(), UseCounter::ExternalAddSearchProvider);
 }
 
 void WebLocalFrameImpl::didCallIsSearchProviderInstalled()
 {
     UseCounter::count(frame(), UseCounter::ExternalIsSearchProviderInstalled);
 }
 
 bool WebLocalFrameImpl::find(int identifier, const WebString& searchText, const WebFindOptions& options, bool wrapWithinFrame, WebRect* selectionRect, bool* activeNow)
 {
-    return ensureTextFinder().find(identifier, searchText, options, wrapWithinFrame, selectionRect, activeNow);
+    // Search for an active match only if this frame is focused or if this is a
+    // find next request.
+    if (isFocused() || options.findNext)
+        return ensureTextFinder().find(identifier, searchText, options, wrapWithinFrame, selectionRect, activeNow);
+
+    return false;
 }
 
-void WebLocalFrameImpl::stopFinding(bool clearSelection)
+void WebLocalFrameImpl::stopFinding(StopFindAction action)
 {
+    bool clearSelection = action == StopFindActionClearSelection;
+    if (clearSelection)
+        executeCommand(WebString::fromUTF8("Unselect"));

[dglazkov, 2016/06/02 01:08:42]

Curious -- why is this necessary now? Why wasn't it necessary before?

[paulmeyer, 2016/06/02 17:36:37]

I saw that you said I could disregard these comments, but I thought I'd explain
a bit anyways.

This was done before and now, and it's because this is the functionality of
StopFindActionClearSelection (and the content-side enum equivalent). You can
specify "clear" as the action you want to take when stopping a find session to
prevent the active match from remaining selected/highlighted, which is the
default behavior.

+
     if (m_textFinder) {
         if (!clearSelection)
             setFindEndstateFocusAndSelection();
         m_textFinder->stopFindingAndClearSelection();
     }
+
+    if (action == StopFindActionActivateSelection && isFocused()) {
+        WebDocument doc = document();
+        if (!doc.isNull()) {

[dglazkov, 2016/06/02 01:08:42]

It's not possible for a frame to be focused and not have a document, is it?

[paulmeyer, 2016/06/02 17:36:36]

I'm actually not sure. It was checked before, so I left the check in. It might
be a candidate for removal in a cleanup CL.

+            WebElement element = doc.focusedElement();
+            if (!element.isNull())
+                element.simulateClick();

[dglazkov, 2016/06/02 01:08:42]

Huh. This is also new code? Why do we need to do this now?

[paulmeyer, 2016/06/02 17:36:37]

This is the functionality of the "activate" action. When a find session is
stopped with that specified action, a click is simulated on the active match to
activate it.

+        }
+    }
 }
 
 void WebLocalFrameImpl::scopeStringMatches(int identifier, const WebString& searchText, const WebFindOptions& options, bool reset)
 {
     ensureTextFinder().scopeStringMatches(identifier, searchText, options, reset);
 }
 
 void WebLocalFrameImpl::cancelPendingScopingEffort()
 {
     if (m_textFinder)
         m_textFinder->cancelPendingScopingEffort();
 }
 
 void WebLocalFrameImpl::increaseMatchCount(int count, int identifier)
 {
-    // This function should only be called on the mainframe.
-    DCHECK(!parent());
     ensureTextFinder().increaseMatchCount(identifier, count);
 }
 
 void WebLocalFrameImpl::resetMatchCount()
 {
     ensureTextFinder().resetMatchCount();
 }
 
 void WebLocalFrameImpl::dispatchMessageEventWithOriginCheck(const WebSecurityOrigin& intendedTargetOrigin, const WebDOMEvent& event)
 {
     DCHECK(!event.isNull());
     frame()->localDOMWindow()->dispatchMessageEventWithOriginCheck(intendedTargetOrigin.get(), event, nullptr);
 }
 
 int WebLocalFrameImpl::findMatchMarkersVersion() const
 {
-    DCHECK(!parent());
-
     if (m_textFinder)
         return m_textFinder->findMatchMarkersVersion();
     return 0;
 }
 
 int WebLocalFrameImpl::selectNearestFindMatch(const WebFloatPoint& point, WebRect* selectionRect)
 {
-    DCHECK(!parent());
     return ensureTextFinder().selectNearestFindMatch(point, selectionRect);
 }
 
+float WebLocalFrameImpl::distanceToNearestFindMatch(const WebFloatPoint& point)
+{
+    float nearestDistance;
+    ensureTextFinder().nearestFindMatch(point, &nearestDistance);
+    return nearestDistance;
+}
+
 WebFloatRect WebLocalFrameImpl::activeFindMatchRect()
 {
-    DCHECK(!parent());
-
     if (m_textFinder)
         return m_textFinder->activeFindMatchRect();
     return WebFloatRect();
 }
 
 void WebLocalFrameImpl::findMatchRects(WebVector<WebFloatRect>& outputRects)
 {
-    DCHECK(!parent());
     ensureTextFinder().findMatchRects(outputRects);
 }
 
 void WebLocalFrameImpl::setTickmarks(const WebVector<WebRect>& tickmarks)
 {
     if (frameView()) {
         Vector<IntRect> tickmarksConverted(tickmarks.size());
         for (size_t i = 0; i < tickmarks.size(); ++i)
             tickmarksConverted[i] = tickmarks[i];
         frameView()->setTickmarks(tickmarksConverted);
@@ skipping 48 matching lines @@
     if (!frame())
         return WebSandboxFlags::None;
     return static_cast<WebSandboxFlags>(frame()->loader().effectiveSandboxFlags());
 }
 
 void WebLocalFrameImpl::forceSandboxFlags(WebSandboxFlags flags)
 {
     frame()->loader().forceSandboxFlags(static_cast<SandboxFlags>(flags));
 }
 
+void WebLocalFrameImpl::clearActiveFindMatch()
+{
+    ensureTextFinder().clearActiveFindMatch();
+}
+
 } // namespace blink