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

Issue 22256002: Remove tabCapture about:flags switch. (Closed)

Created:
7 years, 4 months ago by justinlin
Modified:
7 years, 4 months ago
Reviewers:
sky, miu
CC:
chromium-reviews, chromium-apps-reviews_chromium.org, feature-media-reviews_chromium.org, extensions-reviews_chromium.org, miu+watch_chromium.org
Visibility:
Public.

Description

Remove tabCapture about:flags switch. Feature is stable and this flag has been defaulted to enabled for several milestones. Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=215813 BUG=

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -50 lines) Patch
M chrome/app/generated_resources.grd View 1 chunk +0 lines, -6 lines 0 comments Download
M chrome/browser/about_flags.cc View 1 chunk +0 lines, -8 lines 0 comments Download
M chrome/browser/extensions/api/tab_capture/tab_capture_api.cc View 3 chunks +0 lines, -11 lines 0 comments Download
M chrome/browser/extensions/api/tab_capture/tab_capture_apitest.cc View 3 chunks +0 lines, -9 lines 0 comments Download
M chrome/browser/media/media_capture_devices_dispatcher.cc View 3 chunks +0 lines, -3 lines 0 comments Download
M chrome/common/chrome_switches.cc View 1 chunk +0 lines, -3 lines 0 comments Download
M chrome/common/extensions/feature_switch.cc View 2 chunks +2 lines, -10 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
justinlin
7 years, 4 months ago (2013-08-05 20:27:11 UTC) #1
justinlin
7 years, 4 months ago (2013-08-05 20:34:15 UTC) #2
miu
lgtm
7 years, 4 months ago (2013-08-05 22:56:31 UTC) #3
sky
LGTM
7 years, 4 months ago (2013-08-05 23:58:07 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/justinlin@chromium.org/22256002/1
7 years, 4 months ago (2013-08-06 00:04:06 UTC) #5
commit-bot: I haz the power
7 years, 4 months ago (2013-08-06 06:09:24 UTC) #6
Message was sent while issue was closed.
Change committed as 215813

Powered by Google App Engine
This is Rietveld 408576698