Adding UMA for sync IPCs

content/worker/worker_webkitplatformsupport_impl.cc

 // Copyright (c) 2012 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/worker/worker_webkitplatformsupport_impl.h"
 
 #include "base/logging.h"
 #include "base/platform_file.h"
 #include "base/utf_string_conversions.h"
 #include "content/common/database_util.h"
+#include "content/common/file_utilities_messages.h"
 #include "content/common/fileapi/webblobregistry_impl.h"
 #include "content/common/fileapi/webfilesystem_impl.h"
-#include "content/common/file_utilities_messages.h"
 #include "content/common/indexed_db/proxy_webidbfactory_impl.h"
 #include "content/common/mime_registry_messages.h"
 #include "content/common/webmessageportchannel_impl.h"
+#include "content/public/browser/user_metrics.h"

[jochen (gone - plz use gerrit), 2012/05/30 08:28:09]

you can't include browser headers in worker or renderer.

Instead, you need to send a ViewHostMsg_UserMetrics_RecordAction IPC to the
browser. For the worker, you'll need to add a WorkerHostMsg

Alternatively, and maybe better, can you record the metric in the browser when
you receive a sync ipc?

 #include "content/worker/worker_thread.h"
 #include "ipc/ipc_sync_message_filter.h"
 #include "net/base/mime_util.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebFileInfo.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebBlobRegistry.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebString.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebURL.h"
 #include "webkit/glue/webfileutilities_impl.h"
 #include "webkit/glue/webkit_glue.h"
 
@@ skipping 16 matching lines @@
 class WorkerWebKitPlatformSupportImpl::FileUtilities
     : public webkit_glue::WebFileUtilitiesImpl {
  public:
   virtual bool getFileSize(const WebKit::WebString& path, long long& result);
   virtual bool getFileModificationTime(const WebKit::WebString& path,
                                        double& result);
   virtual bool getFileInfo(const WebString& path, WebFileInfo& result);
 };
 
 static bool SendSyncMessageFromAnyThread(IPC::SyncMessage* msg) {
+  content::RecordAction(UserMetricsAction("SendSyncIPC_Worker"));
+
   WorkerThread* worker_thread = WorkerThread::current();
   if (worker_thread)
     return worker_thread->Send(msg);
 
   scoped_refptr<IPC::SyncMessageFilter> sync_msg_filter(
       ChildThread::current()->sync_message_filter());
   return sync_msg_filter->Send(msg);
 }
 
 bool WorkerWebKitPlatformSupportImpl::FileUtilities::getFileSize(
@@ skipping 243 matching lines @@
       new MimeRegistryMsg_GetPreferredExtensionForMimeType(
           UTF16ToASCII(mime_type), &file_extension));
   return webkit_glue::FilePathStringToWebString(file_extension);
 }
 
 WebBlobRegistry* WorkerWebKitPlatformSupportImpl::blobRegistry() {
   if (!blob_registry_.get())
     blob_registry_.reset(new WebBlobRegistryImpl(WorkerThread::current()));
   return blob_registry_.get();
 }