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

Unified Diff: chrome/browser/net/dns_host_info.cc

Issue 2890: Include chrome/browser/SConscript in Linux build.... (Closed) Base URL: http://src.chromium.org/svn/trunk/src/
Patch Set: '' Created 12 years, 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/net/dns_host_info.h ('k') | chrome/browser/printing/page_number.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/net/dns_host_info.cc
===================================================================
--- chrome/browser/net/dns_host_info.cc (revision 2262)
+++ chrome/browser/net/dns_host_info.cc (working copy)
@@ -202,7 +202,7 @@
static std::string RemoveJs(const std::string& text) {
std::string output(text);
size_t length = output.length();
- for (size_t i = 0; i < 1; i++) {
+ for (size_t i = 0; i < length; i++) {
char next = output[i];
if (isalnum(next) || isspace(next) || '.' == next)
continue;
@@ -279,8 +279,8 @@
return;
}
- char* row_format = "<tr align=right><td>%s</td>"
- "<td>%d</td><td>%d</td><td>%s</td></tr>";
+ const char* row_format = "<tr align=right><td>%s</td>"
+ "<td>%d</td><td>%d</td><td>%s</td></tr>";
output->append("<br><table border=1>");
StringAppendF(output, "<tr><th>%s</th><th>%s</th><th>%s</th><th>%s</th></tr>",
« no previous file with comments | « chrome/browser/net/dns_host_info.h ('k') | chrome/browser/printing/page_number.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698