Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(54)

Issue 28633002: Fix compilation error of public/platform/WebFloatRect.h for MSVC 2013 (Closed)

Created:
7 years, 2 months ago by yosin_UTC9
Modified:
6 years ago
CC:
blink-reviews, jamesr, dglazkov+blink, abarth-chromium
Visibility:
Public.

Description

Fix compilation error of public/platform/WebFloatRect.h for MSVC 2013 This patch fixes compilation error of public/platform/WebFloatRect.h for MSVC 2013. Error message on MSVC 2013: d:\src\w\cr\src\third_party\webkit\public\platform\webfloatrect.h(111) : error C2039: 'max' : is not a member of 'std' BUG=n/a TEST=n/a; no behavior changes. Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=159999

Patch Set 1 #

Patch Set 2 : 2013-10-18T19:02:58 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M public/platform/WebFloatRect.h View 1 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 8 (0 generated)
yosin_UTC9
Could you review this patch? Thanks in advance.
7 years, 2 months ago (2013-10-18 08:55:46 UTC) #1
tkent
> d:\src\w\cr\src\third_party\webkit\public\platform\webfloatrect.h(111) : error C2039: 'max' : is not a member of 'std' What .cpp/.cc ...
7 years, 2 months ago (2013-10-18 09:22:21 UTC) #2
yosin_UTC9
PTAL Compilation error is occurred in src/third_party/WebKit/Source/testing/runner/TestPlugin.cpp. So, I move #include <algorithm> into non-blink implementation ...
7 years, 2 months ago (2013-10-18 10:09:46 UTC) #3
abarth-chromium
lgtm
7 years, 2 months ago (2013-10-18 16:09:25 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/yosin@chromium.org/28633002/50001
7 years, 2 months ago (2013-10-18 16:09:35 UTC) #5
commit-bot: I haz the power
Change committed as 159999
7 years, 2 months ago (2013-10-19 10:37:35 UTC) #6
tamutharlay
On 2013/10/19 10:37:35, I haz the power (commit-bot) wrote: > Change committed as 159999 159999
6 years, 1 month ago (2014-11-20 00:44:16 UTC) #7
tamutharlay
6 years ago (2014-11-24 05:41:43 UTC) #8
Message was sent while issue was closed.
On Thursday, November 20, 2014, <tamutharlay@gmail.com> wrote:
> On 2013/10/19 10:37:35, I haz the power (commit-bot) wrote:
>>
>> Change committed as 159999
>
> 159999
>
> https://codereview.chromium.org/28633002/
>

-- 
Kyaw

To unsubscribe from this group and stop receiving emails from it, send an email
to blink-reviews+unsubscribe@chromium.org.

Powered by Google App Engine
This is Rietveld 408576698