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

Issue 708924 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 0
Type: ----



Sign in to add a comment

compile failing on Android builders - conflict in CSSPropertyAPICounterIncrement.cpp

Project Member Reported by awdf@chromium.org, Apr 6 2017

Issue description

compile failing on 4 builders

Builders failed on: 
- Android Builder: 
  https://build.chromium.org/p/chromium.linux/builders/Android%20Builder
- Android Builder (dbg): 
  https://build.chromium.org/p/chromium.linux/builders/Android%20Builder%20%28dbg%29
- Android Clang Builder (dbg): 
  https://build.chromium.org/p/chromium.linux/builders/Android%20Clang%20Builder%20%28dbg%29
- Cast Android (dbg): 
  https://build.chromium.org/p/chromium.linux/builders/Cast%20Android%20%28dbg%29

../../third_party/WebKit/Source/core/css/properties/CSSPropertyAPICounterIncrement.cpp:11:17: error: prototype for 'const blink::CSSValue* blink::CSSPropertyAPICounterIncrement::parseSingleValue(blink::CSSParserTokenRange&, const blink::CSSParserContext*)' does not match any in class 'blink::CSSPropertyAPICounterIncrement'
 const CSSValue* CSSPropertyAPICounterIncrement::parseSingleValue(
                 ^
In file included from ../../third_party/WebKit/Source/core/css/properties/CSSPropertyAPICounterIncrement.cpp:5:0:
gen/blink/core/css/properties/CSSPropertyAPICounterIncrement.h:18:26: error: candidate is: static const blink::CSSValue* blink::CSSPropertyAPICounterIncrement::parseSingleValue(blink::CSSParserTokenRange&, const blink::CSSParserContext&)
   static const CSSValue* parseSingleValue(CSSParserTokenRange&, const CSSParserContext&);

Assigning to myself until build is green again
 

Comment 2 by awdf@chromium.org, Apr 6 2017

Cc: machenb...@chromium.org
ALso i note the finditforme bot has reverted this CL: https://codereview.chromium.org/2786153004 

Both reverts are in the new build building at https://uberchromegw.corp.google.com/i/chromium/builders/Android/builds/70750 
.. let's hope it still compiles!


Comment 3 by awdf@chromium.org, Apr 6 2017

Hm the autorevert seems to have fixed it so my revert was probably unnecessary - eg. 

https://build.chromium.org/p/chromium.linux/builders/Cast%20Android%20%28dbg%29/builds/69193 

Comment 4 by awdf@chromium.org, Apr 6 2017

Cc: bugsnash@chromium.org
It's back to green so I guess the original CLs just need to be relanded in the right order.

Comment 5 by awdf@chromium.org, Apr 6 2017

Status: Fixed (was: Assigned)

Sign in to add a comment