AU: Really disable URL/version setting over D-Bus on official images.

AU: Really disable URL/version setting over D-Bus on official images.

BUG=7159
TEST=unit tests, tested on production image too...

Change-Id: I82560023a1a231732f56ccfbd8c4837ae713f696

Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=ba3fb17

[petkov, 2010-10-20 21:33:11]

PTAL... There was a latent bug in the pre-existing code that it didn't really
used the new url/version values.

[adlr, 2010-10-20 21:34:22]

LGTM

On Wed, Oct 20, 2010 at 2:33 PM, <petkov@chromium.org> wrote:

> Reviewers: adlr,
>
> Message:
> PTAL... There was a latent bug in the pre-existing code that it didn't
> really
> used the new url/version values.
>
>
> Description:
> AU: Really disable URL/version setting over D-Bus on official images.
>
> BUG=7159
> TEST=unit tests, tested on production image too...
>
> Change-Id: I82560023a1a231732f56ccfbd8c4837ae713f696
>
> Please review this at http://codereview.chromium.org/3992001/show
>
> SVN Base: ssh://git@gitrw.chromium.org:9222/update_engine.git
>
> Affected files:
>  M dbus_service.cc
>
>
> Index: dbus_service.cc
> diff --git a/dbus_service.cc b/dbus_service.cc
> index
>
63fbf501da1f2c841d4166e63c3211ef782be261..08b953dd5f7a831e61d7a3c6157e0226475949f1
> 100644
> --- a/dbus_service.cc
> +++ b/dbus_service.cc
> @@ -69,7 +69,7 @@ gboolean
> update_engine_service_attempt_update(UpdateEngineService* self,
>   }
>   LOG(INFO) << "Attempt update: app_version=\"" << update_app_version <<
> "\" "
>             << "omaha_url=\"" << update_omaha_url << "\"";
> -  self->update_attempter_->CheckForUpdate(app_version, omaha_url);
> +  self->update_attempter_->CheckForUpdate(update_app_version,
> update_omaha_url);
>   return TRUE;
>  }
>
>
>
>