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

Issue 6332012: Re-submitting the cl r72292, while removing incorrect DCHECK()... (Closed)

Created:
9 years, 11 months ago by GeorgeY
Modified:
9 years, 6 months ago
Reviewers:
Ilya Sherman, dhollowa
CC:
chromium-reviews, Ilya Sherman, Paweł Hajdan Jr., James Hawkins, dhollowa
Visibility:
Public.

Description

Re-submitting the cl r72292, while removing incorrect DCHECK() Propagate correct data to the Toolbar servers BUG=67219 TEST=unit-tested Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=72523

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+285 lines, -47 lines) Patch
M chrome/browser/autofill/form_structure.h View 2 chunks +5 lines, -2 lines 0 comments Download
M chrome/browser/autofill/form_structure.cc View 6 chunks +59 lines, -20 lines 0 comments Download
M chrome/browser/autofill/form_structure_unittest.cc View 4 chunks +221 lines, -25 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
GeorgeY
9 years, 11 months ago (2011-01-25 01:06:17 UTC) #1
Ilya Sherman
+David I would prefer to change the unit tests if possible, as this seems like ...
9 years, 11 months ago (2011-01-25 01:10:18 UTC) #2
dhollowa
9 years, 11 months ago (2011-01-25 01:54:45 UTC) #3
LGTM.  George please re-introduce DCHECKs and fixes for unit tests in a separate
CL.  Thanks.

Powered by Google App Engine
This is Rietveld 408576698