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

Issue 48903013: Merge 232142 "Revert 230138 "Delay wallpaper load by 2 * average..." (Closed)

Created:
7 years, 1 month ago by matthewyuan
Modified:
7 years, 1 month ago
Reviewers:
Nikita (slow)
CC:
chromium-reviews, davemoore+watch_chromium.org, stevenjb+watch_chromium.org, arv+watch_chromium.org, oshima+watch_chromium.org, nkostylev+watch_chromium.org
Visibility:
Public.

Description

Merge 232142 "Revert 230138 "Delay wallpaper load by 2 * average..." > Revert 230138 "Delay wallpaper load by 2 * average wallpaper loa..." > > Reason: regression on zgb/alex/mario resulting in login UI being invisible in certain cases. See http://crbug.com/313019 > > > Delay wallpaper load by 2 * average wallpaper load time. > > > > BUG=279102, 313019 > > > > TEST=manual > > > > Review URL: https://codereview.chromium.org/24625003 > > TBR=alemate@chromium.org > > Review URL: https://codereview.chromium.org/54333005 TBR=nkostylev@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=232151

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+53 lines, -257 lines) Patch
M chrome/browser/chromeos/login/wallpaper_manager.h View 5 chunks +0 lines, -18 lines 0 comments Download
M chrome/browser/chromeos/login/wallpaper_manager.cc View 2 chunks +0 lines, -14 lines 0 comments Download
M chrome/browser/resources/chromeos/login/screen_account_picker.js View 2 chunks +1 line, -9 lines 0 comments Download
M chrome/browser/resources/chromeos/login/user_pod_row.js View 9 chunks +51 lines, -17 lines 0 comments Download
D chrome/browser/resources/chromeos/login/wallpaper_loader.js View 1 chunk +0 lines, -185 lines 0 comments Download
M chrome/browser/ui/webui/chromeos/login/signin_screen_handler.h View 3 chunks +1 line, -6 lines 0 comments Download
M chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc View 2 chunks +0 lines, -8 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
matthewyuan
7 years, 1 month ago (2013-10-31 17:30:23 UTC) #1
matthewyuan
7 years, 1 month ago (2013-10-31 17:30:59 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r232151.

Powered by Google App Engine
This is Rietveld 408576698