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

Issue 49463006: chrome.windows.create should return new window id in Guest mode (Closed)

Created:
7 years, 1 month ago by Dmitry Polukhin
Modified:
7 years, 1 month ago
Reviewers:
Matt Perry
CC:
chromium-reviews, chromium-apps-reviews_chromium.org, extensions-reviews_chromium.org
Visibility:
Public.

Description

chrome.windows.create should return new window id in Guest mode Some context: Chrome OS Guest mode is incognito mode and user extensions are disabled but we have couple component extensions that work in Guest mode. QuickOffice is one of them. It uses incognito split mode and runs in incognito profile. CanCrossIncognito returns false if extension runs in not split mode. Therefore include_incognito is false. But in general there is no problem if chrome.windows.create return incognito window if extension itself already works in incognito mode. BUG=313789 TEST=manual Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=232931

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -2 lines) Patch
M chrome/browser/extensions/api/tabs/tabs_api.cc View 1 chunk +4 lines, -2 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
Dmitry Polukhin
PTAL
7 years, 1 month ago (2013-11-02 00:31:02 UTC) #1
Dmitry Polukhin
Friendly ping, please take a look.
7 years, 1 month ago (2013-11-04 20:00:54 UTC) #2
Matt Perry
lgtm
7 years, 1 month ago (2013-11-04 21:14:33 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dpolukhin@chromium.org/49463006/1
7 years, 1 month ago (2013-11-04 23:30:05 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dpolukhin@chromium.org/49463006/1
7 years, 1 month ago (2013-11-05 03:06:20 UTC) #5
commit-bot: I haz the power
7 years, 1 month ago (2013-11-05 06:48:04 UTC) #6
Message was sent while issue was closed.
Change committed as 232931

Powered by Google App Engine
This is Rietveld 408576698