PlzNavigate: support the WebRequest API

chrome/browser/renderer_host/chrome_navigation_ui_data.cc

+// Copyright 2016 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 "chrome/browser/renderer_host/chrome_navigation_ui_data.h"
+
+#include "base/memory/ptr_util.h"
+#include "chrome/browser/sessions/session_tab_helper.h"
+#include "content/public/browser/navigation_handle.h"
+
+ChromeNavigationUIData::ChromeNavigationUIData() {}
+
+ChromeNavigationUIData::ChromeNavigationUIData(
+    content::NavigationHandle* navigation_handle) {
+#if defined(ENABLE_EXTENSIONS)
+  SessionTabHelper* session_tab_helper =
+      SessionTabHelper::FromWebContents(navigation_handle->GetWebContents());
+  int tab_id = session_tab_helper ? session_tab_helper->session_id().id() : -1;

[sky, 2016/09/26 16:45:47]

Is there a define for the -1 you can use?

[clamy, 2016/09/27 14:57:52]

Unfortunately, there doesn't seem to be. Eventually this is used to instantiate
an ExtensionApiFrameIdMap::FrameData, that has default values of -1 for those
values (see
https://cs.chromium.org/chromium/src/extensions/browser/extension_api_frame_i...).
The code in SessionTabHelper doesn't have constant defined either.

+  int window_id =
+      session_tab_helper ? session_tab_helper->window_id().id() : -1;
+  extension_data_ = base::MakeUnique<extensions::ExtensionNavigationUIData>(
+      navigation_handle, tab_id, window_id);
+#endif
+}
+
+ChromeNavigationUIData::~ChromeNavigationUIData() {}
+
+std::unique_ptr<content::NavigationUIData> ChromeNavigationUIData::Clone()
+    const {
+  std::unique_ptr<ChromeNavigationUIData> copy(new ChromeNavigationUIData());

[sky, 2016/09/26 16:45:47]

MakeUnique.

[clamy, 2016/09/27 14:57:52]

Done.

+
+#if defined(ENABLE_EXTENSIONS)
+  if (extension_data_)
+    copy->SetExtensionNavigationUIData(extension_data_->DeepCopy());
+#endif
+
+  return std::move(copy);
+}