• Daniel Bratell's avatar
    There is both ::prefs and ::metrics::prefs so be explicit · 1fe0adbf
    Daniel Bratell authored
    When code inside metrics say "prefs" the default assumption is
    that it means "::metrics::prefs", *if* the compiler knows that
    ::metrics::prefs exists. This code wants to use ::prefs which
    failed in jumbo build experiments where the compiler knew
    that ::metrics::prefs exists.
    
    This patch makes the namespace lookup explicit by writing
    ::prefs instead of just prefs.
    
    Higher level: The coding standard discourages sub namespaces with
    the same name as a top level namespace so it is possible
    ::metrics::prefs should change name at some time.
    
    Bug: 746957
    Change-Id: Ia68378e6a3e7937f5d1b2ebcc960b9111e426fe6
    Reviewed-on: https://chromium-review.googlesource.com/890455Reviewed-by: default avatarGayane Petrosyan <gayane@chromium.org>
    Commit-Queue: Daniel Bratell <bratell@opera.com>
    Cr-Commit-Position: refs/heads/master@{#532934}
    1fe0adbf
tab_stats_tracker.cc 7.42 KB