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/frame_host/frame_navigation_entry.cc

Issue 1407853005: OOPIF: Add frame_unique_name to FrameNavigationEntry. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update comments Created 5 years, 1 month 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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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/frame_host/frame_navigation_entry.h" 5 #include "content/browser/frame_host/frame_navigation_entry.h"
6 6
7 namespace content { 7 namespace content {
8 8
9 FrameNavigationEntry::FrameNavigationEntry(int frame_tree_node_id) 9 FrameNavigationEntry::FrameNavigationEntry(int frame_tree_node_id)
10 : frame_tree_node_id_(frame_tree_node_id), 10 : frame_tree_node_id_(frame_tree_node_id),
11 item_sequence_number_(-1), 11 item_sequence_number_(-1),
12 document_sequence_number_(-1) { 12 document_sequence_number_(-1) {
13 } 13 }
14 14
15 FrameNavigationEntry::FrameNavigationEntry(int frame_tree_node_id, 15 FrameNavigationEntry::FrameNavigationEntry(int frame_tree_node_id,
16 std::string target,
16 int64 item_sequence_number, 17 int64 item_sequence_number,
17 int64 document_sequence_number, 18 int64 document_sequence_number,
18 SiteInstanceImpl* site_instance, 19 SiteInstanceImpl* site_instance,
19 const GURL& url, 20 const GURL& url,
20 const Referrer& referrer) 21 const Referrer& referrer)
21 : frame_tree_node_id_(frame_tree_node_id), 22 : frame_tree_node_id_(frame_tree_node_id),
23 target_(target),
22 item_sequence_number_(item_sequence_number), 24 item_sequence_number_(item_sequence_number),
23 document_sequence_number_(document_sequence_number), 25 document_sequence_number_(document_sequence_number),
24 site_instance_(site_instance), 26 site_instance_(site_instance),
25 url_(url), 27 url_(url),
26 referrer_(referrer) { 28 referrer_(referrer) {
27 } 29 }
28 30
29 FrameNavigationEntry::~FrameNavigationEntry() { 31 FrameNavigationEntry::~FrameNavigationEntry() {
30 } 32 }
31 33
32 FrameNavigationEntry* FrameNavigationEntry::Clone() const { 34 FrameNavigationEntry* FrameNavigationEntry::Clone() const {
33 FrameNavigationEntry* copy = new FrameNavigationEntry(frame_tree_node_id_); 35 FrameNavigationEntry* copy = new FrameNavigationEntry(frame_tree_node_id_);
34 copy->UpdateEntry(item_sequence_number_, document_sequence_number_, 36 copy->UpdateEntry(target_, item_sequence_number_, document_sequence_number_,
35 site_instance_.get(), url_, referrer_, page_state_); 37 site_instance_.get(), url_, referrer_, page_state_);
36 return copy; 38 return copy;
37 } 39 }
38 40
39 void FrameNavigationEntry::UpdateEntry(int64 item_sequence_number, 41 void FrameNavigationEntry::UpdateEntry(std::string target,
42 int64 item_sequence_number,
40 int64 document_sequence_number, 43 int64 document_sequence_number,
41 SiteInstanceImpl* site_instance, 44 SiteInstanceImpl* site_instance,
42 const GURL& url, 45 const GURL& url,
43 const Referrer& referrer, 46 const Referrer& referrer,
44 const PageState& page_state) { 47 const PageState& page_state) {
48 target_ = target;
45 item_sequence_number_ = item_sequence_number; 49 item_sequence_number_ = item_sequence_number;
46 document_sequence_number_ = document_sequence_number; 50 document_sequence_number_ = document_sequence_number;
47 site_instance_ = site_instance; 51 site_instance_ = site_instance;
48 url_ = url; 52 url_ = url;
49 referrer_ = referrer; 53 referrer_ = referrer;
50 page_state_ = page_state; 54 page_state_ = page_state;
51 } 55 }
52 56
53 void FrameNavigationEntry::set_item_sequence_number( 57 void FrameNavigationEntry::set_item_sequence_number(
54 int64 item_sequence_number) { 58 int64 item_sequence_number) {
55 // Once assigned, the item sequence number shouldn't change. 59 // Once assigned, the item sequence number shouldn't change.
56 DCHECK(item_sequence_number_ == -1 || 60 DCHECK(item_sequence_number_ == -1 ||
57 item_sequence_number_ == item_sequence_number); 61 item_sequence_number_ == item_sequence_number);
58 item_sequence_number_ = item_sequence_number; 62 item_sequence_number_ = item_sequence_number;
59 } 63 }
60 64
61 void FrameNavigationEntry::set_document_sequence_number( 65 void FrameNavigationEntry::set_document_sequence_number(
62 int64 document_sequence_number) { 66 int64 document_sequence_number) {
63 // Once assigned, the document sequence number shouldn't change. 67 // Once assigned, the document sequence number shouldn't change.
64 DCHECK(document_sequence_number_ == -1 || 68 DCHECK(document_sequence_number_ == -1 ||
65 document_sequence_number_ == document_sequence_number); 69 document_sequence_number_ == document_sequence_number);
66 document_sequence_number_ = document_sequence_number; 70 document_sequence_number_ = document_sequence_number;
67 } 71 }
68 72
69 } // namespace content 73 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698