Display isolate message queue in Observatory debugger

runtime/vm/message_test.cc

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "platform/assert.h"
 #include "vm/message.h"
 #include "vm/unit_test.h"
 
 namespace dart {
 
@@ skipping 13 matching lines @@
   const char* str2 = "msg2";
   const char* str3 = "msg3";
   const char* str4 = "msg4";
   const char* str5 = "msg5";
   const char* str6 = "msg6";
 
   // Add two messages.
   Message* msg1 = new Message(
       port, AllocMsg(str1), strlen(str1) + 1, Message::kNormalPriority);
   queue.Enqueue(msg1, false);
+  EXPECT(queue.Length() == 1);
   EXPECT(!queue.IsEmpty());
+  // Iterate over messages.
+  MessageQueue::Iterator it(&queue);
+  for (; it.HasNext(); it.Next()) {
+    EXPECT(it.current() == msg1);
+  }

[turnidge, 2015/05/04 17:47:13]

Unroll, as suggested below.

[Cutch, 2015/05/04 19:56:51]

Done.

 
   Message* msg2 = new Message(
       port, AllocMsg(str2), strlen(str2) + 1, Message::kNormalPriority);
   queue.Enqueue(msg2, false);
+  EXPECT(queue.Length() == 2);
   EXPECT(!queue.IsEmpty());
+  it.Reset(&queue);
 
-  // Remove two messages.
+  for (intptr_t it_count = 0; it.HasNext(); it.Next(), it_count++) {
+    EXPECT(it_count < 2);
+    if (it_count == 0) {
+      EXPECT(it.current() == msg1);
+    } else {
+      ASSERT(it_count == 1);
+      EXPECT(it.current() == msg2);
+    }
+  }

[turnidge, 2015/05/04 17:47:13]

I would unroll this, I think it is shorter/simpler.

(Here's how the code would look it you rewrite Next() and HasNext() as I
suggested elsewhere)

EXPECT(it.hasNext());
EXPECT(it.Next() == msg1);
EXPECT(it.hasNext());
EXPECT(it.Next() == msg2);
EXPECT(!it.hasNext());

[Cutch, 2015/05/04 19:56:51]

Done.

+
+  // Lookup messages by id.
+  EXPECT(queue.FindMessageById(reinterpret_cast<intptr_t>(msg1)) == msg1);
+  EXPECT(queue.FindMessageById(reinterpret_cast<intptr_t>(msg2)) == msg2);
+
+  // Lookup bad id.
+  EXPECT(queue.FindMessageById(0x1) == NULL);
+
+  // Remove message 1
   Message* msg = queue.Dequeue();
   EXPECT(msg != NULL);
   EXPECT_STREQ(str1, reinterpret_cast<char*>(msg->data()));
   EXPECT(!queue.IsEmpty());
 
+  // Check that only msg2 is left in the queue.
+  for (it.Reset(&queue); it.HasNext(); it.Next()) {
+    EXPECT(it.current() == msg2);
+  }

[turnidge, 2015/05/04 17:47:13]

Unroll.

[Cutch, 2015/05/04 19:56:51]

Done.

+
+  // Remove message 2
   msg = queue.Dequeue();
   EXPECT(msg != NULL);
   EXPECT_STREQ(str2, reinterpret_cast<char*>(msg->data()));
   EXPECT(queue.IsEmpty());
 
   Message* msg3 = new Message(Message::kIllegalPort,
                               AllocMsg(str3), strlen(str3) + 1,
                               Message::kNormalPriority);
   queue.Enqueue(msg3, true);
   EXPECT(!queue.IsEmpty());
@@ skipping 63 matching lines @@
   queue.Enqueue(msg2, false);
 
   EXPECT(!queue.IsEmpty());
   queue.Clear();
   EXPECT(queue.IsEmpty());
 
   // msg1 and msg2 already delete by FlushAll.
 }
 
 }  // namespace dart