cashew: Service: set "in progress" flag before requesting usage update

This CL fixes an inconsistency in the request_in_progress_ flag that
triggers an assertion failure.

Root cause: OnRequestComplete asserts that the request_in_progress_ flag
is set. RequestUsageUpdate sets the flag *after* initiating an update via
DataPlanProvider::RequestUsageUpdate. However, it's possible for the
OnRequestComplete callback to be invoked from within this initial call,
prior to the flag being set.

Resolution: Set the flag first, then initiate the update.

BUG=chromium-os:7687
TEST=None

Change-Id: I302093d0ae51107aed350e590327e7a9f44a2ae0

Review URL: http://codereview.chromium.org/3764005
1 file changed