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

Unified Diff: win8/test/ui_automation_client.cc

Issue 740653005: Remove implicit conversions from scoped_refptr to T* in win8/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address comments Created 6 years, 1 month 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « win8/metro_driver/ime/text_store.cc ('k') | win8/viewer/metro_viewer_process_host.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: win8/test/ui_automation_client.cc
diff --git a/win8/test/ui_automation_client.cc b/win8/test/ui_automation_client.cc
index bb5082efd992b45534d3d23d60e276cd4dc28187..6f2793cad8f8afc06733b1ce5089854483d561c7 100644
--- a/win8/test/ui_automation_client.cc
+++ b/win8/test/ui_automation_client.cc
@@ -249,11 +249,8 @@ HRESULT UIAutomationClient::Context::InstallWindowObserver() {
event_handler_obj);
result = automation_->AddAutomationEventHandler(
- UIA_Window_WindowOpenedEventId,
- root_element,
- TreeScope_Descendants,
- cache_request,
- event_handler);
+ UIA_Window_WindowOpenedEventId, root_element.get(), TreeScope_Descendants,
+ cache_request.get(), event_handler.get());
if (FAILED(result)) {
LOG(ERROR) << std::hex << result;
@@ -281,9 +278,7 @@ HRESULT UIAutomationClient::Context::RemoveWindowObserver() {
}
result = automation_->RemoveAutomationEventHandler(
- UIA_Window_WindowOpenedEventId,
- root_element,
- event_handler_);
+ UIA_Window_WindowOpenedEventId, root_element.get(), event_handler_.get());
if (FAILED(result)) {
LOG(ERROR) << std::hex << result;
return result;
@@ -423,9 +418,7 @@ HRESULT UIAutomationClient::Context::InvokeDesiredItem(
result = element->FindFirstBuildCache(
static_cast<TreeScope>(TreeScope_Children | TreeScope_Descendants),
- condition,
- cache_request,
- target.Receive());
+ condition.get(), cache_request.get(), target.Receive());
if (FAILED(result)) {
LOG(ERROR) << std::hex << result;
return result;
@@ -489,8 +482,9 @@ HRESULT UIAutomationClient::Context::GetInvokableItems(
return result;
}
- result = automation_->CreateAndCondition(
- invokable_condition, control_view_condition, condition.Receive());
+ result = automation_->CreateAndCondition(invokable_condition.get(),
+ control_view_condition.get(),
+ condition.Receive());
if (FAILED(result)) {
LOG(ERROR) << std::hex << result;
return result;
@@ -506,9 +500,7 @@ HRESULT UIAutomationClient::Context::GetInvokableItems(
result = element->FindAllBuildCache(
static_cast<TreeScope>(TreeScope_Children | TreeScope_Descendants),
- condition,
- cache_request,
- element_array.Receive());
+ condition.get(), cache_request.get(), element_array.Receive());
if (FAILED(result)) {
LOG(ERROR) << std::hex << result;
return result;
« no previous file with comments | « win8/metro_driver/ime/text_store.cc ('k') | win8/viewer/metro_viewer_process_host.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698