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

Side by Side Diff: chromeos/dbus/session_manager_client.cc

Issue 1966133002: Run RemoveArcData after a user has opted out (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Adressed comments Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « chromeos/dbus/session_manager_client.h ('k') | components/arc.gypi » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chromeos/dbus/session_manager_client.h" 5 #include "chromeos/dbus/session_manager_client.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <stdint.h> 8 #include <stdint.h>
9 #include <sys/socket.h> 9 #include <sys/socket.h>
10 10
(...skipping 362 matching lines...) Expand 10 before | Expand all | Expand 10 after
373 dbus::MethodCall method_call( 373 dbus::MethodCall method_call(
374 login_manager::kSessionManagerInterface, 374 login_manager::kSessionManagerInterface,
375 login_manager::kSessionManagerGetArcStartTimeTicks); 375 login_manager::kSessionManagerGetArcStartTimeTicks);
376 376
377 session_manager_proxy_->CallMethod( 377 session_manager_proxy_->CallMethod(
378 &method_call, dbus::ObjectProxy::TIMEOUT_USE_DEFAULT, 378 &method_call, dbus::ObjectProxy::TIMEOUT_USE_DEFAULT,
379 base::Bind(&SessionManagerClientImpl::OnGetArcStartTime, 379 base::Bind(&SessionManagerClientImpl::OnGetArcStartTime,
380 weak_ptr_factory_.GetWeakPtr(), callback)); 380 weak_ptr_factory_.GetWeakPtr(), callback));
381 } 381 }
382 382
383 void RemoveArcData(const cryptohome::Identification& cryptohome_id) override {
384 dbus::MethodCall method_call(login_manager::kSessionManagerInterface,
385 login_manager::kSessionManagerRemoveArcData);
386 dbus::MessageWriter writer(&method_call);
387 writer.AppendString(cryptohome_id.id());
388 session_manager_proxy_->CallMethod(
389 &method_call, dbus::ObjectProxy::TIMEOUT_USE_DEFAULT,
390 base::Bind(&SessionManagerClientImpl::OnArcMethod,
391 weak_ptr_factory_.GetWeakPtr(),
392 login_manager::kSessionManagerRemoveArcData, ArcCallback()));
393 }
394
383 protected: 395 protected:
384 void Init(dbus::Bus* bus) override { 396 void Init(dbus::Bus* bus) override {
385 session_manager_proxy_ = bus->GetObjectProxy( 397 session_manager_proxy_ = bus->GetObjectProxy(
386 login_manager::kSessionManagerServiceName, 398 login_manager::kSessionManagerServiceName,
387 dbus::ObjectPath(login_manager::kSessionManagerServicePath)); 399 dbus::ObjectPath(login_manager::kSessionManagerServicePath));
388 blocking_method_caller_.reset( 400 blocking_method_caller_.reset(
389 new BlockingMethodCaller(bus, session_manager_proxy_)); 401 new BlockingMethodCaller(bus, session_manager_proxy_));
390 402
391 // Signals emitted on the session manager's interface. 403 // Signals emitted on the session manager's interface.
392 session_manager_proxy_->ConnectToSignal( 404 session_manager_proxy_->ConnectToSignal(
(...skipping 518 matching lines...) Expand 10 before | Expand all | Expand 10 after
911 } 923 }
912 924
913 void StopArcInstance(const ArcCallback& callback) override { 925 void StopArcInstance(const ArcCallback& callback) override {
914 callback.Run(false); 926 callback.Run(false);
915 } 927 }
916 928
917 void GetArcStartTime(const GetArcStartTimeCallback& callback) override { 929 void GetArcStartTime(const GetArcStartTimeCallback& callback) override {
918 callback.Run(false, base::TimeTicks::Now()); 930 callback.Run(false, base::TimeTicks::Now());
919 } 931 }
920 932
933 void RemoveArcData(const cryptohome::Identification& cryptohome_id) override {
934 }
935
921 private: 936 private:
922 StubDelegate* delegate_; // Weak pointer; may be NULL. 937 StubDelegate* delegate_; // Weak pointer; may be NULL.
923 base::ObserverList<Observer> observers_; 938 base::ObserverList<Observer> observers_;
924 std::string device_policy_; 939 std::string device_policy_;
925 bool screen_is_locked_; 940 bool screen_is_locked_;
926 941
927 DISALLOW_COPY_AND_ASSIGN(SessionManagerClientStubImpl); 942 DISALLOW_COPY_AND_ASSIGN(SessionManagerClientStubImpl);
928 }; 943 };
929 944
930 SessionManagerClient::SessionManagerClient() { 945 SessionManagerClient::SessionManagerClient() {
931 } 946 }
932 947
933 SessionManagerClient::~SessionManagerClient() { 948 SessionManagerClient::~SessionManagerClient() {
934 } 949 }
935 950
936 SessionManagerClient* SessionManagerClient::Create( 951 SessionManagerClient* SessionManagerClient::Create(
937 DBusClientImplementationType type) { 952 DBusClientImplementationType type) {
938 if (type == REAL_DBUS_CLIENT_IMPLEMENTATION) 953 if (type == REAL_DBUS_CLIENT_IMPLEMENTATION)
939 return new SessionManagerClientImpl(); 954 return new SessionManagerClientImpl();
940 DCHECK_EQ(STUB_DBUS_CLIENT_IMPLEMENTATION, type); 955 DCHECK_EQ(STUB_DBUS_CLIENT_IMPLEMENTATION, type);
941 return new SessionManagerClientStubImpl(); 956 return new SessionManagerClientStubImpl();
942 } 957 }
943 958
944 } // namespace chromeos 959 } // namespace chromeos
OLDNEW
« no previous file with comments | « chromeos/dbus/session_manager_client.h ('k') | components/arc.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698