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

Issue 1944293004: [MD settings] url change for passwords and languages (Closed)

Created:
4 years, 7 months ago by dschuyler
Modified:
4 years, 4 months ago
Reviewers:
tommycli
CC:
chromium-reviews, michaelpg+watch-md-settings_chromium.org, michaelpg+watch-md-ui_chromium.org, dbeam+watch-settings_chromium.org, stevenjb+watch-md-settings_chromium.org, arv+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

[MD settings] url change for passwords and languages This CL changes the root section URLs for passwords and languages, replacing those URLs previously used for managing passwords and languages with URLs that have a manage* prefix. (It's shuffling names around). BUG=606876 CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation Committed: https://crrev.com/0770a99fd400547704026a9ab6b0f6e5c84aff60 Cr-Commit-Position: refs/heads/master@{#392118}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -4 lines) Patch
M chrome/browser/resources/settings/settings_page/settings_router.js View 1 chunk +4 lines, -4 lines 0 comments Download

Messages

Total messages: 10 (4 generated)
dschuyler
4 years, 7 months ago (2016-05-04 23:51:55 UTC) #3
tommycli
lgtm
4 years, 7 months ago (2016-05-06 17:49:51 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1944293004/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1944293004/1
4 years, 7 months ago (2016-05-06 18:25:10 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 7 months ago (2016-05-06 19:22:30 UTC) #7
commit-bot: I haz the power
Patchset 1 (id:??) landed as https://crrev.com/0770a99fd400547704026a9ab6b0f6e5c84aff60 Cr-Commit-Position: refs/heads/master@{#392118}
4 years, 7 months ago (2016-05-06 19:24:10 UTC) #9
Johnashley1066
4 years, 4 months ago (2016-08-05 12:15:13 UTC) #10
Message was sent while issue was closed.
On 2016/05/06 19:24:10, commit-bot: I haz the power wrote:
> Patchset 1 (id:??) landed as
> https://crrev.com/0770a99fd400547704026a9ab6b0f6e5c84aff60
> Cr-Commit-Position: refs/heads/master@{#392118}
This CL changes the root section URLs for passwords and languages,
replacing those URLs previously used for managing passwords and
languages with URLs that have a manage* prefix.
http://www.qwiknumbers.com/bt-customer-services

Powered by Google App Engine
This is Rietveld 408576698