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

Issue 240333011: Always load signin profile on Chrome OS startup. (Closed)

Created:
6 years, 8 months ago by dzhioev (left Google)
Modified:
5 years, 9 months ago
Reviewers:
Nikita (slow)
CC:
chromium-reviews, stevenjb+watch_chromium.org, davemoore+watch_chromium.org, oshima+watch_chromium.org, nkostylev+watch_chromium.org
Visibility:
Public.

Description

Always load signin profile on Chrome OS startup. BUG=346763 COLLABORATOR=ginkage@chromium.org

Patch Set 1 #

Patch Set 2 : Fixed several tests. #

Patch Set 3 : Next iteration. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+39 lines, -29 lines) Patch
M chrome/browser/chromeos/chrome_browser_main_chromeos.cc View 1 2 1 chunk +6 lines, -0 lines 0 comments Download
M chrome/browser/chromeos/login/oobe_base_test.cc View 1 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/chromeos/profiles/profile_helper.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/extensions/api/file_system/file_system_apitest_chromeos.cc View 1 2 chunks +9 lines, -0 lines 0 comments Download
M chrome/browser/extensions/extension_browsertest.h View 1 1 chunk +0 lines, -6 lines 0 comments Download
M chrome/browser/extensions/extension_browsertest.cc View 1 2 2 chunks +0 lines, -15 lines 0 comments Download
M chrome/browser/profiles/profile_manager.cc View 1 2 1 chunk +4 lines, -0 lines 0 comments Download
M chrome/browser/safe_browsing/safe_browsing_service_browsertest.cc View 1 2 2 chunks +12 lines, -0 lines 0 comments Download
M chrome/test/data/extensions/api_test/file_browser/mount_test/test.js View 1 2 5 chunks +6 lines, -6 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
dzhioev (left Google)
6 years, 8 months ago (2014-04-17 15:23:41 UTC) #1
Nikita (slow)
lgtm
6 years, 8 months ago (2014-04-17 16:08:04 UTC) #2
dzhioev (left Google)
The CQ bit was checked by dzhioev@chromium.org
6 years, 8 months ago (2014-04-17 17:36:40 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dzhioev@chromium.org/240333011/1
6 years, 8 months ago (2014-04-17 17:37:31 UTC) #4
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 8 months ago (2014-04-17 18:50:39 UTC) #5
commit-bot: I haz the power
6 years, 8 months ago (2014-04-17 18:50:39 UTC) #6
Try jobs failed on following builders:
  tryserver.chromium on linux_chromium_chromeos_rel

Powered by Google App Engine
This is Rietveld 408576698