Commit 0cda37ff authored by thakis@chromium.org's avatar thakis@chromium.org

mac: Let gcapi install chrome as the CGSession owner

This way, chrome should be able to set up user keystone
on first run in most cases.

Also don't offer installation if /Applications/Google Chrome.app already
exists.

BUG=128462
TEST=
1.) Run as user.
Have chrome with system ticket and user keystone installed anywhere.
  -> Shouldn't offer installation.
Have chrome with user ticket and user keystone installed anywhere.
  -> Shouldn't offer installation.

Have chrome with system ticket and system keystone installed anywhere.
  -> Shouldn't offer installation.
Have chrome with system ticket and system keystone installed anywhere.
  -> Shouldn't offer installation.

Have chrome in /Applications, no matter owned by whom and no matter
if with keystone set up.
  -> Shouldn't offer installation.

Don't have chrome installed
  -> Offers to install, and creates /Applications/Google Chrome.app which
    is owned by the current user. On first launch, Chrome registers itself
    successfully for a user ticket without user interaction.

2.) Same as with 1, but run as root. Should have EXACTLY the same results,
i.e. chrome is still installed as current user, and when run as the current
user (not root) successfully registers itself.

Review URL: https://chromiumcodereview.appspot.com/10834102

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151324 0039d316-1c4b-4281-b951-d872f2087c98
parent 32bd0b6f
mark@chromium.org
thakis@chromium.org
......@@ -6,12 +6,11 @@
#define CHROME_INSTALLER_GCAPI_MAC_GCAPI_H_
// Error conditions for GoogleChromeCompatibilityCheck().
#define GCCC_ERROR_USERLEVELALREADYPRESENT (1 << 0)
#define GCCC_ERROR_SYSTEMLEVELALREADYPRESENT (1 << 1)
#define GCCC_ERROR_ACCESSDENIED (1 << 2)
#define GCCC_ERROR_OSNOTSUPPORTED (1 << 3)
#define GCCC_ERROR_ALREADYOFFERED (1 << 4)
#define GCCC_ERROR_INTEGRITYLEVEL (1 << 5)
#define GCCC_ERROR_ALREADYPRESENT (1 << 0)
#define GCCC_ERROR_ACCESSDENIED (1 << 1)
#define GCCC_ERROR_OSNOTSUPPORTED (1 << 2)
#define GCCC_ERROR_ALREADYOFFERED (1 << 3)
#define GCCC_ERROR_INTEGRITYLEVEL (1 << 4)
#ifdef __cplusplus
extern "C" {
......
This diff is collapsed.
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment