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

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

Issue 596543002: Remove WebContentsObserver::ProvisionalChangeToMainFrameUrl and all the code supporting it. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@redirprerender
Patch Set: done Created 6 years, 3 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
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 484 matching lines...) Expand 10 before | Expand all | Expand 10 after
495 RenderFrameHostImpl* render_frame_host) OVERRIDE; 495 RenderFrameHostImpl* render_frame_host) OVERRIDE;
496 virtual void DidFailProvisionalLoadWithError( 496 virtual void DidFailProvisionalLoadWithError(
497 RenderFrameHostImpl* render_frame_host, 497 RenderFrameHostImpl* render_frame_host,
498 const FrameHostMsg_DidFailProvisionalLoadWithError_Params& params) 498 const FrameHostMsg_DidFailProvisionalLoadWithError_Params& params)
499 OVERRIDE; 499 OVERRIDE;
500 virtual void DidFailLoadWithError( 500 virtual void DidFailLoadWithError(
501 RenderFrameHostImpl* render_frame_host, 501 RenderFrameHostImpl* render_frame_host,
502 const GURL& url, 502 const GURL& url,
503 int error_code, 503 int error_code,
504 const base::string16& error_description) OVERRIDE; 504 const base::string16& error_description) OVERRIDE;
505 virtual void DidRedirectProvisionalLoad(
506 RenderFrameHostImpl* render_frame_host,
507 const GURL& validated_target_url) OVERRIDE;
508 virtual void DidCommitProvisionalLoad( 505 virtual void DidCommitProvisionalLoad(
509 RenderFrameHostImpl* render_frame_host, 506 RenderFrameHostImpl* render_frame_host,
510 const GURL& url, 507 const GURL& url,
511 ui::PageTransition transition_type) OVERRIDE; 508 ui::PageTransition transition_type) OVERRIDE;
512 virtual void DidNavigateMainFramePreCommit( 509 virtual void DidNavigateMainFramePreCommit(
513 bool navigation_is_within_page) OVERRIDE; 510 bool navigation_is_within_page) OVERRIDE;
514 virtual void DidNavigateMainFramePostCommit( 511 virtual void DidNavigateMainFramePostCommit(
515 const LoadCommittedDetails& details, 512 const LoadCommittedDetails& details,
516 const FrameHostMsg_DidCommitProvisionalLoad_Params& params) OVERRIDE; 513 const FrameHostMsg_DidCommitProvisionalLoad_Params& params) OVERRIDE;
517 virtual void DidNavigateAnyFramePostCommit( 514 virtual void DidNavigateAnyFramePostCommit(
(...skipping 724 matching lines...) Expand 10 before | Expand all | Expand 10 after
1242 AudioStreamMonitor audio_stream_monitor_; 1239 AudioStreamMonitor audio_stream_monitor_;
1243 1240
1244 base::WeakPtrFactory<WebContentsImpl> loading_weak_factory_; 1241 base::WeakPtrFactory<WebContentsImpl> loading_weak_factory_;
1245 1242
1246 DISALLOW_COPY_AND_ASSIGN(WebContentsImpl); 1243 DISALLOW_COPY_AND_ASSIGN(WebContentsImpl);
1247 }; 1244 };
1248 1245
1249 } // namespace content 1246 } // namespace content
1250 1247
1251 #endif // CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_IMPL_H_ 1248 #endif // CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_IMPL_H_
OLDNEW
« no previous file with comments | « content/browser/frame_host/render_frame_host_impl.cc ('k') | content/browser/web_contents/web_contents_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698