Reviewed by Maciej Stachowiak.
[WebKit-https.git] / WebKit / WebView.subproj / WebView.m
index 5e772fa..8dd919a 100644 (file)
@@ -190,11 +190,6 @@ macro(yankAndSelect) \
 - (BOOL)_shouldAutoscrollForDraggingInfo:(id)dragInfo;
 @end
 
-@interface NSObject (WebDocumentSearchingHack)
-// FIXME: this should be part of a protocol (new version of <WebDocumentSearching>)
-- (BOOL)_searchFor:(NSString *)string direction:(BOOL)forward caseSensitive:(BOOL)caseFlag wrap:(BOOL)wrapFlag findInSelection:(BOOL)inSelectionFlag;
-@end
-
 @interface WebView (WebFileInternal)
 - (void)_preflightSpellChecker;
 - (BOOL)_continuousCheckingAllowed;
@@ -2169,9 +2164,7 @@ static WebFrame *incrementFrame(WebFrame *curr, BOOL forward, BOOL wrapFlag)
                    : [curr _previousFrameWithWrap:wrapFlag];
 }
 
-// Search from the end of the currently selected location, or from the beginning of the
-// document if nothing is selected.  Deals with subframes.
-- (BOOL)_searchFor:(NSString *)string direction:(BOOL)forward caseSensitive:(BOOL)caseFlag wrap:(BOOL)wrapFlag findInSelection:(BOOL)findInSelection
+- (BOOL)searchFor:(NSString *)string direction:(BOOL)forward caseSensitive:(BOOL)caseFlag wrap:(BOOL)wrapFlag
 {
     // Get the frame holding the selection, or start with the main frame
     WebFrame *startFrame = [self _frameForCurrentSelection];
@@ -2199,13 +2192,7 @@ static WebFrame *incrementFrame(WebFrame *curr, BOOL forward, BOOL wrapFlag)
             
             // Note at this point we are assuming the search will be done top-to-bottom,
             // not starting at any selection that exists.  See 3228554.
-            BOOL success;
-            if ([searchView respondsToSelector:@selector(_searchFor:direction:caseSensitive:wrap:findInSelection:)]) {
-                success = [searchView _searchFor:string direction:forward caseSensitive:caseFlag wrap:NO findInSelection:findInSelection];
-            } else {
-                success = [searchView searchFor:string direction:forward caseSensitive:caseFlag wrap:NO];
-            }
-            if (success) {
+            if ([searchView searchFor:string direction:forward caseSensitive:caseFlag wrap:NO]) {
                 [[self window] makeFirstResponder:searchView];
                 return YES;
             }
@@ -2216,13 +2203,7 @@ static WebFrame *incrementFrame(WebFrame *curr, BOOL forward, BOOL wrapFlag)
     // Search contents of startFrame, on the other side of the selection that we did earlier.
     // We cheat a bit and just research with wrap on
     if (wrapFlag && startHasSelection && startSearchView) {
-        BOOL success;
-        if ([startSearchView respondsToSelector:@selector(_searchFor:direction:caseSensitive:wrap:findInSelection:)]) {
-            success = [startSearchView _searchFor:string direction:forward caseSensitive:caseFlag wrap:YES findInSelection:findInSelection];
-        } else {
-            success = [startSearchView searchFor:string direction:forward caseSensitive:caseFlag wrap:YES];
-        }
-        if (success) {
+        if ([startSearchView searchFor:string direction:forward caseSensitive:caseFlag wrap:YES]) {
             [[self window] makeFirstResponder:startSearchView];
             return YES;
         }
@@ -2230,12 +2211,6 @@ static WebFrame *incrementFrame(WebFrame *curr, BOOL forward, BOOL wrapFlag)
     return NO;
 }
 
-- (BOOL)searchFor:(NSString *)string direction:(BOOL)forward caseSensitive:(BOOL)caseFlag wrap:(BOOL)wrapFlag
-{
-    return [self _searchFor:string direction:forward caseSensitive:caseFlag wrap:wrapFlag findInSelection:NO];
-}
-
-
 + (void)registerViewClass:(Class)viewClass representationClass:(Class)representationClass forMIMEType:(NSString *)MIMEType
 {
     [[WebFrameView _viewTypesAllowImageTypeOmission:YES] setObject:viewClass forKey:MIMEType];