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

Side by Side Diff: content/browser/webrtc/webrtc_internals_browsertest.cc

Issue 2539363004: Make base::Value::TYPE a scoped enum. (Closed)
Patch Set: Rebase Created 4 years 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
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <stddef.h> 5 #include <stddef.h>
6 #include <stdint.h> 6 #include <stdint.h>
7 7
8 #include "base/command_line.h" 8 #include "base/command_line.h"
9 #include "base/json/json_reader.h" 9 #include "base/json/json_reader.h"
10 #include "base/strings/utf_string_conversions.h" 10 #include "base/strings/utf_string_conversions.h"
(...skipping 221 matching lines...) Expand 10 before | Expand all | Expand 10 after
232 const std::vector<UserMediaRequestEntry>& requests) { 232 const std::vector<UserMediaRequestEntry>& requests) {
233 string json_requests; 233 string json_requests;
234 ASSERT_TRUE( 234 ASSERT_TRUE(
235 ExecuteScriptAndExtractString(shell(), 235 ExecuteScriptAndExtractString(shell(),
236 "window.domAutomationController.send(" 236 "window.domAutomationController.send("
237 " JSON.stringify(userMediaRequests));", 237 " JSON.stringify(userMediaRequests));",
238 &json_requests)); 238 &json_requests));
239 std::unique_ptr<base::Value> value_requests = 239 std::unique_ptr<base::Value> value_requests =
240 base::JSONReader::Read(json_requests); 240 base::JSONReader::Read(json_requests);
241 241
242 EXPECT_EQ(base::Value::TYPE_LIST, value_requests->GetType()); 242 EXPECT_EQ(base::Value::Type::LIST, value_requests->GetType());
243 243
244 base::ListValue* list_request = 244 base::ListValue* list_request =
245 static_cast<base::ListValue*>(value_requests.get()); 245 static_cast<base::ListValue*>(value_requests.get());
246 EXPECT_EQ(requests.size(), list_request->GetSize()); 246 EXPECT_EQ(requests.size(), list_request->GetSize());
247 247
248 for (size_t i = 0; i < requests.size(); ++i) { 248 for (size_t i = 0; i < requests.size(); ++i) {
249 base::DictionaryValue* dict = NULL; 249 base::DictionaryValue* dict = NULL;
250 ASSERT_TRUE(list_request->GetDictionary(i, &dict)); 250 ASSERT_TRUE(list_request->GetDictionary(i, &dict));
251 int pid, rid; 251 int pid, rid;
252 std::string origin, audio, video; 252 std::string origin, audio, video;
(...skipping 179 matching lines...) Expand 10 before | Expand all | Expand 10 after
432 return count; 432 return count;
433 } 433 }
434 434
435 // Verifies |dump| contains |peer_connection_number| peer connection dumps, 435 // Verifies |dump| contains |peer_connection_number| peer connection dumps,
436 // each containing |update_number| updates and |stats_number| stats tables. 436 // each containing |update_number| updates and |stats_number| stats tables.
437 void VerifyPageDumpStructure(base::Value* dump, 437 void VerifyPageDumpStructure(base::Value* dump,
438 int peer_connection_number, 438 int peer_connection_number,
439 int update_number, 439 int update_number,
440 int stats_number) { 440 int stats_number) {
441 EXPECT_NE((base::Value*)NULL, dump); 441 EXPECT_NE((base::Value*)NULL, dump);
442 EXPECT_EQ(base::Value::TYPE_DICTIONARY, dump->GetType()); 442 EXPECT_EQ(base::Value::Type::DICTIONARY, dump->GetType());
443 443
444 base::DictionaryValue* dict_dump = 444 base::DictionaryValue* dict_dump =
445 static_cast<base::DictionaryValue*>(dump); 445 static_cast<base::DictionaryValue*>(dump);
446 EXPECT_EQ((size_t) peer_connection_number, dict_dump->size()); 446 EXPECT_EQ((size_t) peer_connection_number, dict_dump->size());
447 447
448 base::DictionaryValue::Iterator it(*dict_dump); 448 base::DictionaryValue::Iterator it(*dict_dump);
449 for (; !it.IsAtEnd(); it.Advance()) { 449 for (; !it.IsAtEnd(); it.Advance()) {
450 base::Value* value = NULL; 450 base::Value* value = NULL;
451 dict_dump->Get(it.key(), &value); 451 dict_dump->Get(it.key(), &value);
452 EXPECT_EQ(base::Value::TYPE_DICTIONARY, value->GetType()); 452 EXPECT_EQ(base::Value::Type::DICTIONARY, value->GetType());
453 base::DictionaryValue* pc_dump = 453 base::DictionaryValue* pc_dump =
454 static_cast<base::DictionaryValue*>(value); 454 static_cast<base::DictionaryValue*>(value);
455 EXPECT_TRUE(pc_dump->HasKey("updateLog")); 455 EXPECT_TRUE(pc_dump->HasKey("updateLog"));
456 EXPECT_TRUE(pc_dump->HasKey("stats")); 456 EXPECT_TRUE(pc_dump->HasKey("stats"));
457 457
458 // Verifies the number of updates. 458 // Verifies the number of updates.
459 pc_dump->Get("updateLog", &value); 459 pc_dump->Get("updateLog", &value);
460 EXPECT_EQ(base::Value::TYPE_LIST, value->GetType()); 460 EXPECT_EQ(base::Value::Type::LIST, value->GetType());
461 base::ListValue* list = static_cast<base::ListValue*>(value); 461 base::ListValue* list = static_cast<base::ListValue*>(value);
462 EXPECT_EQ((size_t) update_number, list->GetSize()); 462 EXPECT_EQ((size_t) update_number, list->GetSize());
463 463
464 // Verifies the number of stats tables. 464 // Verifies the number of stats tables.
465 pc_dump->Get("stats", &value); 465 pc_dump->Get("stats", &value);
466 EXPECT_EQ(base::Value::TYPE_DICTIONARY, value->GetType()); 466 EXPECT_EQ(base::Value::Type::DICTIONARY, value->GetType());
467 base::DictionaryValue* dict = static_cast<base::DictionaryValue*>(value); 467 base::DictionaryValue* dict = static_cast<base::DictionaryValue*>(value);
468 EXPECT_EQ((size_t) stats_number, dict->size()); 468 EXPECT_EQ((size_t) stats_number, dict->size());
469 } 469 }
470 } 470 }
471 471
472 // Verifies |dump| contains the correct statsTable and statsDataSeries for 472 // Verifies |dump| contains the correct statsTable and statsDataSeries for
473 // |pc|. 473 // |pc|.
474 void VerifyStatsDump(base::Value* dump, 474 void VerifyStatsDump(base::Value* dump,
475 const PeerConnectionEntry& pc, 475 const PeerConnectionEntry& pc,
476 const string& report_type, 476 const string& report_type,
477 const string& report_id, 477 const string& report_id,
478 const StatsUnit& stats) { 478 const StatsUnit& stats) {
479 EXPECT_NE((base::Value*)NULL, dump); 479 EXPECT_NE((base::Value*)NULL, dump);
480 EXPECT_EQ(base::Value::TYPE_DICTIONARY, dump->GetType()); 480 EXPECT_EQ(base::Value::Type::DICTIONARY, dump->GetType());
481 481
482 base::DictionaryValue* dict_dump = 482 base::DictionaryValue* dict_dump =
483 static_cast<base::DictionaryValue*>(dump); 483 static_cast<base::DictionaryValue*>(dump);
484 base::Value* value = NULL; 484 base::Value* value = NULL;
485 dict_dump->Get(pc.getIdString(), &value); 485 dict_dump->Get(pc.getIdString(), &value);
486 base::DictionaryValue* pc_dump = static_cast<base::DictionaryValue*>(value); 486 base::DictionaryValue* pc_dump = static_cast<base::DictionaryValue*>(value);
487 487
488 // Verifies there is one data series per stats name. 488 // Verifies there is one data series per stats name.
489 value = NULL; 489 value = NULL;
490 pc_dump->Get("stats", &value); 490 pc_dump->Get("stats", &value);
491 EXPECT_EQ(base::Value::TYPE_DICTIONARY, value->GetType()); 491 EXPECT_EQ(base::Value::Type::DICTIONARY, value->GetType());
492 492
493 base::DictionaryValue* dataSeries = 493 base::DictionaryValue* dataSeries =
494 static_cast<base::DictionaryValue*>(value); 494 static_cast<base::DictionaryValue*>(value);
495 EXPECT_EQ(stats.values.size(), dataSeries->size()); 495 EXPECT_EQ(stats.values.size(), dataSeries->size());
496 } 496 }
497 }; 497 };
498 498
499 IN_PROC_BROWSER_TEST_F(MAYBE_WebRtcInternalsBrowserTest, 499 IN_PROC_BROWSER_TEST_F(MAYBE_WebRtcInternalsBrowserTest,
500 AddAndRemovePeerConnection) { 500 AddAndRemovePeerConnection) {
501 GURL url("chrome://webrtc-internals"); 501 GURL url("chrome://webrtc-internals");
(...skipping 372 matching lines...) Expand 10 before | Expand all | Expand 10 after
874 ASSERT_TRUE(ExecuteScriptAndExtractInt( 874 ASSERT_TRUE(ExecuteScriptAndExtractInt(
875 shell(), 875 shell(),
876 "window.domAutomationController.send(" 876 "window.domAutomationController.send("
877 " peerConnectionDataStore['" + pc.getIdString() + "']" + 877 " peerConnectionDataStore['" + pc.getIdString() + "']" +
878 " .getDataSeries('" + data_series_id + "').getCount())", 878 " .getDataSeries('" + data_series_id + "').getCount())",
879 &count)); 879 &count));
880 EXPECT_EQ(3, count); 880 EXPECT_EQ(3, count);
881 } 881 }
882 882
883 } // namespace content 883 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/webrtc/webrtc_getusermedia_browsertest.cc ('k') | content/browser/webui/web_ui_data_source_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698