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

Issue 572083002: Disable Hangouts Remote Desktop Integration until it passes security review (Closed)

Created:
6 years, 3 months ago by kelvinp
Modified:
6 years, 3 months ago
Reviewers:
Jamie
CC:
chromium-reviews, chromoting-reviews_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

Disable Hangouts Remote Desktop Integration until it passes security review Hangout Remote Desktop integration is no longer targetting M39. Since it hasn't gone through security review yet, we should disable the feature until we got a sign off from security. BUG=414463 Committed: https://crrev.com/7f47ffd4fff6546c8a9ee78704a5f245d28bd59f Cr-Commit-Position: refs/heads/master@{#294976}

Patch Set 1 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -3 lines) Patch
M remoting/webapp/background/background.js View 1 chunk +7 lines, -3 lines 0 comments Download

Messages

Total messages: 9 (4 generated)
kelvinp
PTAL
6 years, 3 months ago (2014-09-15 22:09:36 UTC) #4
Jamie
lgtm
6 years, 3 months ago (2014-09-15 23:42:24 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patchset/572083002/40001
6 years, 3 months ago (2014-09-16 00:37:41 UTC) #7
commit-bot: I haz the power
Committed patchset #1 (id:40001) as b96ea91de7be64cad82999b8350f27cf335770a5
6 years, 3 months ago (2014-09-16 02:13:09 UTC) #8
commit-bot: I haz the power
6 years, 3 months ago (2014-09-16 02:15:30 UTC) #9
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/7f47ffd4fff6546c8a9ee78704a5f245d28bd59f
Cr-Commit-Position: refs/heads/master@{#294976}

Powered by Google App Engine
This is Rietveld 408576698