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

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

Issue 297973002: Navigation transitions: Block first response until after transitions have run. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Changes from review. Created 6 years, 6 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_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/kill.h" 8 #include "base/process/kill.h"
9 #include "base/process/process_handle.h" 9 #include "base/process/process_handle.h"
10 #include "content/common/content_export.h" 10 #include "content/common/content_export.h"
(...skipping 228 matching lines...) Expand 10 before | Expand all | Expand 10 after
239 239
240 // This method is invoked when the renderer has completed its first paint 240 // This method is invoked when the renderer has completed its first paint
241 // after a non-empty layout. 241 // after a non-empty layout.
242 virtual void DidFirstVisuallyNonEmptyPaint() {} 242 virtual void DidFirstVisuallyNonEmptyPaint() {}
243 243
244 // These two methods correspond to the points in time when the spinner of the 244 // These two methods correspond to the points in time when the spinner of the
245 // tab starts and stops spinning. 245 // tab starts and stops spinning.
246 virtual void DidStartLoading(RenderViewHost* render_view_host) {} 246 virtual void DidStartLoading(RenderViewHost* render_view_host) {}
247 virtual void DidStopLoading(RenderViewHost* render_view_host) {} 247 virtual void DidStopLoading(RenderViewHost* render_view_host) {}
248 248
249 // This method is invoked when the request is deferred immediately after
250 // receiving response headers.
251 virtual void DidDeferAfterResponseStarted() {}
jam 2014/06/10 05:54:56 similarly, since this notification is only consume
shatch 2014/06/10 20:20:24 I thought this was needed this to make its way to
jam 2014/06/11 02:00:21 WebContentsImpl can call the ContentViewCoreImpl m
shatch 2014/06/14 00:41:37 Ah great, thanks! Moved this to call ContentViewCo
252
249 // When WebContents::Stop() is called, the WebContents stops loading and then 253 // When WebContents::Stop() is called, the WebContents stops loading and then
250 // invokes this method. If there are ongoing navigations, their respective 254 // invokes this method. If there are ongoing navigations, their respective
251 // failure methods will also be invoked. 255 // failure methods will also be invoked.
252 virtual void NavigationStopped() {} 256 virtual void NavigationStopped() {}
253 257
254 // This indicates that the next navigation was triggered by a user gesture. 258 // This indicates that the next navigation was triggered by a user gesture.
255 virtual void DidGetUserGesture() {} 259 virtual void DidGetUserGesture() {}
256 260
257 // This method is invoked when a RenderViewHost of this WebContents was 261 // This method is invoked when a RenderViewHost of this WebContents was
258 // configured to ignore UI events, and an UI event took place. 262 // configured to ignore UI events, and an UI event took place.
(...skipping 106 matching lines...) Expand 10 before | Expand all | Expand 10 after
365 void ResetWebContents(); 369 void ResetWebContents();
366 370
367 WebContentsImpl* web_contents_; 371 WebContentsImpl* web_contents_;
368 372
369 DISALLOW_COPY_AND_ASSIGN(WebContentsObserver); 373 DISALLOW_COPY_AND_ASSIGN(WebContentsObserver);
370 }; 374 };
371 375
372 } // namespace content 376 } // namespace content
373 377
374 #endif // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_ 378 #endif // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_
OLDNEW
« content/public/browser/web_contents.h ('K') | « content/public/browser/web_contents.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698