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

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

Issue 232463007: Don't leave aborted URLs in the omnibox unless we're on a new tab. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 6 years, 8 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 #include "content/browser/web_contents/web_contents_impl.h" 5 #include "content/browser/web_contents/web_contents_impl.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/debug/trace_event.h" 10 #include "base/debug/trace_event.h"
(...skipping 2354 matching lines...) Expand 10 before | Expand all | Expand 10 after
2365 FOR_EACH_OBSERVER(WebContentsObserver, observers_, 2365 FOR_EACH_OBSERVER(WebContentsObserver, observers_,
2366 DidOpenRequestedURL(new_contents, 2366 DidOpenRequestedURL(new_contents,
2367 params.url, 2367 params.url,
2368 params.referrer, 2368 params.referrer,
2369 params.disposition, 2369 params.disposition,
2370 params.transition, 2370 params.transition,
2371 source_render_frame_id)); 2371 source_render_frame_id));
2372 } 2372 }
2373 } 2373 }
2374 2374
2375 bool WebContentsImpl::ShouldPreserveAbortedURLs() {
2376 return delegate_->ShouldPreserveAbortedURLs(this);
2377 }
2378
2375 void WebContentsImpl::DidRedirectProvisionalLoad( 2379 void WebContentsImpl::DidRedirectProvisionalLoad(
2376 RenderFrameHostImpl* render_frame_host, 2380 RenderFrameHostImpl* render_frame_host,
2377 const GURL& validated_target_url) { 2381 const GURL& validated_target_url) {
2378 // Notify observers about the provisional change in the main frame URL. 2382 // Notify observers about the provisional change in the main frame URL.
2379 FOR_EACH_OBSERVER( 2383 FOR_EACH_OBSERVER(
2380 WebContentsObserver, 2384 WebContentsObserver,
2381 observers_, 2385 observers_,
2382 ProvisionalChangeToMainFrameUrl(validated_target_url, 2386 ProvisionalChangeToMainFrameUrl(validated_target_url,
2383 render_frame_host)); 2387 render_frame_host));
2384 } 2388 }
(...skipping 1496 matching lines...) Expand 10 before | Expand all | Expand 10 after
3881 3885
3882 void WebContentsImpl::OnPreferredSizeChanged(const gfx::Size& old_size) { 3886 void WebContentsImpl::OnPreferredSizeChanged(const gfx::Size& old_size) {
3883 if (!delegate_) 3887 if (!delegate_)
3884 return; 3888 return;
3885 const gfx::Size new_size = GetPreferredSize(); 3889 const gfx::Size new_size = GetPreferredSize();
3886 if (new_size != old_size) 3890 if (new_size != old_size)
3887 delegate_->UpdatePreferredSize(this, new_size); 3891 delegate_->UpdatePreferredSize(this, new_size);
3888 } 3892 }
3889 3893
3890 } // namespace content 3894 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/web_contents/web_contents_impl.h ('k') | content/public/browser/web_contents_delegate.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698