Inject renderer recieve message time into resource notifications.

content/common/resource_dispatcher.h

 // 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.
 
 // See http://dev.chromium.org/developers/design-documents/multi-process-resource-loading
 
 #ifndef CONTENT_COMMON_RESOURCE_DISPATCHER_H_
 #define CONTENT_COMMON_RESOURCE_DISPATCHER_H_
 
 #include <deque>
@@ skipping 53 matching lines @@
   // Indicates the priority of the specified request changed.
   void DidChangePriority(int routing_id, int request_id,
                          net::RequestPriority new_priority);
 
   // This does not take ownership of the delegate. It is expected that the
   // delegate have a longer lifetime than the ResourceDispatcher.
   void set_delegate(ResourceDispatcherDelegate* delegate) {
     delegate_ = delegate;
   }
 
+  // Remembers IO thread timestamp for next resource message.
+  void set_io_timestamp(base::TimeTicks io_timestamp) {
+    io_timestamp_ = io_timestamp;

[James Simonsen, 2013/04/26 23:02:28]

What happens if there are many resource messages queued? Admittedly it's better
than what we have today, but it's still not exactly what we want. Could we add a
field to the messages themselves instead?

[eustas, 2013/04/29 08:59:03]

I've got exactly the same idea first (see Patch Set 1).
I've added fields to messages and tampered them in filter.
But this looks too hacky, because (conceptually) filters should not change
messages (they get const IPC::Message).

If I'm not mistaken, setting timestamps and receiving messages tasks will be
interleaved, so each message will correspond to its own timestamp.

+  }
+
  private:
   friend class ResourceDispatcherTest;
 
   typedef std::deque<IPC::Message*> MessageQueue;
   struct PendingRequestInfo {
     PendingRequestInfo();
 
     PendingRequestInfo(webkit_glue::ResourceLoaderBridge::Peer* peer,
                        ResourceType::Type resource_type,
                        const GURL& request_url);
@@ skipping 89 matching lines @@
 
   IPC::Sender* message_sender_;
 
   // All pending requests issued to the host
   PendingRequestList pending_requests_;
 
   base::WeakPtrFactory<ResourceDispatcher> weak_factory_;
 
   ResourceDispatcherDelegate* delegate_;
 
+  base::TimeTicks io_timestamp_;
+
   DISALLOW_COPY_AND_ASSIGN(ResourceDispatcher);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_COMMON_RESOURCE_DISPATCHER_H_