PS - Adjusting webRequest API for use in Public Sessions

chrome/browser/extensions/api/web_request/web_request_event_details_unittest.cc

Index: chrome/browser/extensions/api/web_request/web_request_event_details_unittest.cc
diff --git a/chrome/browser/extensions/api/web_request/web_request_event_details_unittest.cc b/chrome/browser/extensions/api/web_request/web_request_event_details_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..d6aa30e4fadb325583eac22133531d6ceb614c70
--- /dev/null
+++ b/chrome/browser/extensions/api/web_request/web_request_event_details_unittest.cc
@@ -0,0 +1,66 @@
+// 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 "extensions/browser/api/web_request/web_request_event_details.h"
+
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace extensions {
+
+TEST(WebRequestEventDetailsTest, WhitelistedCopyForPublicSession) {
+  // Create the original instance and populate it with some values.
+  std::unique_ptr<WebRequestEventDetails> orig(new WebRequestEventDetails);
+
+  orig->render_process_id_ = 1;
+  orig->render_frame_id_ = 2;
+  orig->extra_info_spec_ = 3;
+
+  orig->request_body_.reset(new base::DictionaryValue);
+  orig->request_headers_.reset(new base::ListValue);
+  orig->response_headers_.reset(new base::ListValue);
+
+  const char* const safe_attributes[] = {
+    "method", "requestId", "timeStamp", "type", "tabId", "frameId",
+    "parentFrameId", "fromCache", "error", "ip", "statusLine", "statusCode"
+  };
+
+  for (const char* safe_attr : safe_attributes) {
+    orig->dict_.SetString(safe_attr, safe_attr);
+  }
+
+  orig->dict_.SetString("url", "http://www.foo.bar/baz");
+
+  // Add some extra dict_ values that should be filtered out.
+  orig->dict_.SetString("requestBody", "request body value");
+  orig->dict_.SetString("requestHeaders", "request headers value");
+
+  // Make a copy and check that filtering works.
+  std::unique_ptr<WebRequestEventDetails> copy(
+      orig->WhitelistedCopyForPublicSession());
+
+  EXPECT_EQ(orig->render_process_id_, copy->render_process_id_);
+  EXPECT_EQ(orig->render_frame_id_, copy->render_frame_id_);
+  EXPECT_EQ(0, copy->extra_info_spec_);
+
+  EXPECT_EQ(nullptr, copy->request_body_);
+  EXPECT_EQ(nullptr, copy->request_headers_);
+  EXPECT_EQ(nullptr, copy->response_headers_);
+
+  for (const char* safe_attr : safe_attributes) {
+    std::string copy_str, orig_str;
+    copy->dict_.GetString(safe_attr, &copy_str);
+    orig->dict_.GetString(safe_attr, &orig_str);
+    EXPECT_EQ(orig_str, copy_str);
+  }
+
+  // URL is stripped down to origin.
+  std::string url;
+  copy->dict_.GetString("url", &url);
+  EXPECT_EQ("http://www.foo.bar/", url);
+
+  // Extras are filtered out (+1 for url).
+  EXPECT_EQ(arraysize(safe_attributes) + 1, copy->dict_.size());
+}
+
+}  // namespace extensions