diff options
author | gab <gab@chromium.org> | 2015-09-02 07:44:10 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-09-02 14:44:50 +0000 |
commit | c62bb4ac21755405ec6b8eb9379b2b551478df26 (patch) | |
tree | ff82990175a4c58cb12b8132bac3f1cb5a71c66f /win8 | |
parent | 3adf304a1210759ccb20edc2bee8f59e48f03712 (diff) | |
download | chromium_src-c62bb4ac21755405ec6b8eb9379b2b551478df26.zip chromium_src-c62bb4ac21755405ec6b8eb9379b2b551478df26.tar.gz chromium_src-c62bb4ac21755405ec6b8eb9379b2b551478df26.tar.bz2 |
Make GetCommandExecuteImplClsid return the CLSID as a string directly.
BUG=Unjustified single out-param.
Review URL: https://codereview.chromium.org/1321143002
Cr-Commit-Position: refs/heads/master@{#346915}
Diffstat (limited to 'win8')
-rw-r--r-- | win8/delegate_execute/delegate_execute.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/win8/delegate_execute/delegate_execute.cc b/win8/delegate_execute/delegate_execute.cc index a41fb5b..01e1fd4 100644 --- a/win8/delegate_execute/delegate_execute.cc +++ b/win8/delegate_execute/delegate_execute.cc @@ -46,10 +46,10 @@ class DelegateExecuteModule HRESULT PreMessageLoop(int nShowCmd) { HRESULT hr = S_OK; - base::string16 clsid_string; GUID clsid; BrowserDistribution* dist = BrowserDistribution::GetDistribution(); - if (!dist->GetCommandExecuteImplClsid(&clsid_string)) + const base::string16 clsid_string = dist->GetCommandExecuteImplClsid(); + if (clsid_string.empty()) return E_FAIL; hr = ::CLSIDFromString(clsid_string.c_str(), &clsid); if (FAILED(hr)) |