Add metrics to record TCP/UDP connections made from Flash

content/browser/renderer_host/pepper/browser_ppapi_host_impl.cc

Index: content/browser/renderer_host/pepper/browser_ppapi_host_impl.cc
diff --git a/content/browser/renderer_host/pepper/browser_ppapi_host_impl.cc b/content/browser/renderer_host/pepper/browser_ppapi_host_impl.cc
index d7f4647567d86f0f970a67b0f81433dc7bd944fd..d263ee1c91eb06ce2a315b981dfe1799e80b6072 100644
--- a/content/browser/renderer_host/pepper/browser_ppapi_host_impl.cc
+++ b/content/browser/renderer_host/pepper/browser_ppapi_host_impl.cc
@@ -141,6 +141,14 @@ void BrowserPpapiHostImpl::SetOnKeepaliveCallback(
   on_keepalive_callback_ = callback;
 }
 
+bool BrowserPpapiHostImpl::IsPotentiallySecurePluginContext(
+    PP_Instance instance) {
+  auto* data = instance_map_.get(instance);

[Ryan Sleevi, 2015/05/12 22:56:05]

nit: why the auto*? Why not let auto bind to the appropriate type?

[dcheng, 2015/05/12 23:56:20]

(I'm not raymes@ but I personally prefer auto* when appropriate, to make it
clear that there's no accidental copy happening)

[raymes, 2015/05/13 00:07:19]

I copied the code from other functions in here. I've changed it.

[raymes, 2015/05/13 00:08:00]

based on dcheng's comment, I left it as-is :P but really I don't feel strongly.

+  if (data == nullptr)
+    return false;
+  return data->renderer_data.is_potentially_secure_plugin_context;
+}
+
 void BrowserPpapiHostImpl::AddInstance(
     PP_Instance instance,
     const PepperRendererInstanceData& renderer_instance_data) {