Commit b5775b47 authored by Raphael Kubo da Costa's avatar Raphael Kubo da Costa Committed by Commit Bot

signin: Fix bogus assignment in ternary operator

Instead of doing an assignment within one of the expressions, just use the
constant being assigned to. This also fixes a warning produced by GCC:

    ../../chrome/browser/signin/mutable_profile_oauth2_token_service_delegate.cc: In function ‘void {anonymous}::RecordTokenChanged(const string&, const string&)’:
    ../../chrome/browser/signin/mutable_profile_oauth2_token_service_delegate.cc:95:16: warning: operation on ‘transition’ may be undefined [-Wsequence-point]
         transition =
         ~~~~~~~~~~~^
             (new_token ==
             ~~~~~~~~~~~~~

              MutableProfileOAuth2TokenServiceDelegate::kInvalidRefreshToken)
              ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                 ? TokenStateTransition::kRegularToInvalid
                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                 : transition = TokenStateTransition::kRegularToRegular;
                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Change-Id: I8ea1dbd6ca499f3ba13a36dce41e152dc8263917
Reviewed-on: https://chromium-review.googlesource.com/992323Reviewed-by: default avatarDavid Roger <droger@chromium.org>
Commit-Queue: Raphael Kubo da Costa <raphael.kubo.da.costa@intel.com>
Cr-Commit-Position: refs/heads/master@{#547757}
parent ae910bcc
......@@ -96,7 +96,7 @@ void RecordTokenChanged(const std::string& existing_token,
(new_token ==
MutableProfileOAuth2TokenServiceDelegate::kInvalidRefreshToken)
? TokenStateTransition::kRegularToInvalid
: transition = TokenStateTransition::kRegularToRegular;
: TokenStateTransition::kRegularToRegular;
}
DCHECK_NE(TokenStateTransition::kCount, transition);
RecordTokenStateTransition(transition);
......
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