The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/9c02705b00898f8dd026c3ad9af909d0607526de

commit 9c02705b00898f8dd026c3ad9af909d0607526de
Author: Tommy Nyquist <nyquist@chromium.org>
Date: Thu Jun 01 00:10:22 2017

Ensure that the NeverAvailabilityModel initializes.

Currently, the NeverAvailabilityModel ignores the call to Initialize.

This leads to issues with the In-Product Help demo mode which
requires the model to be initialized, which depended on the callback
passed to NeverAvailabilityModel::Initialize(...) to be invoked.

This CL therefore adds support for invoking the callback and setting
the ready flag for the NeverAvailabilityModel and updates the
tests to take this into account.

BUG= 728292 

Change-Id: Ia636d9e9f0490fe78a12d5fcdd7d128e7b410e26
Reviewed-on: https://chromium-review.googlesource.com/519104
Reviewed-by: David Trainor <dtrainor@chromium.org>
Commit-Queue: Tommy Nyquist <nyquist@chromium.org>
Cr-Commit-Position: refs/heads/master@{#476104}
[modify] https://crrev.com/9c02705b00898f8dd026c3ad9af909d0607526de/components/feature_engagement_tracker/internal/never_availability_model.cc
[modify] https://crrev.com/9c02705b00898f8dd026c3ad9af909d0607526de/components/feature_engagement_tracker/internal/never_availability_model.h
[modify] https://crrev.com/9c02705b00898f8dd026c3ad9af909d0607526de/components/feature_engagement_tracker/internal/never_availability_model_unittest.cc

Your change meets the bar and is auto-approved for M60. Please go ahead and merge the CL to branch 3112 manually. Please contact milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), josafat@(ChromeOS), bustamante@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5941265e5d42830b69ce233785ef8b14e0210a5a

commit 5941265e5d42830b69ce233785ef8b14e0210a5a
Author: Tommy Nyquist <nyquist@chromium.org>
Date: Thu Jun 01 00:35:53 2017

Ensure that the NeverAvailabilityModel initializes.

Currently, the NeverAvailabilityModel ignores the call to Initialize.

This leads to issues with the In-Product Help demo mode which
requires the model to be initialized, which depended on the callback
passed to NeverAvailabilityModel::Initialize(...) to be invoked.

This CL therefore adds support for invoking the callback and setting
the ready flag for the NeverAvailabilityModel and updates the
tests to take this into account.

BUG= 728292 
TBR=nyquist@chromium.org

(cherry picked from commit 9c02705b00898f8dd026c3ad9af909d0607526de)

Change-Id: Ia636d9e9f0490fe78a12d5fcdd7d128e7b410e26
Reviewed-on: https://chromium-review.googlesource.com/519104
Reviewed-by: David Trainor <dtrainor@chromium.org>
Commit-Queue: Tommy Nyquist <nyquist@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#476104}
Reviewed-on: https://chromium-review.googlesource.com/520485
Reviewed-by: Tommy Nyquist <nyquist@chromium.org>
Cr-Commit-Position: refs/branch-heads/3112@{#75}
Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897}
[modify] https://crrev.com/5941265e5d42830b69ce233785ef8b14e0210a5a/components/feature_engagement_tracker/internal/never_availability_model.cc
[modify] https://crrev.com/5941265e5d42830b69ce233785ef8b14e0210a5a/components/feature_engagement_tracker/internal/never_availability_model.h
[modify] https://crrev.com/5941265e5d42830b69ce233785ef8b14e0210a5a/components/feature_engagement_tracker/internal/never_availability_model_unittest.cc