Do not reinvent the wheel / use content APIs to find a frame by name.

content/browser/webui/web_ui_impl.h

 // Copyright (c) 2012 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.
 
 #ifndef CONTENT_BROWSER_WEBUI_WEB_UI_IMPL_H_
 #define CONTENT_BROWSER_WEBUI_WEB_UI_IMPL_H_
 
 #include <map>
 #include <memory>
 #include <set>
@@ skipping 78 matching lines @@
                    const base::ListValue& args);
 
   // Execute a string of raw JavaScript on the page.
   void ExecuteJavascript(const base::string16& javascript);
 
   // Finds the frame in which to execute JavaScript based on |frame_name_|. If
   // |frame_name_| is empty, the main frame is returned. May return NULL if no
   // frame of the specified name exists in the page.
   RenderFrameHost* TargetFrame();
 
-  // A helper function for TargetFrame; adds a frame to the specified set if its
-  // name matches |frame_name_|.
-  void AddToSetIfFrameNameMatches(std::set<RenderFrameHost*>* frame_set,
-                                  RenderFrameHost* host);
-
   // Called internally and by the owned MainFrameNavigationObserver.
   void DisallowJavascriptOnAllHandlers();
 
   // A map of message name -> message handling callback.
   typedef std::map<std::string, MessageCallback> MessageCallbackMap;
   MessageCallbackMap message_callbacks_;
 
   // Options that may be overridden by individual Web UI implementations. The
   // bool options default to false. See the public getters for more information.
   base::string16 overridden_title_;  // Defaults to empty string.
@@ skipping 14 matching lines @@
   const std::string frame_name_;
 
   std::unique_ptr<WebUIController> controller_;
 
   DISALLOW_COPY_AND_ASSIGN(WebUIImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_WEBUI_WEB_UI_IMPL_H_