Refactor NavigationState to use NavigationParams

content/renderer/navigation_state_impl.cc

+// Copyright 2015 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/renderer/navigation_state_impl.h"
+
+namespace content {
+
+NavigationStateImpl* NavigationStateImpl::CreateBrowserInitiated(
+    const CommonNavigationParams& common_params,
+    const StartNavigationParams& start_params,
+    const HistoryNavigationParams& history_params) {
+  return new NavigationStateImpl(common_params, start_params, history_params,
+                                 false);
+}
+
+NavigationStateImpl* NavigationStateImpl::CreateContentInitiated() {
+  return new NavigationStateImpl(CommonNavigationParams(),
+                                 StartNavigationParams(),
+                                 HistoryNavigationParams(), true);
+}
+
+NavigationStateImpl::NavigationStateImpl(
+    const CommonNavigationParams& common_params,
+    const StartNavigationParams& start_params,
+    const HistoryNavigationParams& history_params,
+    bool is_content_initiated)
+    : request_committed_(false),
+      was_within_same_page_(false),
+      is_content_initiated_(is_content_initiated),
+      common_params_(common_params),
+      start_params_(start_params),
+      history_params_(history_params) {
+}
+
+NavigationStateImpl::~NavigationStateImpl() {

[Charlie Reis, 2015/03/17 16:29:06]

Style nit: Please list these in the same order as the .h file.

[clamy, 2015/03/17 17:19:54]

Done.

+}
+
+ui::PageTransition NavigationStateImpl::GetTransitionType() {
+  return common_params_.transition;
+}
+
+bool NavigationStateImpl::WasWithinSamePage() {
+  return was_within_same_page_;
+}
+
+bool NavigationStateImpl::IsContentInitiated() {
+  return is_content_initiated_;
+}
+
+}  // namespace content