Issue 961593002: Merge: Update the Drive state in Files app when the refresh token is fetched.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Issue 961593002: Merge: Update the Drive state in Files app when the refresh token is fetched. (Closed)

Created:
5 years, 10 months ago by mtomasz

Modified:
5 years, 10 months ago

Reviewers:
hirono

CC:
chromium-reviews, extensions-reviews_chromium.org, rginda+watch_chromium.org, oshima+watch_chromium.org, chromium-apps-reviews_chromium.org, stevenjb+watch_chromium.org, davemoore+watch_chromium.org

Base URL:
https://chromium.googlesource.com/chromium/src.git@2311

Target Ref:
refs/pending/branch-heads/2311

Project:
chromium

Visibility:
Public.

More Reviews

Description

Merge: Update the Drive state in Files app when the refresh token is fetched. Before, we were not notifying Files app about such fact, so it was still thinking that Drive is not available, and showing the banner. TBR=hirono@chromium.org # For merge. TEST=Tested manually when logging to an empty profile. BUG=461247 Review URL: https://codereview.chromium.org/953273002 Cr-Commit-Position: refs/heads/master@{#317995} (cherry picked from commit eaffcdbc813caebe3160789ed9ecc503b5276926) Committed: https://chromium.googlesource.com/chromium/src/+/61c27c926461d801ba41ace1e819edca73e5ee5b

Patch Set 1 #

Created: 5 years, 10 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+11 lines, -0 lines)			Patch
	M	chrome/browser/chromeos/extensions/file_manager/event_router.h	View		1 chunk	+1 line, -0 lines	0 comments	Download
	M	chrome/browser/chromeos/extensions/file_manager/event_router.cc	View		1 chunk	+10 lines, -0 lines	0 comments	Download

Messages

Total messages: 1 (0 generated)

Expand Messages | Collapse Messages

mtomasz

5 years, 10 months ago (2015-02-26 05:01:21 UTC) #1

Message was sent while issue was closed.

Committed patchset #1 (id:1) manually as
61c27c926461d801ba41ace1e819edca73e5ee5b (presubmit successful).

Expand Messages | Collapse Messages