Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(589)

Side by Side Diff: runtime/vm/report_test.cc

Issue 1162133002: Introduce Instance.kind. (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: regis feedback Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « runtime/vm/object_test.cc ('k') | runtime/vm/service/service.md » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 #include "platform/assert.h" 5 #include "platform/assert.h"
6 #include "vm/report.h" 6 #include "vm/report.h"
7 #include "vm/unit_test.h" 7 #include "vm/unit_test.h"
8 8
9 namespace dart { 9 namespace dart {
10 10
(...skipping 15 matching lines...) Expand all
26 trace_buffer->PrintToJSONStream(&js); 26 trace_buffer->PrintToJSONStream(&js);
27 EXPECT_SUBSTRING("{\"type\":\"TraceBuffer\",\"members\":[" 27 EXPECT_SUBSTRING("{\"type\":\"TraceBuffer\",\"members\":["
28 "{\"type\":\"TraceBufferEntry\",\"time\":", 28 "{\"type\":\"TraceBufferEntry\",\"time\":",
29 js.ToCString()); 29 js.ToCString());
30 // Skip time. 30 // Skip time.
31 EXPECT_SUBSTRING("\"message\":{\"type\":\"JSCompatibilityWarning\"," 31 EXPECT_SUBSTRING("\"message\":{\"type\":\"JSCompatibilityWarning\","
32 "\"script\":{\"type\":\"@Script\",\"fixedId\":true," 32 "\"script\":{\"type\":\"@Script\",\"fixedId\":true,"
33 "\"id\":\"libraries\\/-1\\/scripts\\/Plug\"," 33 "\"id\":\"libraries\\/-1\\/scripts\\/Plug\","
34 "\"uri\":\"Plug\"," 34 "\"uri\":\"Plug\","
35 "\"_kind\":\"script\"},\"tokenPos\":0," 35 "\"_kind\":\"script\"},\"tokenPos\":0,"
36 "\"message\":{\"type\":\"@String\"", 36 "\"message\":{\"type\":\"@Instance\"",
37 js.ToCString()); 37 js.ToCString());
38 // Skip private _OneByteString. 38 // Skip private _OneByteString.
39 EXPECT_SUBSTRING("\"valueAsString\":\"High Voltage\"", 39 EXPECT_SUBSTRING("\"valueAsString\":\"High Voltage\"",
40 js.ToCString()); 40 js.ToCString());
41 } 41 }
42 } 42 }
43 { 43 {
44 const intptr_t token_pos = 1; 44 const intptr_t token_pos = 1;
45 const char* message = "Low Voltage"; 45 const char* message = "Low Voltage";
46 Report::MessageF(Report::kJSWarning, script, token_pos, "%s", message); 46 Report::MessageF(Report::kJSWarning, script, token_pos, "%s", message);
47 } 47 }
48 EXPECT_EQ(2, trace_buffer->Length()); 48 EXPECT_EQ(2, trace_buffer->Length());
49 EXPECT_SUBSTRING("{\"type\":\"JSCompatibilityWarning\",\"script\":{\"type\":" 49 EXPECT_SUBSTRING("{\"type\":\"JSCompatibilityWarning\",\"script\":{\"type\":"
50 "\"@Script\",\"fixedId\":true," 50 "\"@Script\",\"fixedId\":true,"
51 "\"id\":\"libraries\\/-1\\/scripts\\/Plug\"," 51 "\"id\":\"libraries\\/-1\\/scripts\\/Plug\","
52 "\"uri\":\"Plug\"," 52 "\"uri\":\"Plug\","
53 "\"_kind\":\"script\"},\"tokenPos\":0," 53 "\"_kind\":\"script\"},\"tokenPos\":0,"
54 "\"message\":{\"type\":\"@String\"", 54 "\"message\":{\"type\":\"@Instance\"",
55 trace_buffer->At(0)->message); 55 trace_buffer->At(0)->message);
56 // Skip private _OneByteString. 56 // Skip private _OneByteString.
57 EXPECT_SUBSTRING("\"valueAsString\":\"High Voltage\"", 57 EXPECT_SUBSTRING("\"valueAsString\":\"High Voltage\"",
58 trace_buffer->At(0)->message); 58 trace_buffer->At(0)->message);
59 59
60 EXPECT_SUBSTRING("{\"type\":\"JSCompatibilityWarning\",\"script\":{\"type\":" 60 EXPECT_SUBSTRING("{\"type\":\"JSCompatibilityWarning\",\"script\":{\"type\":"
61 "\"@Script\",\"fixedId\":true," 61 "\"@Script\",\"fixedId\":true,"
62 "\"id\":\"libraries\\/-1\\/scripts\\/Plug\"," 62 "\"id\":\"libraries\\/-1\\/scripts\\/Plug\","
63 "\"uri\":\"Plug\"," 63 "\"uri\":\"Plug\","
64 "\"_kind\":\"script\"},\"tokenPos\":1," 64 "\"_kind\":\"script\"},\"tokenPos\":1,"
65 "\"message\":{\"type\":\"@String\"", 65 "\"message\":{\"type\":\"@Instance\"",
66 trace_buffer->At(1)->message); 66 trace_buffer->At(1)->message);
67 // Skip private _OneByteString. 67 // Skip private _OneByteString.
68 EXPECT_SUBSTRING("\"valueAsString\":\"Low Voltage\"", 68 EXPECT_SUBSTRING("\"valueAsString\":\"Low Voltage\"",
69 trace_buffer->At(1)->message); 69 trace_buffer->At(1)->message);
70 70
71 delete trace_buffer; 71 delete trace_buffer;
72 } 72 }
73 73
74 } // namespace dart 74 } // namespace dart
OLDNEW
« no previous file with comments | « runtime/vm/object_test.cc ('k') | runtime/vm/service/service.md » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698