 Chromium Code Reviews
 Chromium Code Reviews Issue 1534583002:
  Migrate Local Discovery from net::IPAddressNumber to net::IPAddress.  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master
    
  
    Issue 1534583002:
  Migrate Local Discovery from net::IPAddressNumber to net::IPAddress.  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master| OLD | NEW | 
|---|---|
| 1 // Copyright 2015 The Chromium Authors. All rights reserved. | 1 // Copyright 2015 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 "chrome/browser/devtools/device/cast_device_provider.h" | 5 #include "chrome/browser/devtools/device/cast_device_provider.h" | 
| 6 | 6 | 
| 7 #include "chrome/browser/devtools/device/android_device_manager.h" | 7 #include "chrome/browser/devtools/device/android_device_manager.h" | 
| 8 #include "net/base/host_port_pair.h" | 8 #include "net/base/host_port_pair.h" | 
| 9 #include "net/base/ip_address.h" | |
| 9 #include "testing/gtest/include/gtest/gtest.h" | 10 #include "testing/gtest/include/gtest/gtest.h" | 
| 10 | 11 | 
| 11 namespace { | 12 namespace { | 
| 12 | 13 | 
| 13 typedef AndroidDeviceManager::DeviceInfo DeviceInfo; | 14 typedef AndroidDeviceManager::DeviceInfo DeviceInfo; | 
| 14 typedef AndroidDeviceManager::BrowserInfo BrowserInfo; | 15 typedef AndroidDeviceManager::BrowserInfo BrowserInfo; | 
| 15 | 16 | 
| 16 void CompareDeviceInfo(bool* was_run, | 17 void CompareDeviceInfo(bool* was_run, | 
| 17 const DeviceInfo& expected, | 18 const DeviceInfo& expected, | 
| 18 const DeviceInfo& actual) { | 19 const DeviceInfo& actual) { | 
| (...skipping 23 matching lines...) Expand all Loading... | |
| 42 const std::string cast_service_type = "_googlecast._tcp.local"; | 43 const std::string cast_service_type = "_googlecast._tcp.local"; | 
| 43 const std::string cast_service_model = "Fake Cast Device"; | 44 const std::string cast_service_model = "Fake Cast Device"; | 
| 44 | 45 | 
| 45 ServiceDescription cast_service; | 46 ServiceDescription cast_service; | 
| 46 cast_service.service_name = | 47 cast_service.service_name = | 
| 47 cast_service_display_name + "." + cast_service_type; | 48 cast_service_display_name + "." + cast_service_type; | 
| 48 cast_service.address = net::HostPortPair("192.168.1.101", 8009); | 49 cast_service.address = net::HostPortPair("192.168.1.101", 8009); | 
| 49 cast_service.metadata.push_back("id=0123456789abcdef0123456789abcdef"); | 50 cast_service.metadata.push_back("id=0123456789abcdef0123456789abcdef"); | 
| 50 cast_service.metadata.push_back("ve=00"); | 51 cast_service.metadata.push_back("ve=00"); | 
| 51 cast_service.metadata.push_back("md=" + cast_service_model); | 52 cast_service.metadata.push_back("md=" + cast_service_model); | 
| 52 cast_service.ip_address = {192, 168, 1, 101}; | 53 uint8_t address1[4] = {192, 168, 1, 101}; | 
| 54 cast_service.ip_address = net::IPAddress(address1, sizeof(address1)); | |
| 
eroman
2015/12/21 20:47:33
Or alternately:
ASSERT_TRUE(net::IPAddress::FromI
 
martijnc
2016/01/27 22:50:52
Done.
 | |
| 53 | 55 | 
| 54 device_provider_->OnDeviceChanged(true, cast_service); | 56 device_provider_->OnDeviceChanged(true, cast_service); | 
| 55 | 57 | 
| 56 BrowserInfo exp_browser_info; | 58 BrowserInfo exp_browser_info; | 
| 57 exp_browser_info.socket_name = "9222"; | 59 exp_browser_info.socket_name = "9222"; | 
| 58 exp_browser_info.display_name = cast_service_display_name; | 60 exp_browser_info.display_name = cast_service_display_name; | 
| 59 exp_browser_info.type = BrowserInfo::kTypeChrome; | 61 exp_browser_info.type = BrowserInfo::kTypeChrome; | 
| 60 | 62 | 
| 61 DeviceInfo expected; | 63 DeviceInfo expected; | 
| 62 expected.model = cast_service_model; // From metadata::md | 64 expected.model = cast_service_model; // From metadata::md | 
| (...skipping 13 matching lines...) Expand all Loading... | |
| 76 const std::string other_service_type = "_other._tcp.local"; | 78 const std::string other_service_type = "_other._tcp.local"; | 
| 77 const std::string other_service_model = "Some Other Device"; | 79 const std::string other_service_model = "Some Other Device"; | 
| 78 | 80 | 
| 79 ServiceDescription other_service; | 81 ServiceDescription other_service; | 
| 80 other_service.service_name = | 82 other_service.service_name = | 
| 81 other_service_display_name + "." + other_service_type; | 83 other_service_display_name + "." + other_service_type; | 
| 82 other_service.address = net::HostPortPair("10.64.1.101", 1234); | 84 other_service.address = net::HostPortPair("10.64.1.101", 1234); | 
| 83 other_service.metadata.push_back("id=0123456789abcdef0123456789abcdef"); | 85 other_service.metadata.push_back("id=0123456789abcdef0123456789abcdef"); | 
| 84 other_service.metadata.push_back("ve=00"); | 86 other_service.metadata.push_back("ve=00"); | 
| 85 other_service.metadata.push_back("md=" + other_service_model); | 87 other_service.metadata.push_back("md=" + other_service_model); | 
| 86 other_service.ip_address = {10, 64, 1, 101}; | 88 uint8_t address2[4] = {10, 64, 1, 101}; | 
| 89 other_service.ip_address = net::IPAddress(address2, sizeof(address2)); | |
| 87 | 90 | 
| 88 // Callback should not be run, since this service is not yet discovered. | 91 // Callback should not be run, since this service is not yet discovered. | 
| 89 device_provider_->QueryDeviceInfo(other_service.address.host(), | 92 device_provider_->QueryDeviceInfo(other_service.address.host(), | 
| 90 base::Bind(&DummyCallback, &was_run)); | 93 base::Bind(&DummyCallback, &was_run)); | 
| 91 ASSERT_FALSE(was_run); | 94 ASSERT_FALSE(was_run); | 
| 92 | 95 | 
| 93 device_provider_->OnDeviceChanged(true, other_service); | 96 device_provider_->OnDeviceChanged(true, other_service); | 
| 94 | 97 | 
| 95 // Callback should not be run, since non-cast services are not discovered by | 98 // Callback should not be run, since non-cast services are not discovered by | 
| 96 // this device provider. | 99 // this device provider. | 
| 97 device_provider_->QueryDeviceInfo(other_service.address.host(), | 100 device_provider_->QueryDeviceInfo(other_service.address.host(), | 
| 98 base::Bind(&DummyCallback, &was_run)); | 101 base::Bind(&DummyCallback, &was_run)); | 
| 99 ASSERT_FALSE(was_run); | 102 ASSERT_FALSE(was_run); | 
| 100 | 103 | 
| 101 // Remove the cast service. | 104 // Remove the cast service. | 
| 102 device_provider_->OnDeviceRemoved(cast_service.service_name); | 105 device_provider_->OnDeviceRemoved(cast_service.service_name); | 
| 103 | 106 | 
| 104 // Callback should not be run, since the cast service has been removed. | 107 // Callback should not be run, since the cast service has been removed. | 
| 105 device_provider_->QueryDeviceInfo(cast_service.address.host(), | 108 device_provider_->QueryDeviceInfo(cast_service.address.host(), | 
| 106 base::Bind(&DummyCallback, &was_run)); | 109 base::Bind(&DummyCallback, &was_run)); | 
| 107 ASSERT_FALSE(was_run); | 110 ASSERT_FALSE(was_run); | 
| 108 } | 111 } | 
| OLD | NEW |