Add a target_frame_id to the history serialization format.

content/public/renderer/history_item_serialization.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/public/renderer/history_item_serialization.h"
 
 #include "content/common/page_state_serialization.h"
 #include "content/public/common/page_state.h"
 #include "third_party/WebKit/public/platform/WebHTTPBody.h"
 #include "third_party/WebKit/public/platform/WebPoint.h"
@@ skipping 73 matching lines @@
   state->url_string = item.urlString();
   state->original_url_string = item.originalURLString();
   state->referrer = item.referrer();
   state->target = item.target();
   if (!item.stateObject().isNull())
     state->state_object = item.stateObject().toString();
   state->scroll_offset = item.scrollOffset();
   state->item_sequence_number = item.itemSequenceNumber();
   state->document_sequence_number =
       item.documentSequenceNumber();
+  state->target_frame_id = item.targetFrameID();
   state->page_scale_factor = item.pageScaleFactor();
   ToNullableString16Vector(item.documentState(), &state->document_state);
 
   state->http_body.http_content_type = item.httpContentType();
   const WebHTTPBody& http_body = item.httpBody();
   if (!(state->http_body.is_null = http_body.isNull())) {
     state->http_body.identifier = http_body.identifier();
     state->http_body.elements.resize(http_body.elementCount());
     for (size_t i = 0; i < http_body.elementCount(); ++i) {
       WebHTTPBody::Element element;
@@ skipping 27 matching lines @@
   item->setScrollOffset(state.scroll_offset);
   item->setPageScaleFactor(state.page_scale_factor);
 
   // These values are generated at WebHistoryItem construction time, and we
   // only want to override those new values with old values if the old values
   // are defined.  A value of 0 means undefined in this context.
   if (state.item_sequence_number)
     item->setItemSequenceNumber(state.item_sequence_number);
   if (state.document_sequence_number)
     item->setDocumentSequenceNumber(state.document_sequence_number);
+  if (state.target_frame_id)

[darin (slow to review), 2013/10/29 23:05:50]

The above comment does not apply to target frame ID. Setting a value of 0 seems
OK for frame ID. We should probably treat this property more like
setPageScaleFactor, right?

[Nate Chapin, 2013/10/29 23:16:26]

That's probably true. Will fix.

+    item->setTargetFrameID(state.target_frame_id);
 
   item->setHTTPContentType(state.http_body.http_content_type);
   if (!state.http_body.is_null) {
     WebHTTPBody http_body;
     http_body.initialize();
     http_body.setIdentifier(state.http_body.identifier);
     for (size_t i = 0; i < state.http_body.elements.size(); ++i)
       AppendHTTPBodyElement(state.http_body.elements[i], &http_body);
     item->setHTTPBody(http_body);
   }
@@ skipping 33 matching lines @@
 
   WebHistoryItem item;
   item.initialize();
   if (!RecursivelyGenerateHistoryItem(state.top, &item))
     return WebHistoryItem();
 
   return item;
 }
 
 }  // namespace content