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

Issue 12466005: Do not use a relative path to launch the import process. (Closed)

Created:
7 years, 9 months ago by gab
Modified:
7 years, 9 months ago
Reviewers:
cpu_(ooo_6.6-7.5)
CC:
chromium-reviews
Visibility:
Public.

Description

Do not use a relative path to launch the import process. Chrome already changes the CWD to its directory when loading (in LoadChromeWithDirectory()); thus if the command-line used a relative path, that relative path is no longer valid. BUG=DCHECK because of failed import when running out\Debug\chrome.exe --force-first-run Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=186367

Patch Set 1 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -1 line) Patch
M chrome/browser/first_run/first_run_win.cc View 1 chunk +8 lines, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
gab
Carlos, please take a look. Thanks! Gab
7 years, 9 months ago (2013-03-05 18:28:26 UTC) #1
cpu_(ooo_6.6-7.5)
lgtm
7 years, 9 months ago (2013-03-06 01:33:40 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/gab@chromium.org/12466005/2001
7 years, 9 months ago (2013-03-06 03:31:47 UTC) #3
commit-bot: I haz the power
7 years, 9 months ago (2013-03-06 04:55:53 UTC) #4
Message was sent while issue was closed.
Change committed as 186367

Powered by Google App Engine
This is Rietveld 408576698