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

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

Issue 10316020: Remove WebContentsImpl::OnDidRedirectProvisionalLoad. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: ...and remove from WebContentsObserver. 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
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,
52 const GURL& opener_url) {}
53 virtual void DidCommitProvisionalLoadForFrame( 51 virtual void DidCommitProvisionalLoadForFrame(
54 int64 frame_id, 52 int64 frame_id,
55 bool is_main_frame, 53 bool is_main_frame,
56 const GURL& url, 54 const GURL& url,
57 PageTransition transition_type) {} 55 PageTransition transition_type) {}
58 virtual void DidFailProvisionalLoad(int64 frame_id, 56 virtual void DidFailProvisionalLoad(int64 frame_id,
59 bool is_main_frame, 57 bool is_main_frame,
60 const GURL& validated_url, 58 const GURL& validated_url,
61 int error_code, 59 int error_code,
62 const string16& error_description) {} 60 const string16& error_description) {}
(...skipping 80 matching lines...) Expand 10 before | Expand all | Expand 10 after
143 void WebContentsImplDestroyed(); 141 void WebContentsImplDestroyed();
144 142
145 WebContentsImpl* web_contents_; 143 WebContentsImpl* web_contents_;
146 144
147 DISALLOW_COPY_AND_ASSIGN(WebContentsObserver); 145 DISALLOW_COPY_AND_ASSIGN(WebContentsObserver);
148 }; 146 };
149 147
150 } // namespace content 148 } // namespace content
151 149
152 #endif // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_ 150 #endif // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698