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

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

Issue 24649002: Clean up a few more unused globals. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: thestig comment Created 7 years, 2 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 133 matching lines...) Expand 10 before | Expand all | Expand 10 after
144 // - The pending renderer sends a FrameNavigate message that invokes the 144 // - The pending renderer sends a FrameNavigate message that invokes the
145 // DidNavigate method. This replaces the current RVH with the 145 // DidNavigate method. This replaces the current RVH with the
146 // pending RVH. 146 // pending RVH.
147 // - The previous renderer is kept swapped out in RenderViewHostManager in case 147 // - The previous renderer is kept swapped out in RenderViewHostManager in case
148 // the user goes back. The process only stays live if another tab is using 148 // the user goes back. The process only stays live if another tab is using
149 // it, but if so, the existing frame relationships will be maintained. 149 // it, but if so, the existing frame relationships will be maintained.
150 150
151 namespace content { 151 namespace content {
152 namespace { 152 namespace {
153 153
154 // Amount of time we wait between when a key event is received and the renderer
155 // is queried for its state and pushed to the NavigationEntry.
156 const int kQueryStateDelay = 5000;
157
158 const int kSyncWaitDelay = 40;
159
160 const char kDotGoogleDotCom[] = ".google.com"; 154 const char kDotGoogleDotCom[] = ".google.com";
161 155
162 base::LazyInstance<std::vector<WebContents::CreatedCallback> > 156 base::LazyInstance<std::vector<WebContents::CreatedCallback> >
163 g_created_callbacks = LAZY_INSTANCE_INITIALIZER; 157 g_created_callbacks = LAZY_INSTANCE_INITIALIZER;
164 158
165 static int StartDownload(content::RenderViewHost* rvh, 159 static int StartDownload(content::RenderViewHost* rvh,
166 const GURL& url, 160 const GURL& url,
167 bool is_favicon, 161 bool is_favicon,
168 uint32_t max_bitmap_size) { 162 uint32_t max_bitmap_size) {
169 static int g_next_image_download_id = 0; 163 static int g_next_image_download_id = 0;
(...skipping 3695 matching lines...) Expand 10 before | Expand all | Expand 10 after
3865 gfx::Size WebContentsImpl::GetSizeForNewRenderView() const { 3859 gfx::Size WebContentsImpl::GetSizeForNewRenderView() const {
3866 gfx::Size size; 3860 gfx::Size size;
3867 if (delegate_) 3861 if (delegate_)
3868 size = delegate_->GetSizeForNewRenderView(this); 3862 size = delegate_->GetSizeForNewRenderView(this);
3869 if (size.IsEmpty()) 3863 if (size.IsEmpty())
3870 size = view_->GetContainerSize(); 3864 size = view_->GetContainerSize();
3871 return size; 3865 return size;
3872 } 3866 }
3873 3867
3874 } // namespace content 3868 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698