Commit a8edacba authored by Chunbo Hua's avatar Chunbo Hua Committed by Commit Bot

Replace ComPtr::GetAddressOf() usage with "&" operator

Ban use of ComPtr::GetAddressOf() according to http://crbug.com/914910.

Bug: 914910
Change-Id: I4287f7acd6969d7f236f4dc64fb6a1c76a8d5c9e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2148673
Commit-Queue: Scott Violet <sky@chromium.org>
Reviewed-by: default avatarScott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#759256}
parent 9ca0661e
...@@ -38,7 +38,7 @@ void CreateDWriteFactory(IDWriteFactory** factory) { ...@@ -38,7 +38,7 @@ void CreateDWriteFactory(IDWriteFactory** factory) {
Microsoft::WRL::ComPtr<IUnknown> factory_unknown; Microsoft::WRL::ComPtr<IUnknown> factory_unknown;
HRESULT hr = HRESULT hr =
DWriteCreateFactory(DWRITE_FACTORY_TYPE_SHARED, __uuidof(IDWriteFactory), DWriteCreateFactory(DWRITE_FACTORY_TYPE_SHARED, __uuidof(IDWriteFactory),
factory_unknown.GetAddressOf()); &factory_unknown);
if (FAILED(hr)) { if (FAILED(hr)) {
base::debug::Alias(&hr); base::debug::Alias(&hr);
CHECK(false); CHECK(false);
...@@ -55,7 +55,7 @@ void InitializeDirectWrite() { ...@@ -55,7 +55,7 @@ void InitializeDirectWrite() {
TRACE_EVENT0("fonts", "gfx::InitializeDirectWrite"); TRACE_EVENT0("fonts", "gfx::InitializeDirectWrite");
Microsoft::WRL::ComPtr<IDWriteFactory> factory; Microsoft::WRL::ComPtr<IDWriteFactory> factory;
CreateDWriteFactory(factory.GetAddressOf()); CreateDWriteFactory(&factory);
CHECK(!!factory); CHECK(!!factory);
SetDirectWriteFactory(factory.Get()); SetDirectWriteFactory(factory.Get());
......
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