Add security handler stub for sending the lock icon status to DevTools.

content/browser/devtools/protocol/security_handler.h

Index: content/browser/devtools/protocol/security_handler.h
diff --git a/content/browser/devtools/protocol/security_handler.h b/content/browser/devtools/protocol/security_handler.h
new file mode 100644
index 0000000000000000000000000000000000000000..e2acc66f8d92175057af9a6a7cf572dc9b48e98e
--- /dev/null
+++ b/content/browser/devtools/protocol/security_handler.h
@@ -0,0 +1,34 @@
+// 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.
+
+#ifndef CONTENT_BROWSER_DEVTOOLS_PROTOCOL_SECURITY_HANDLER_H_
+#define CONTENT_BROWSER_DEVTOOLS_PROTOCOL_SECURITY_HANDLER_H_
+
+#include "content/browser/devtools/devtools_protocol_handler.h"
+#include "content/public/common/security_style.h"

[pfeldman, 2015/06/09 18:35:53]

Unused?

[lgarron, 2015/06/09 18:51:23]

Indeed. It'll be needed later, but I've removed it anyhow.

+
+namespace content {
+namespace devtools {
+namespace security {
+
+class SecurityHandler {
+ public:
+  typedef DevToolsProtocolClient::Response Response;
+
+  SecurityHandler();
+  ~SecurityHandler();

[pfeldman, 2015/06/09 18:35:53]

virtual

[lgarron, 2015/06/09 18:51:23]

Are you saying I should make it virtual?

Other handlers are inconsistent about whether to mark this as `virtual`,
`override`, or neither.

[pfeldman, 2015/06/09 19:04:23]

virtual for now, change to override later.

+

[estark, 2015/06/09 18:48:25]

Sorry, I should have been more specific in my last comment. I think we *do* need
a SetClient() method here (since generated code will attempt to call it:
https://code.google.com/p/chromium/codesearch#chromium/src/out/Debug/gen/cont...).
But SetClient() can't take a devtools::security::Client as an argument because
that class isn't defined yet. Instead, I guess it should take a
scoped_ptr<DevToolsProtocolClient*> as an argument (which is the base class of
the devtools::*::Client classes), and we can change that to a
devtools::security::Client once the protocol.json changes land.

[pfeldman, 2015/06/09 19:04:23]

This is a good point - you need a generic SetClient for the first commit.

+  Response Enable();
+  Response Disable();
+
+ private:
+
+  DISALLOW_COPY_AND_ASSIGN(SecurityHandler);
+};
+
+}  // namespace security
+}  // namespace devtools
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_DEVTOOLS_PROTOCOL_SECURITY_HANDLER_H_