Working version against 8.x-1.x patch

+++ b/lib/Drupal/xhprof/EventSubscriber/XHProfSubscriber.phpundefined
@@ -0,0 +1,34 @@
+// use Symfony\Component\HttpKernel\KernelEvents;
+// use Symfony\Component\HttpKernel\Event\GetResponseEvent;

+++ b/lib/Drupal/xhprof/XHProfBundle.phpundefined
@@ -0,0 +1,22 @@
\ No newline at end of file

Should be cleaned-up

+++ b/lib/Drupal/xhprof/EventSubscriber/XHProfSubscriber.phpundefined
@@ -0,0 +1,34 @@
+  public function onKernelRequestTest() {
+    drupal_set_message(t('My event subscriber fired!'));
...
+  static function getSubscribedEvents() {
+    $events[KernelEvents::REQUEST][] = array('onKernelRequestTest', 100);

is not needed

+++ b/lib/Drupal/xhprof/XHProfSubscriber.phpundefined
@@ -0,0 +1,42 @@
+    if (xhprof_xhprof_enable()) {
...
+  static function getSubscribedEvents() {
+    $events[KernelEvents::REQUEST][] = array('XHProfLoad', 999);

Suppose better make the check xhprof_xhprof_enable() in getSubscribedEvents()

+++ b/lib/Drupal/xhprof/XHProfSubscriber.phpundefined
@@ -0,0 +1,42 @@
+      drupal_set_message('XHProf: subscribed');

No reason for message

Attached is an updated patch with some of the latest 8.x changes. Not quite tested fully yet.

Suppose better make the check xhprof_xhprof_enable() in getSubscribedEvents()

Not sure that's the right design choice. getSubscribedEvents() should just get the events, not do any business logic.

Anyone up for continuing this port to D8?

Hi, can I propose myself to continue the porting? I'm developing the webprofiler module and integrate the Drupal profiled data with the xhprof data would be very interesting

Shouldn't we at least get something in first so that patches could be filed against 8.x next?

anavarre I'll first port the module for D8 (more or less as is), then we can think how we can integrate it with webprofiler

Here there is a first working version for D8:

https://github.com/lussoluca/xhprof

For now there is a simple report page which shows the top 100 function. Basic integration with webprofiler is done.

If it's all ok I can attach a patch here

Here is the patch, it's aligned with the current D8 HEAD

pushed few PR to github to fix some issues

Sine this module is so lightly maintained, Iadded andypost and lussoluca to the Maintainers list. You guys can commit here, manage issues, etc. Thanks for your help!

pushed changes from github, so module now works for 8.0.x