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

Issue 8491064: Add permission for relay.google.com to the chromoting webapp. (Closed)

Created:
9 years, 1 month ago by Sergey Ulanov
Modified:
9 years, 1 month ago
Reviewers:
Wez, Jamie
CC:
chromium-reviews, jamiewalch+watch_chromium.org, hclam+watch_chromium.org, simonmorris+watch_chromium.org, wez+watch_chromium.org, dmaclach+watch_chromium.org, garykac+watch_chromium.org, lambroslambrou+watch_chromium.org, ajwong+watch_chromium.org, sergeyu+watch_chromium.org
Visibility:
Public.

Description

Add permission for relay.google.com to the chromoting webapp. BUG=103992 TEST=Chromoting can use relay servers

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -1 line) Patch
M remoting/webapp/me2mom/manifest.json View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
Sergey Ulanov
9 years, 1 month ago (2011-11-12 01:28:35 UTC) #1
Wez
lgtm
9 years, 1 month ago (2011-11-12 02:19:29 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sergeyu@chromium.org/8491064/1
9 years, 1 month ago (2011-11-12 02:52:13 UTC) #3
Sergey Ulanov
9 years, 1 month ago (2011-11-14 18:17:15 UTC) #4
Actually, instead of adding this permission, I think the right thing to do would
be to change the API to allow cross-origin requests for trusted plugins - it is
the way URLLoader API works in M15. I'll create another CL for that.

Powered by Google App Engine
This is Rietveld 408576698