commit ee6b3d7f9d49fa52072a352fbb59f06127b1ba4c
author Paul Stewart <pstew@chromium.org> Fri Jul 12 16:07:51 2013 -0700
committer ChromeBot <chrome-bot@google.com> Mon Jul 15 17:02:33 2013 -0700
tree 999fcdd0f1ef2a021d070e4c04f49dee00847493
parent f6f9648d5b2beb387ada690b8a20482c694433f1

shill: Map crosbugs to their crbug equivalent

This makes things more consistent with the current issue URLs.

BUG=chromium:260016
TEST=Unit tests;
sed -i $(for crosbug in $(grep 'crosbug.com/[0-9]' *.cc *.h | sed -e 's#.*crosbug.com/\([0-9]*\).*#\1#' | sort | uniq); do crbug=$(curl -s "https://code.google.com/p/chromium-os/issues/detail?id=$crosbug" | grep HREF | sed -e 's/.*id=\([0-9]*\).*/\1/'); echo "-e s#crosbug.com/$crosbug#crbug.com/$crbug#"; done) *.cc *.h

Change-Id: I28e31c6c30186dd2a7cd1b36a170c33c2797d97f
Reviewed-on: https://gerrit.chromium.org/gerrit/61794
Commit-Queue: Paul Stewart <pstew@chromium.org>
Reviewed-by: Paul Stewart <pstew@chromium.org>
Tested-by: Paul Stewart <pstew@chromium.org>

manager_unittest.cc

diff --git a/manager_unittest.cc b/manager_unittest.cc
index eaf68d7..e2f5aca 100644
--- a/manager_unittest.cc
+++ b/manager_unittest.cc
@@ -525,7 +525,7 @@
       .WillRepeatedly(Return(service1_name));
   EXPECT_CALL(*mock_service2.get(), GetRpcIdentifier())
       .WillRepeatedly(Return(service2_name));
-  // TODO(quiche): make this EXPECT_CALL work (crosbug.com/20154)
+  // TODO(quiche): make this EXPECT_CALL work (crbug.com/203247)
   // EXPECT_CALL(*dynamic_cast<ManagerMockAdaptor *>(manager.adaptor_.get()),
   //             EmitRpcIdentifierArrayChanged(flimflam::kServicesProperty, _));
 
@@ -2216,7 +2216,7 @@
 TEST_F(ManagerTest, SortServices) {
   // TODO(quiche): Some of these tests would probably fit better in
   // service_unittest, since the actual comparison of Services is
-  // implemented in Service. (crosbug.com/23370)
+  // implemented in Service. (crbug.com/206367)
 
   scoped_refptr<MockService> mock_service0(
       new NiceMock<MockService>(control_interface(),
@@ -2646,7 +2646,7 @@
   manager()->UpdateService(mock_service);
   // We can't EXPECT_CALL(..., MakeFavorite), because that requires us
   // to mock out MakeFavorite. And mocking that out would break the
-  // SortServices test. (crosbug.com/23370)
+  // SortServices test. (crbug.com/206367)
   EXPECT_TRUE(mock_service->favorite());
   EXPECT_TRUE(mock_service->auto_connect());
 }
@@ -2677,7 +2677,7 @@
   manager()->UpdateService(mock_service);
   // We can't EXPECT_CALL(..., MakeFavorite), because that requires us
   // to mock out MakeFavorite. And mocking that out would break the
-  // SortServices test. (crosbug.com/23370)
+  // SortServices test. (crbug.com/206367)
   EXPECT_TRUE(mock_service->favorite());
   EXPECT_TRUE(mock_service->auto_connect());
   // This releases the ref on the mock profile.