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

Issue 6350008: Create a new code path to generate sync strings for the new tab page differen... (Closed)

Created:
9 years, 11 months ago by John Gregg
Modified:
9 years ago
CC:
chromium-reviews, Raghu Simha, ncarter (slow), tim (not reviewing), idana
Visibility:
Public.

Description

Create a new code path to generate sync strings for the new tab page different from those used on the settings' page. This will allow us to show on the settings page the current sync status as well as a message regarding passwords. BUG=69622 TEST=see bug Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=72171

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+67 lines, -8 lines) Patch
M chrome/app/generated_resources.grd View 1 chunk +7 lines, -0 lines 0 comments Download
M chrome/browser/dom_ui/new_tab_page_sync_handler.cc View 1 chunk +3 lines, -1 line 0 comments Download
M chrome/browser/sync/sync_ui_util.h View 1 chunk +5 lines, -0 lines 0 comments Download
M chrome/browser/sync/sync_ui_util.cc View 3 chunks +52 lines, -7 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
John Gregg
9 years, 11 months ago (2011-01-21 08:20:37 UTC) #1
tim (not reviewing)
9 years, 11 months ago (2011-01-21 17:44:10 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698