Rename RFH::IsDocumentOnLoadCompletedInPrimaryMainFrame to InMainFrame

Currently, RenderFrameHost::IsDocumentOnLoadCompletedForPrimaryMainFrame
returns the value of just the is_on_load_completed_in_main_document()
of any  page. Correct the naming to represent the current behavior.

Bug: 1307726
Change-Id: I609743bda24d71505a45639917c4fad33fb82e29
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3534698
Reviewed-by: Alex Moshchuk <[email protected]>
Owners-Override: Alex Moshchuk <[email protected]>
Commit-Queue: Sreeja Kamishetty <[email protected]>
Cr-Commit-Position: refs/heads/main@{#983504}

Author: sreejakshetty

chrome/browser/android/oom_intervention/oom_intervention_tab_helper.cc

@@ -164,7 +164,7 @@ void OomInterventionTabHelper::DidStartNavigation(
 }
 
 void OomInterventionTabHelper::PrimaryPageChanged(content::Page& page) {
-  if (!page.GetMainDocument().IsDocumentOnLoadCompletedInPrimaryMainFrame())
+  if (!page.GetMainDocument().IsDocumentOnLoadCompletedInMainFrame())
     return;
   if (IsLastVisibleWebContents(web_contents()))
     StartMonitoringIfNeeded();

components/favicon/content/content_favicon_driver.cc

@@ -173,7 +173,7 @@ void ContentFaviconDriver::DidUpdateFaviconURL(
   if (!entry)
     return;
 
-  if (!rfh->IsDocumentOnLoadCompletedInPrimaryMainFrame())
+  if (!rfh->IsDocumentOnLoadCompletedInMainFrame())
     return;
 
   OnUpdateCandidates(rfh->GetLastCommittedURL(),
@@ -188,7 +188,7 @@ void ContentFaviconDriver::DidUpdateWebManifestURL(
   // occur when loading an initially blank page.
   content::NavigationEntry* entry =
       web_contents()->GetController().GetLastCommittedEntry();
-  if (!entry || !rfh->IsDocumentOnLoadCompletedInPrimaryMainFrame())
+  if (!entry || !rfh->IsDocumentOnLoadCompletedInMainFrame())
     return;
 
   DocumentManifestData* document_data =

content/browser/renderer_host/render_frame_host_impl.cc

@@ -13022,7 +13022,7 @@ void RenderFrameHostImpl::DisableWebRtcEventLogOutput(int lid) {
   GetPeerConnectionTrackerHost().StopEventLog(lid);
 }
 
-bool RenderFrameHostImpl::IsDocumentOnLoadCompletedInPrimaryMainFrame() {
+bool RenderFrameHostImpl::IsDocumentOnLoadCompletedInMainFrame() {
   return GetPage().is_on_load_completed_in_main_document();
 }
 

content/browser/renderer_host/render_frame_host_impl.h

@@ -2333,7 +2333,7 @@ class CONTENT_EXPORT RenderFrameHostImpl
       mojo::PendingReceiver<blink::mojom::PeerConnectionTrackerHost> receiver);
   void EnableWebRtcEventLogOutput(int lid, int output_period_ms) override;
   void DisableWebRtcEventLogOutput(int lid) override;
-  bool IsDocumentOnLoadCompletedInPrimaryMainFrame() override;
+  bool IsDocumentOnLoadCompletedInMainFrame() override;
   const std::vector<blink::mojom::FaviconURLPtr>& FaviconURLs() override;
 
 #if BUILDFLAG(ENABLE_MDNS)

content/public/browser/render_frame_host.h

@@ -1007,7 +1007,7 @@ class CONTENT_EXPORT RenderFrameHost : public IPC::Listener,
   virtual void DisableWebRtcEventLogOutput(int lid) = 0;
 
   // Return true if onload has been executed in the renderer in the main frame.
-  virtual bool IsDocumentOnLoadCompletedInPrimaryMainFrame() = 0;
+  virtual bool IsDocumentOnLoadCompletedInMainFrame() = 0;
 
   // Returns the raw list of favicon candidates as reported to observers via
   // since the last navigation start. If called on a subframe, returns the