Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(83)

Side by Side Diff: content/browser/web_contents/web_contents_impl.h

Issue 135723003: Move DidCommitProvisionalLoad code from RenderView to RenderFrame. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Some clean up, ready to start reviewing. Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_IMPL_H_ 5 #ifndef CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_IMPL_H_
6 #define CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_IMPL_H_ 6 #define CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_IMPL_H_
7 7
8 #include <map> 8 #include <map>
9 #include <set> 9 #include <set>
10 #include <string> 10 #include <string>
(...skipping 312 matching lines...) Expand 10 before | Expand all | Expand 10 after
323 // virtual WebContents* GetAsWebContents() OVERRIDE; 323 // virtual WebContents* GetAsWebContents() OVERRIDE;
324 virtual gfx::Rect GetRootWindowResizerRect() const OVERRIDE; 324 virtual gfx::Rect GetRootWindowResizerRect() const OVERRIDE;
325 virtual void RenderViewCreated(RenderViewHost* render_view_host) OVERRIDE; 325 virtual void RenderViewCreated(RenderViewHost* render_view_host) OVERRIDE;
326 virtual void RenderViewReady(RenderViewHost* render_view_host) OVERRIDE; 326 virtual void RenderViewReady(RenderViewHost* render_view_host) OVERRIDE;
327 virtual void RenderViewTerminated(RenderViewHost* render_view_host, 327 virtual void RenderViewTerminated(RenderViewHost* render_view_host,
328 base::TerminationStatus status, 328 base::TerminationStatus status,
329 int error_code) OVERRIDE; 329 int error_code) OVERRIDE;
330 virtual void RenderViewDeleted(RenderViewHost* render_view_host) OVERRIDE; 330 virtual void RenderViewDeleted(RenderViewHost* render_view_host) OVERRIDE;
331 virtual void DidNavigate( 331 virtual void DidNavigate(
332 RenderViewHost* render_view_host, 332 RenderViewHost* render_view_host,
333 const ViewHostMsg_FrameNavigate_Params& params) OVERRIDE; 333 const FrameHostMsg_DidCommitProvisionalLoad_Params& params) OVERRIDE;
334 virtual void UpdateState(RenderViewHost* render_view_host, 334 virtual void UpdateState(RenderViewHost* render_view_host,
335 int32 page_id, 335 int32 page_id,
336 const PageState& page_state) OVERRIDE; 336 const PageState& page_state) OVERRIDE;
337 virtual void UpdateTitle(RenderViewHost* render_view_host, 337 virtual void UpdateTitle(RenderViewHost* render_view_host,
338 int32 page_id, 338 int32 page_id,
339 const base::string16& title, 339 const base::string16& title,
340 base::i18n::TextDirection title_direction) OVERRIDE; 340 base::i18n::TextDirection title_direction) OVERRIDE;
341 virtual void UpdateEncoding(RenderViewHost* render_view_host, 341 virtual void UpdateEncoding(RenderViewHost* render_view_host,
342 const std::string& encoding) OVERRIDE; 342 const std::string& encoding) OVERRIDE;
343 virtual void UpdateTargetURL(int32 page_id, const GURL& url) OVERRIDE; 343 virtual void UpdateTargetURL(int32 page_id, const GURL& url) OVERRIDE;
(...skipping 113 matching lines...) Expand 10 before | Expand all | Expand 10 after
457 const GURL& validated_url, 457 const GURL& validated_url,
458 bool is_error_page, 458 bool is_error_page,
459 bool is_iframe_srcdoc) OVERRIDE; 459 bool is_iframe_srcdoc) OVERRIDE;
460 virtual void DidFailProvisionalLoadWithError( 460 virtual void DidFailProvisionalLoadWithError(
461 RenderFrameHostImpl* render_frame_host, 461 RenderFrameHostImpl* render_frame_host,
462 const FrameHostMsg_DidFailProvisionalLoadWithError_Params& params) 462 const FrameHostMsg_DidFailProvisionalLoadWithError_Params& params)
463 OVERRIDE; 463 OVERRIDE;
464 virtual void DidRedirectProvisionalLoad( 464 virtual void DidRedirectProvisionalLoad(
465 RenderFrameHostImpl* render_frame_host, 465 RenderFrameHostImpl* render_frame_host,
466 const GURL& validated_target_url) OVERRIDE; 466 const GURL& validated_target_url) OVERRIDE;
467 virtual void DidCommitProvisionalLoad(
468 int64 frame_id,
469 const base::string16& frame_unique_name,
470 bool is_main_frame,
471 const GURL& url,
472 PageTransition transition_type,
473 RenderFrameHostImpl* render_frame_host) OVERRIDE;
474 virtual void DidNavigateMainFramePostCommit(
475 const LoadCommittedDetails& details,
476 const FrameHostMsg_DidCommitProvisionalLoad_Params& params) OVERRIDE;
477 virtual void DidNavigateAnyFramePostCommit(
478 RenderFrameHostImpl* render_frame_host,
479 const LoadCommittedDetails& details,
480 const FrameHostMsg_DidCommitProvisionalLoad_Params& params) OVERRIDE;
481 virtual void SetMainFrameMimeType(const std::string& mime_type) OVERRIDE;
482 virtual bool CanOverscrollContent() OVERRIDE;
467 virtual void NotifyChangedNavigationState( 483 virtual void NotifyChangedNavigationState(
468 InvalidateTypes changed_flags) OVERRIDE; 484 InvalidateTypes changed_flags) OVERRIDE;
469 virtual void AboutToNavigateRenderFrame( 485 virtual void AboutToNavigateRenderFrame(
470 RenderFrameHostImpl* render_frame_host) OVERRIDE; 486 RenderFrameHostImpl* render_frame_host) OVERRIDE;
471 virtual void DidStartNavigationToPendingEntry( 487 virtual void DidStartNavigationToPendingEntry(
472 RenderFrameHostImpl* render_frame_host, 488 RenderFrameHostImpl* render_frame_host,
473 const GURL& url, 489 const GURL& url,
474 NavigationController::ReloadType reload_type) OVERRIDE; 490 NavigationController::ReloadType reload_type) OVERRIDE;
475 491
476 // RenderWidgetHostDelegate -------------------------------------------------- 492 // RenderWidgetHostDelegate --------------------------------------------------
(...skipping 262 matching lines...) Expand 10 before | Expand all | Expand 10 after
739 void SetNotWaitingForResponse() { waiting_for_response_ = false; } 755 void SetNotWaitingForResponse() { waiting_for_response_ = false; }
740 756
741 // Navigation helpers -------------------------------------------------------- 757 // Navigation helpers --------------------------------------------------------
742 // 758 //
743 // These functions are helpers for Navigate() and DidNavigate(). 759 // These functions are helpers for Navigate() and DidNavigate().
744 760
745 // Handles post-navigation tasks in DidNavigate AFTER the entry has been 761 // Handles post-navigation tasks in DidNavigate AFTER the entry has been
746 // committed to the navigation controller. Note that the navigation entry is 762 // committed to the navigation controller. Note that the navigation entry is
747 // not provided since it may be invalid/changed after being committed. The 763 // not provided since it may be invalid/changed after being committed. The
748 // current navigation entry is in the NavigationController at this point. 764 // current navigation entry is in the NavigationController at this point.
749 void DidNavigateMainFramePostCommit(
750 const LoadCommittedDetails& details,
751 const ViewHostMsg_FrameNavigate_Params& params);
752 void DidNavigateAnyFramePostCommit(
753 RenderViewHost* render_view_host,
754 const LoadCommittedDetails& details,
755 const ViewHostMsg_FrameNavigate_Params& params);
756 765
757 // Specifies whether the passed in URL should be assigned as the site of the 766 // Specifies whether the passed in URL should be assigned as the site of the
758 // current SiteInstance, if it does not yet have a site. 767 // current SiteInstance, if it does not yet have a site.
759 bool ShouldAssignSiteForURL(const GURL& url); 768 bool ShouldAssignSiteForURL(const GURL& url);
760 769
761 // If our controller was restored, update the max page ID associated with the 770 // If our controller was restored, update the max page ID associated with the
762 // given RenderViewHost to be larger than the number of restored entries. 771 // given RenderViewHost to be larger than the number of restored entries.
763 // This is called in CreateRenderView before any navigations in the RenderView 772 // This is called in CreateRenderView before any navigations in the RenderView
764 // have begun, to prevent any races in updating RenderView::next_page_id. 773 // have begun, to prevent any races in updating RenderView::next_page_id.
765 void UpdateMaxPageIDIfNecessary(RenderViewHost* rvh); 774 void UpdateMaxPageIDIfNecessary(RenderViewHost* rvh);
(...skipping 278 matching lines...) Expand 10 before | Expand all | Expand 10 after
1044 // Whether this WebContents is responsible for displaying a subframe in a 1053 // Whether this WebContents is responsible for displaying a subframe in a
1045 // different process from its parent page. 1054 // different process from its parent page.
1046 bool is_subframe_; 1055 bool is_subframe_;
1047 1056
1048 DISALLOW_COPY_AND_ASSIGN(WebContentsImpl); 1057 DISALLOW_COPY_AND_ASSIGN(WebContentsImpl);
1049 }; 1058 };
1050 1059
1051 } // namespace content 1060 } // namespace content
1052 1061
1053 #endif // CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_IMPL_H_ 1062 #endif // CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698