Modify a favicon DCHECK.

chrome/browser/favicon/favicon_service.cc

Index: chrome/browser/favicon/favicon_service.cc
diff --git a/chrome/browser/favicon/favicon_service.cc b/chrome/browser/favicon/favicon_service.cc
index a70111e8d02f8fbbcd4401582824fb552929ed3b..aab1e893793378fdf82eef2f3b1957d025eb89a8 100644
--- a/chrome/browser/favicon/favicon_service.cc
+++ b/chrome/browser/favicon/favicon_service.cc
@@ -403,7 +403,7 @@ void FaviconService::RunFaviconRawBitmapCallbackWithBitmapResults(
     return;
   }
 
-  DCHECK_EQ(1u, favicon_bitmap_results.size());
+  DCHECK_LE(1u, favicon_bitmap_results.size());

[stevenjb, 2014/07/25 18:45:19]

We early exit above if favicon_bitmap_results.empty(), so this is effectively
the same test.

[stevenjb, 2014/07/25 18:47:31]

I *think* (it's been a long time) was that the expectation was that at this
point there would only be one valid result? That may have changed, so maybe a GE
test would be correct here?
+sky@

[Daniel Nishi, 2014/07/25 21:13:37]

Yeah, you're right. I've changed it back to the removed version.

   favicon_base::FaviconRawBitmapResult bitmap_result =
       favicon_bitmap_results[0];