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

Issue 10889051: Re-Merge 153610 - remoting_me2me_host: Don't try to display UI on top of Mac login screen (Closed)

Created:
8 years, 3 months ago by Wez
Modified:
8 years, 3 months ago
Reviewers:
Lambros
CC:
chromium-reviews, jamiewalch+watch_chromium.org, dcaiafa+watch_chromium.org, simonmorris+watch_chromium.org, hclam+watch_chromium.org, wez+watch_chromium.org, amit, sanjeevr, garykac+watch_chromium.org, lambroslambrou+watch_chromium.org, alexeypa+watch_chromium.org, sergeyu+watch_chromium.org
Visibility:
Public.

Description

Re-Merge 153610 - remoting_me2me_host: Don't try to display UI on top of Mac login screen r153610 depends upon an include that was added by an intermediate CL, and so broke the branch build; this re-land adds the missing include. Trying to display the Disconnect window causes rejection from the Window Server on OS X 10.7.4, preventing the capturer from working. BUG=140984 TEST=Verify Chromoting works on the system login screen. Review URL: https://chromiumcodereview.appspot.com/10883033 TBR=lambroslambrou@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=154036

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+25 lines, -5 lines) Patch
M remoting/host/remoting_me2me_host.cc View 3 chunks +25 lines, -5 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
Wez
8 years, 3 months ago (2012-08-30 01:06:07 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698