mac_views_browser=true compile regression in r492896 |
||
Issue descriptionChrome 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.
,
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
,
Aug 21 2017
compile is green in https://uberchromegw.corp.google.com/i/chromium.fyi/builders/Chromium%20Mac%2010.10%20MacViews/builds/27635 |
||
►
Sign in to add a comment |
||
Comment 1 by tapted@chromium.org
, Aug 21 2017Status: Started (was: Assigned)