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

Side by Side Diff: content/browser/renderer_host/render_view_host_impl.h

Issue 1258273006: Remove page id from title and state updating. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@switch
Patch Set: comment nit Created 4 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
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_RENDERER_HOST_RENDER_VIEW_HOST_IMPL_H_ 5 #ifndef CONTENT_BROWSER_RENDERER_HOST_RENDER_VIEW_HOST_IMPL_H_
6 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_VIEW_HOST_IMPL_H_ 6 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_VIEW_HOST_IMPL_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 #include <stdint.h> 9 #include <stdint.h>
10 10
(...skipping 269 matching lines...) Expand 10 before | Expand all | Expand 10 after
280 const NativeWebKeyboardEvent& key_event) override; 280 const NativeWebKeyboardEvent& key_event) override;
281 281
282 // IPC message handlers. 282 // IPC message handlers.
283 void OnShowView(int route_id, 283 void OnShowView(int route_id,
284 WindowOpenDisposition disposition, 284 WindowOpenDisposition disposition,
285 const gfx::Rect& initial_rect, 285 const gfx::Rect& initial_rect,
286 bool user_gesture); 286 bool user_gesture);
287 void OnShowWidget(int route_id, const gfx::Rect& initial_rect); 287 void OnShowWidget(int route_id, const gfx::Rect& initial_rect);
288 void OnShowFullscreenWidget(int route_id); 288 void OnShowFullscreenWidget(int route_id);
289 void OnRenderProcessGone(int status, int error_code); 289 void OnRenderProcessGone(int status, int error_code);
290 void OnUpdateState(int32_t page_id, const PageState& state); 290 void OnUpdateState(const PageState& state);
291 void OnUpdateTargetURL(const GURL& url); 291 void OnUpdateTargetURL(const GURL& url);
292 void OnClose(); 292 void OnClose();
293 void OnRequestMove(const gfx::Rect& pos); 293 void OnRequestMove(const gfx::Rect& pos);
294 void OnDocumentAvailableInMainFrame(bool uses_temporary_zoom_level); 294 void OnDocumentAvailableInMainFrame(bool uses_temporary_zoom_level);
295 void OnDidContentsPreferredSizeChange(const gfx::Size& new_size); 295 void OnDidContentsPreferredSizeChange(const gfx::Size& new_size);
296 void OnPasteFromSelectionClipboard(); 296 void OnPasteFromSelectionClipboard();
297 void OnRouteCloseEvent(); 297 void OnRouteCloseEvent();
298 void OnStartDragging(const DropData& drop_data, 298 void OnStartDragging(const DropData& drop_data,
299 blink::WebDragOperationsMask operations_allowed, 299 blink::WebDragOperationsMask operations_allowed,
300 const SkBitmap& bitmap, 300 const SkBitmap& bitmap,
(...skipping 51 matching lines...) Expand 10 before | Expand all | Expand 10 after
352 352
353 // The SiteInstance associated with this RenderViewHost. All pages drawn 353 // The SiteInstance associated with this RenderViewHost. All pages drawn
354 // in this RenderViewHost are part of this SiteInstance. Cannot change 354 // in this RenderViewHost are part of this SiteInstance. Cannot change
355 // over time. 355 // over time.
356 scoped_refptr<SiteInstanceImpl> instance_; 356 scoped_refptr<SiteInstanceImpl> instance_;
357 357
358 // A bitwise OR of bindings types that have been enabled for this RenderView. 358 // A bitwise OR of bindings types that have been enabled for this RenderView.
359 // See BindingsPolicy for details. 359 // See BindingsPolicy for details.
360 int enabled_bindings_; 360 int enabled_bindings_;
361 361
362 // The most recent page ID we've heard from the renderer process. This is
363 // used as context when other session history related IPCs arrive.
364 // TODO(creis): Allocate this in WebContents/NavigationController instead.
365 int32_t page_id_;
366
367 // Tracks whether this RenderViewHost is in an active state. False if the 362 // Tracks whether this RenderViewHost is in an active state. False if the
368 // main frame is pending swap out, pending deletion, or swapped out, because 363 // main frame is pending swap out, pending deletion, or swapped out, because
369 // it is not visible to the user in any of these cases. 364 // it is not visible to the user in any of these cases.
370 bool is_active_; 365 bool is_active_;
371 366
372 // Tracks whether the main frame RenderFrameHost is swapped out. Unlike 367 // Tracks whether the main frame RenderFrameHost is swapped out. Unlike
373 // is_active_, this is false when the frame is pending swap out or deletion. 368 // is_active_, this is false when the frame is pending swap out or deletion.
374 // TODO(creis): Remove this when we no longer use swappedout://. 369 // TODO(creis): Remove this when we no longer use swappedout://.
375 // See http://crbug.com/357747. 370 // See http://crbug.com/357747.
376 bool is_swapped_out_; 371 bool is_swapped_out_;
(...skipping 25 matching lines...) Expand all
402 bool render_view_ready_on_process_launch_; 397 bool render_view_ready_on_process_launch_;
403 398
404 base::WeakPtrFactory<RenderViewHostImpl> weak_factory_; 399 base::WeakPtrFactory<RenderViewHostImpl> weak_factory_;
405 400
406 DISALLOW_COPY_AND_ASSIGN(RenderViewHostImpl); 401 DISALLOW_COPY_AND_ASSIGN(RenderViewHostImpl);
407 }; 402 };
408 403
409 } // namespace content 404 } // namespace content
410 405
411 #endif // CONTENT_BROWSER_RENDERER_HOST_RENDER_VIEW_HOST_IMPL_H_ 406 #endif // CONTENT_BROWSER_RENDERER_HOST_RENDER_VIEW_HOST_IMPL_H_
OLDNEW
« no previous file with comments | « content/browser/renderer_host/render_view_host_delegate.h ('k') | content/browser/renderer_host/render_view_host_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698