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

Side by Side Diff: content/public/browser/web_contents_observer.h

Issue 10392072: Pass the render view host with all provisional load related callbacks on the web contents observer (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 8 years, 7 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
« no previous file with comments | « content/browser/web_contents/web_contents_impl.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_ 5 #ifndef CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_
6 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_ 6 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_
7 7
8 #include "base/process_util.h" 8 #include "base/process_util.h"
9 #include "content/common/content_export.h" 9 #include "content/common/content_export.h"
10 #include "content/public/browser/navigation_controller.h" 10 #include "content/public/browser/navigation_controller.h"
(...skipping 30 matching lines...) Expand all
41 const LoadCommittedDetails& details, 41 const LoadCommittedDetails& details,
42 const FrameNavigateParams& params) {} 42 const FrameNavigateParams& params) {}
43 // |render_view_host| is the RenderViewHost for which the provisional load is 43 // |render_view_host| is the RenderViewHost for which the provisional load is
44 // happening. 44 // happening.
45 virtual void DidStartProvisionalLoadForFrame( 45 virtual void DidStartProvisionalLoadForFrame(
46 int64 frame_id, 46 int64 frame_id,
47 bool is_main_frame, 47 bool is_main_frame,
48 const GURL& validated_url, 48 const GURL& validated_url,
49 bool is_error_page, 49 bool is_error_page,
50 RenderViewHost* render_view_host) {} 50 RenderViewHost* render_view_host) {}
51 virtual void ProvisionalChangeToMainFrameUrl(const GURL& url, 51 virtual void ProvisionalChangeToMainFrameUrl(
52 const GURL& opener_url) {} 52 const GURL& url,
53 const GURL& opener_url,
54 RenderViewHost* render_view_host) {}
53 virtual void DidCommitProvisionalLoadForFrame( 55 virtual void DidCommitProvisionalLoadForFrame(
54 int64 frame_id, 56 int64 frame_id,
55 bool is_main_frame, 57 bool is_main_frame,
56 const GURL& url, 58 const GURL& url,
57 PageTransition transition_type) {} 59 PageTransition transition_type,
60 RenderViewHost* render_view_host) {}
58 virtual void DidFailProvisionalLoad(int64 frame_id, 61 virtual void DidFailProvisionalLoad(int64 frame_id,
59 bool is_main_frame, 62 bool is_main_frame,
60 const GURL& validated_url, 63 const GURL& validated_url,
61 int error_code, 64 int error_code,
62 const string16& error_description) {} 65 const string16& error_description,
66 RenderViewHost* render_view_host) {}
63 virtual void DocumentAvailableInMainFrame() {} 67 virtual void DocumentAvailableInMainFrame() {}
64 virtual void DocumentLoadedInFrame(int64 frame_id) {} 68 virtual void DocumentLoadedInFrame(int64 frame_id) {}
65 virtual void DidFinishLoad(int64 frame_id, 69 virtual void DidFinishLoad(int64 frame_id,
66 const GURL& validated_url, 70 const GURL& validated_url,
67 bool is_main_frame) {} 71 bool is_main_frame) {}
68 virtual void DidFailLoad(int64 frame_id, 72 virtual void DidFailLoad(int64 frame_id,
69 const GURL& validated_url, 73 const GURL& validated_url,
70 bool is_main_frame, 74 bool is_main_frame,
71 int error_code, 75 int error_code,
72 const string16& error_description) {} 76 const string16& error_description) {}
(...skipping 70 matching lines...) Expand 10 before | Expand all | Expand 10 after
143 void WebContentsImplDestroyed(); 147 void WebContentsImplDestroyed();
144 148
145 WebContentsImpl* web_contents_; 149 WebContentsImpl* web_contents_;
146 150
147 DISALLOW_COPY_AND_ASSIGN(WebContentsObserver); 151 DISALLOW_COPY_AND_ASSIGN(WebContentsObserver);
148 }; 152 };
149 153
150 } // namespace content 154 } // namespace content
151 155
152 #endif // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_ 156 #endif // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_
OLDNEW
« no previous file with comments | « content/browser/web_contents/web_contents_impl.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698