Make GetCommandExecuteImplClsid return the CLSID as a string directly.

Make GetCommandExecuteImplClsid return the CLSID as a string directly.

BUG=Unjustified single out-param.

Committed: https://crrev.com/c62bb4ac21755405ec6b8eb9379b2b551478df26
Cr-Commit-Position: refs/heads/master@{#346915}

[gab, 2015-08-29 14:37:53]

Patchset #1 (id:1) has been deleted

[gab, 2015-08-29 14:38:52]

gab@chromium.org changed reviewers:
+ grt@chromium.org

[gab, 2015-08-29 14:38:53]

Greg: as discussed on https://codereview.chromium.org/1294863006/, PTAL.

[gab, 2015-09-01 16:21:50]

Patchset #2 (id:40001) has been deleted

[gab, 2015-09-01 16:22:06]

On 2015/08/29 14:38:53, gab wrote:
> Greg: as discussed on https://codereview.chromium.org/1294863006/, PTAL.

ping, thanks!

[grt (UTC plus 2), 2015-09-01 20:02:27]

good old google_chrome_distribution_dummy.cc!

lgtm w/ below changes. thanks for this cleanup!

https://codereview.chromium.org/1321143002/diff/20001/chrome/installer/util/c...
File chrome/installer/util/chrome_frame_distribution.cc (right):

https://codereview.chromium.org/1321143002/diff/20001/chrome/installer/util/c...
chrome/installer/util/chrome_frame_distribution.cc:112: return L"";
  return base::string16();

https://codereview.chromium.org/1321143002/diff/20001/chrome/installer/util/c...
File chrome/installer/util/chromium_binaries_distribution.cc (right):

https://codereview.chromium.org/1321143002/diff/20001/chrome/installer/util/c...
chrome/installer/util/chromium_binaries_distribution.cc:110: return L"";
  return base::string16;

https://codereview.chromium.org/1321143002/diff/20001/chrome/installer/util/g...
File chrome/installer/util/google_chrome_distribution.cc (right):

https://codereview.chromium.org/1321143002/diff/20001/chrome/installer/util/g...
chrome/installer/util/google_chrome_distribution.cc:239: base::string16
GoogleChromeDistribution::GetCommandExecuteImplClsid() {
this needs to be updated in google_chrome_distribution_dummy.cc, too

[gab, 2015-09-02 14:23:00]

Thanks, comments addressed.

https://codereview.chromium.org/1321143002/diff/20001/chrome/installer/util/c...
File chrome/installer/util/chrome_frame_distribution.cc (right):

https://codereview.chromium.org/1321143002/diff/20001/chrome/installer/util/c...
chrome/installer/util/chrome_frame_distribution.cc:112: return L"";
On 2015/09/01 20:02:27, grt wrote:
>   return base::string16();

Done.

https://codereview.chromium.org/1321143002/diff/20001/chrome/installer/util/c...
File chrome/installer/util/chromium_binaries_distribution.cc (right):

https://codereview.chromium.org/1321143002/diff/20001/chrome/installer/util/c...
chrome/installer/util/chromium_binaries_distribution.cc:110: return L"";
On 2015/09/01 20:02:27, grt wrote:
>   return base::string16;

Done.

https://codereview.chromium.org/1321143002/diff/20001/chrome/installer/util/g...
File chrome/installer/util/google_chrome_distribution.cc (right):

https://codereview.chromium.org/1321143002/diff/20001/chrome/installer/util/g...
chrome/installer/util/google_chrome_distribution.cc:239: base::string16
GoogleChromeDistribution::GetCommandExecuteImplClsid() {
On 2015/09/01 20:02:27, grt wrote:
> this needs to be updated in google_chrome_distribution_dummy.cc, too

Done.

[gab, 2015-09-02 14:23:03]

The CQ bit was checked by gab@chromium.org

[gab, 2015-09-02 14:23:03]

The patchset sent to the CQ was uploaded after l-g-t-m from grt@chromium.org
Link to the patchset: https://codereview.chromium.org/1321143002/#ps60001
(title: "review:grt")

[commit-bot: I haz the power, 2015-09-02 14:23:10]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1321143002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1321143002/60001

[commit-bot: I haz the power, 2015-09-02 14:44:18]

Committed patchset #2 (id:60001)

[commit-bot: I haz the power, 2015-09-02 14:45:04]

Patchset 2 (id:??) landed as
https://crrev.com/c62bb4ac21755405ec6b8eb9379b2b551478df26
Cr-Commit-Position: refs/heads/master@{#346915}