New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 756706 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 3
Type: Bug



Sign in to add a comment

mac_views_browser=true compile regression in r492896

Project Member Reported by tapted@chromium.org, Aug 18 2017

Issue description

Chrome Version       : 62.0.3178.0

go/macviewsbuilder regressed compile in 

https://uberchromegw.corp.google.com/i/chromium.fyi/builders/Chromium%20Mac%2010.10%20MacViews/builds/26926

suspect


Changed by	catherinechung@google.com
Revision	5795ebbaa40775bda2ae1046add17ae69ec3ebba
Wed Aug 09 08:10:42 2017

Connecting frontend and backend code for the NewTabTracker

- Browsertest added to confirm the tracker can call the promo UI
- Added static helper in NewTabButton to show the promo for the
  last active browser
- The method DismissNewTracker was renamed to OnPromoClosed

Bug: 734132,  737830 
Change-Id: Ia6881474661b44c3994cc641b10333cd96fc1172
Reviewed-on: https://chromium-review.googlesource.com/596274
Cr-Commit-Position: refs/heads/master@{#492896}


errors

FAILED: obj/chrome/test/browser_tests/new_tab_tracker_browsertest.o /* snip */ -c ../../chrome/browser/feature_engagement/new_tab/new_tab_tracker_browsertest.cc -o obj/chrome/test/browser_tests/new_tab_tracker_browsertest.o
../../chrome/browser/feature_engagement/new_tab/new_tab_tracker_browsertest.cc:92:65: error: no member named 'kSessionTime' in namespace 'feature_engagement::events'
  EXPECT_CALL(*feature_engagement_tracker_, NotifyEvent(events::kSessionTime));
                                                        ~~~~~~~~^
../../third_party/googletest/src/googlemock/include/gmock/gmock-spec-builders.h:1848:61: note: expanded from macro 'EXPECT_CALL'
#define EXPECT_CALL(obj, call) GMOCK_EXPECT_CALL_IMPL_(obj, call)
                                                            ^~~~
../../third_party/googletest/src/googlemock/include/gmock/gmock-spec-builders.h:1847:20: note: expanded from macro 'GMOCK_EXPECT_CALL_IMPL_'
    ((obj).gmock_##call).InternalExpectedAt(__FILE__, __LINE__, #obj, #call)
                   ^~~~
../../chrome/browser/feature_engagement/new_tab/new_tab_tracker_browsertest.cc:99:35: error: no member named 'kOmniboxInteraction' in namespace 'feature_engagement::events'
              NotifyEvent(events::kOmniboxInteraction));
                          ~~~~~~~~^
../../third_party/googletest/src/googlemock/include/gmock/gmock-spec-builders.h:1848:61: note: expanded from macro 'EXPECT_CALL'
#define EXPECT_CALL(obj, call) GMOCK_EXPECT_CALL_IMPL_(obj, call)
                                                            ^~~~
../../third_party/googletest/src/googlemock/include/gmock/gmock-spec-builders.h:1847:20: note: expanded from macro 'GMOCK_EXPECT_CALL_IMPL_'
    ((obj).gmock_##call).InternalExpectedAt(__FILE__, __LINE__, #obj, #call)
                   ^~~~
../../chrome/browser/feature_engagement/new_tab/new_tab_tracker_browsertest.cc:110:45: error: use of undeclared identifier 'kIPHNewTabFeature'; did you mean 'kIPHDummyFeature'?
              ShouldTriggerHelpUI(IsFeature(kIPHNewTabFeature)))
                                            ^~~~~~~~~~~~~~~~~
                                            kIPHDummyFeature
../../third_party/googletest/src/googlemock/include/gmock/gmock-spec-builders.h:1848:61: note: expanded from macro 'EXPECT_CALL'
#define EXPECT_CALL(obj, call) GMOCK_EXPECT_CALL_IMPL_(obj, call)
                                                            ^
../../third_party/googletest/src/googlemock/include/gmock/gmock-spec-builders.h:1847:20: note: expanded from macro 'GMOCK_EXPECT_CALL_IMPL_'
    ((obj).gmock_##call).InternalExpectedAt(__FILE__, __LINE__, #obj, #call)
                   ^
../../components/feature_engagement/public/feature_constants.h:17:28: note: 'kIPHDummyFeature' declared here
extern const base::Feature kIPHDummyFeature;
                           ^
../../chrome/browser/feature_engagement/new_tab/new_tab_tracker_browsertest.cc:125:35: error: use of undeclared identifier 'kIPHNewTabFeature'; did you mean 'kIPHDummyFeature'?
              Dismissed(IsFeature(kIPHNewTabFeature)));
                                  ^~~~~~~~~~~~~~~~~
                                  kIPHDummyFeature
../../third_party/googletest/src/googlemock/include/gmock/gmock-spec-builders.h:1848:61: note: expanded from macro 'EXPECT_CALL'
#define EXPECT_CALL(obj, call) GMOCK_EXPECT_CALL_IMPL_(obj, call)
                                                            ^
../../third_party/googletest/src/googlemock/include/gmock/gmock-spec-builders.h:1847:20: note: expanded from macro 'GMOCK_EXPECT_CALL_IMPL_'
    ((obj).gmock_##call).InternalExpectedAt(__FILE__, __LINE__, #obj, #call)
                   ^
../../components/feature_engagement/public/feature_constants.h:17:28: note: 'kIPHDummyFeature' declared here
extern const base::Feature kIPHDummyFeature;
                           ^
4 errors generated.
 

Comment 1 by tapted@chromium.org, Aug 21 2017

Cc: dyaros...@yandex-team.ru
Status: Started (was: Assigned)
-> https://chromium-review.googlesource.com/622455
Project Member

Comment 2 by bugdroid1@chromium.org, Aug 21 2017

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

commit de621358ff53b03b4b9f239f6184a33e56fa4e17
Author: Trent Apted <tapted@chromium.org>
Date: Mon Aug 21 02:59:35 2017

Fix mac_views_browser build

- Only build new_tab_tracker_browsertest on Windows and Linux (r492896).
- Move some Cocoa-browser-only tests/util files to the appropriate place.

TBR=ellyjones@chromium.org

Bug:  756706 
Change-Id: I8619fb8e6b37f2cb61a14cd81004a7f4972ed275
Reviewed-on: https://chromium-review.googlesource.com/622455
Reviewed-by: Trent Apted <tapted@chromium.org>
Commit-Queue: Trent Apted <tapted@chromium.org>
Cr-Commit-Position: refs/heads/master@{#495864}
[modify] https://crrev.com/de621358ff53b03b4b9f239f6184a33e56fa4e17/chrome/browser/ui/BUILD.gn
[modify] https://crrev.com/de621358ff53b03b4b9f239f6184a33e56fa4e17/chrome/test/BUILD.gn

Sign in to add a comment