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

Unified Diff: content/browser/renderer_host/render_view_host_impl.cc

Issue 10073002: TabContents -> WebContentsImpl, part 11. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: missed a few Created 8 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/renderer_host/render_view_host_impl.cc
diff --git a/content/browser/renderer_host/render_view_host_impl.cc b/content/browser/renderer_host/render_view_host_impl.cc
index e31de0f182530c5e9c400978464681233c61a479..f33392587ba895e9c6f4699cbfc54ad47b7403db 100644
--- a/content/browser/renderer_host/render_view_host_impl.cc
+++ b/content/browser/renderer_host/render_view_host_impl.cc
@@ -74,7 +74,8 @@ using WebKit::WebPluginAction;
namespace {
-// Delay to wait on closing the tab for a beforeunload/unload handler to fire.
+// Delay to wait on closing the WebContents for a beforeunload/unload handler to
+// fire.
const int kUnloadTimeoutMS = 1000;
// Translate a WebKit text direction into a base::i18n one.
@@ -287,7 +288,7 @@ void RenderViewHostImpl::Navigate(const ViewMsg_Navigate_Params& params) {
if (navigations_suspended_) {
// Shouldn't be possible to have a second navigation while suspended, since
// navigations will only be suspended during a cross-site request. If a
- // second navigation occurs, TabContents will cancel this pending RVH
+ // second navigation occurs, WebContentsImpl will cancel this pending RVH
// create a new pending RVH.
DCHECK(!suspended_nav_message_.get());
suspended_nav_message_.reset(nav_message);
@@ -1478,7 +1479,7 @@ void RenderViewHostImpl::SetAltErrorPageURL(const GURL& url) {
void RenderViewHostImpl::ExitFullscreen() {
RejectMouseLockOrUnlockIfNecessary();
- // We need to notify the tab that its fullscreen state has changed. This
+ // We need to notify the contents that its fullscreen state has changed. This
// is done as part of the resize message.
WasResized();
}
« no previous file with comments | « content/browser/renderer_host/render_view_host_impl.h ('k') | content/browser/renderer_host/render_view_host_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698