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

Issue 8974033: Fix the GURL change [compile error, two small changes to make things safe while (Closed)

Created:
8 years, 11 months ago by ericu
Modified:
8 years, 8 months ago
Reviewers:
brettw
CC:
chromium-reviews
Visibility:
Public.

Description

Fix the GURL change [compile error, two small changes to make things safe while the rest of the changes aren't in yet].

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+26 lines, -24 lines) Patch
M src/gurl.cc View 1 chunk +1 line, -1 line 0 comments Download
M src/url_canon_unittest.cc View 2 chunks +2 lines, -0 lines 0 comments Download
M src/url_parse.cc View 2 chunks +23 lines, -23 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
ericu
Brett, here's what looks like a fix for the build breakage, and two other issues.
8 years, 11 months ago (2012-01-05 00:29:29 UTC) #1
brettw
lgtm
8 years, 11 months ago (2012-01-05 00:30:37 UTC) #2
brettw
8 years, 11 months ago (2012-01-05 00:31:36 UTC) #3
Checked in as googleurl@167, I did not roll deps.

Powered by Google App Engine
This is Rietveld 408576698