Fix bug 455: Active match in Find is not selected and links are not focused a...

webkit/glue/webframe_impl.h

 /*
  * Copyright (C) 2006 Apple Computer, 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:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 106 matching lines @@
   virtual void CallJSGC();
 
   virtual void* GetFrameImplementation() { return frame(); }
 
   virtual void GetContentAsPlainText(int max_chars, std::wstring* text) const;
   virtual bool Find(const FindInPageRequest& request,
                     bool wrap_within_frame,
                     gfx::Rect* selection_rect);
   virtual bool FindNext(const FindInPageRequest& request,
                         bool wrap_within_frame);
-  virtual void StopFinding();
+  virtual void StopFinding(bool clear_selection);
   virtual void ScopeStringMatches(FindInPageRequest request, bool reset);
   virtual void CancelPendingScopingEffort();
   virtual void ResetMatchCount();
   virtual bool Visible();
   virtual void SelectAll();
   virtual void Copy();
   virtual void Cut();
   virtual void Paste();
   virtual void Replace(const std::wstring& text);
   virtual void Delete();
@@ skipping 106 matching lines @@
   // value can be NULL if no frame has an active tickmark.
   const WebFrameImpl* active_tickmark_frame() const {
     return active_tickmark_frame_;
   }
 
   // Returns the index of the active tickmark for this frame.
   size_t active_tickmark_index() const {
     return active_tickmark_;
   }
 
+  // When a Find operation ends, we want to set the selection to what was active
+  // and set focus to the first focusable node we find (starting with the first
+  // node in the matched range and going up the inheritance chain). If we find
+  // nothing to focus we focus the first focusable node in the range. This
+  // allows us to set focus to a link (when we find text inside a link), which
+  // allows us to navigate by pressing Enter after closing the Find box.
+  void SetFindEndstateFocusAndSelection();
+
   // Sets whether the WebFrameImpl allows its document to be scrolled.
   // If the parameter is true, allow the document to be scrolled.
   // Otherwise, disallow scrolling
   void SetAllowsScrolling(bool flag);
 
   // Returns true if the frame CSS is in "printing" mode.
   bool printing() const { return printing_; }
 
   virtual bool HasUnloadListener();
   virtual bool IsReloadAllowingStaleData() const;
@@ skipping 167 matching lines @@
   // In "printing" mode. Used as a state check.
   bool printing_;
 
   // For each printed page, the view of the document in pixels.
   Vector<WebCore::IntRect> pages_;
 
   DISALLOW_COPY_AND_ASSIGN(WebFrameImpl);
 };
 
 #endif  // WEBKIT_GLUE_WEBFRAME_IMPL_H_