New issue
Advanced search Search tips

Issue 709371 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: May 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

authpolicy: Switch AuthenticateUser D-Bus call back to simple

Project Member Reported by ljusten@chromium.org, Apr 7 2017

Issue description

Search for this bug in platform2. Wait until Chrome side changed.

 
Labels: -Pri-3 Pri-1
Labels: -Pri-1 Pri-2
Status: Started (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, May 19 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/0ae32d31c8033ead1d75e62f002135ab27dcf22e

commit 0ae32d31c8033ead1d75e62f002135ab27dcf22e
Author: Lutz Justen <ljusten@chromium.org>
Date: Fri May 19 18:57:49 2017

authpolicy: Switch AuthenticateUser D-Bus call back to simple

During a transition phase AuthenticateUser handled missing account_id
parameters transparently while Chrome wasn't switched over to a new
interface that sends account_id. Chrome uses the new interface now,
so the workaround can be removed.

BUG= chromium:709371 
TEST=Tested on device, did not explode

Change-Id: I0f2612f59d540afae9c3c04b28aeeceb0814d5c2
Reviewed-on: https://chromium-review.googlesource.com/509710
Commit-Ready: Lutz Justen <ljusten@chromium.org>
Tested-by: Lutz Justen <ljusten@chromium.org>
Reviewed-by: Roman Sorokin <rsorokin@chromium.org>

[modify] https://crrev.com/0ae32d31c8033ead1d75e62f002135ab27dcf22e/authpolicy/authpolicy.cc
[modify] https://crrev.com/0ae32d31c8033ead1d75e62f002135ab27dcf22e/authpolicy/authpolicy.h
[modify] https://crrev.com/0ae32d31c8033ead1d75e62f002135ab27dcf22e/authpolicy/dbus_bindings/org.chromium.AuthPolicy.xml

Status: Fixed (was: Started)
Status: Verified (was: Fixed)
bulk Verify of older or not-user-facing Chromad bugs

Sign in to add a comment