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

Side by Side Diff: content/browser/frame_host/navigation_entry_impl.h

Issue 1156733003: Move PageState to FrameNavigationEntry. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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_FRAME_HOST_NAVIGATION_ENTRY_IMPL_H_ 5 #ifndef CONTENT_BROWSER_FRAME_HOST_NAVIGATION_ENTRY_IMPL_H_
6 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATION_ENTRY_IMPL_H_ 6 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATION_ENTRY_IMPL_H_
7 7
8 #include "base/basictypes.h" 8 #include "base/basictypes.h"
9 #include "base/memory/ref_counted.h" 9 #include "base/memory/ref_counted.h"
10 #include "base/memory/scoped_vector.h" 10 #include "base/memory/scoped_vector.h"
(...skipping 147 matching lines...) Expand 10 before | Expand all | Expand 10 after
158 158
159 // Finds the TreeNode associated with |frame_tree_node_id| to add or update 159 // Finds the TreeNode associated with |frame_tree_node_id| to add or update
160 // its FrameNavigationEntry. A new FrameNavigationEntry is added if none 160 // its FrameNavigationEntry. A new FrameNavigationEntry is added if none
161 // exists, or else the existing one (which might be shared with other 161 // exists, or else the existing one (which might be shared with other
162 // NavigationEntries) is updated with the given parameters. 162 // NavigationEntries) is updated with the given parameters.
163 // Does nothing if there is no entry already and |url| is about:blank, since 163 // Does nothing if there is no entry already and |url| is about:blank, since
164 // that does not count as a real commit. 164 // that does not count as a real commit.
165 void AddOrUpdateFrameEntry(FrameTreeNode* frame_tree_node, 165 void AddOrUpdateFrameEntry(FrameTreeNode* frame_tree_node,
166 SiteInstanceImpl* site_instance, 166 SiteInstanceImpl* site_instance,
167 const GURL& url, 167 const GURL& url,
168 const Referrer& referrer); 168 const Referrer& referrer,
169 const PageState& page_state);
169 170
170 // Returns whether this entry has a FrameNavigationEntry for the given 171 // Returns whether this entry has a FrameNavigationEntry for the given
171 // |frame_tree_node|. 172 // |frame_tree_node|.
172 bool HasFrameEntry(FrameTreeNode* frame_tree_node) const; 173 bool HasFrameEntry(FrameTreeNode* frame_tree_node) const;
173 174
174 void set_unique_id(int unique_id) { 175 void set_unique_id(int unique_id) {
175 unique_id_ = unique_id; 176 unique_id_ = unique_id;
176 } 177 }
177 178
178 // The SiteInstance represents which pages must share processes. This is a 179 // The SiteInstance represents which pages must share processes. This is a
(...skipping 158 matching lines...) Expand 10 before | Expand all | Expand 10 after
337 338
338 // See the accessors above for descriptions. 339 // See the accessors above for descriptions.
339 int unique_id_; 340 int unique_id_;
340 // TODO(creis): Persist bindings_. http://crbug.com/173672. 341 // TODO(creis): Persist bindings_. http://crbug.com/173672.
341 int bindings_; 342 int bindings_;
342 PageType page_type_; 343 PageType page_type_;
343 GURL virtual_url_; 344 GURL virtual_url_;
344 bool update_virtual_url_with_url_; 345 bool update_virtual_url_with_url_;
345 base::string16 title_; 346 base::string16 title_;
346 FaviconStatus favicon_; 347 FaviconStatus favicon_;
347 PageState page_state_;
348 int32 page_id_; 348 int32 page_id_;
349 SSLStatus ssl_; 349 SSLStatus ssl_;
350 ui::PageTransition transition_type_; 350 ui::PageTransition transition_type_;
351 GURL user_typed_url_; 351 GURL user_typed_url_;
352 bool has_post_data_; 352 bool has_post_data_;
353 int64 post_id_; 353 int64 post_id_;
354 RestoreType restore_type_; 354 RestoreType restore_type_;
355 GURL original_request_url_; 355 GURL original_request_url_;
356 bool is_overriding_user_agent_; 356 bool is_overriding_user_agent_;
357 base::Time timestamp_; 357 base::Time timestamp_;
(...skipping 90 matching lines...) Expand 10 before | Expand all | Expand 10 after
448 // persisted, unless specific data is taken out/put back in at save/restore 448 // persisted, unless specific data is taken out/put back in at save/restore
449 // time (see TabNavigation for an example of this). 449 // time (see TabNavigation for an example of this).
450 std::map<std::string, base::string16> extra_data_; 450 std::map<std::string, base::string16> extra_data_;
451 451
452 DISALLOW_COPY_AND_ASSIGN(NavigationEntryImpl); 452 DISALLOW_COPY_AND_ASSIGN(NavigationEntryImpl);
453 }; 453 };
454 454
455 } // namespace content 455 } // namespace content
456 456
457 #endif // CONTENT_BROWSER_FRAME_HOST_NAVIGATION_ENTRY_IMPL_H_ 457 #endif // CONTENT_BROWSER_FRAME_HOST_NAVIGATION_ENTRY_IMPL_H_
OLDNEW
« no previous file with comments | « content/browser/frame_host/navigation_controller_impl.cc ('k') | content/browser/frame_host/navigation_entry_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698