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

Issue 552743005: Reset the experimental app list to the start page on dismiss. (Closed)

Created:
6 years, 3 months ago by calamity
Modified:
6 years, 3 months ago
Reviewers:
Matt Giuca
CC:
chromium-reviews, chrome-apps-syd-reviews_chromium.org, tfarina
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

Reset the experimental app list to the start page on dismiss. This CL fixes and issue where the experimental app list would not reset to the start page on Windows and Linux. BUG=412227 Committed: https://crrev.com/8a83898033242f000c27781cc6050d3464bf24d3 Cr-Commit-Position: refs/heads/master@{#294307}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -0 lines) Patch
M ui/app_list/views/app_list_main_view.cc View 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 11 (4 generated)
calamity
6 years, 3 months ago (2014-09-09 09:20:16 UTC) #2
Matt Giuca
lgtm
6 years, 3 months ago (2014-09-10 04:58:53 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/calamity@chromium.org/552743005/1
6 years, 3 months ago (2014-09-10 05:03:15 UTC) #5
commit-bot: I haz the power
Try jobs failed on following builders: android_aosp on tryserver.chromium.linux (http://build.chromium.org/p/tryserver.chromium.linux/builders/android_aosp/builds/13827)
6 years, 3 months ago (2014-09-10 05:13:54 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patchset/552743005/1
6 years, 3 months ago (2014-09-11 00:04:11 UTC) #9
commit-bot: I haz the power
Committed patchset #1 (id:1) as c7b2fbe91864099d381ed97b88b29fb846ac7c63
6 years, 3 months ago (2014-09-11 03:07:18 UTC) #10
commit-bot: I haz the power
6 years, 3 months ago (2014-09-11 03:11:44 UTC) #11
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/8a83898033242f000c27781cc6050d3464bf24d3
Cr-Commit-Position: refs/heads/master@{#294307}

Powered by Google App Engine
This is Rietveld 408576698