New issue
Advanced search Search tips

Issue 791561 link

Starred by 2 users

Issue metadata

Status: Archived
Owner: ----
Closed: Dec 5
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug



Sign in to add a comment

SQLTransactionBackend.h : build failed using GCC 6.4.1 (include SQLError.h missing)

Reported by fval...@connected-labs.com, Dec 4 2017

Issue description

UserAgent: Mozilla/5.0 (X11; Linux x86_64; rv:50.0) Gecko/20100101 Firefox/50.0

Steps to reproduce the problem:
using chromium HEAD (webkit 419d8078283ee), building using GCC 6.4.1 

gn args 
is_clang=false
use_jumbo_build=true

What is the expected behavior?

What went wrong?

from gen/out/Default/gen/blink/bindings/modules/v8/bindings_modules_impl_jumbo_13.cc:5: ../../buildtools/third_party/libc++/trunk/include/memory: In instantiation of ‘void std::__1::default_delete<_Tp>::operator()(_Tp*) const [with _Tp = blink::SQLErrorData]’: ../../buildtools/third_party/libc++/trunk/include/memory:2546:22:   required from ‘void std::__1::unique_ptr<_Tp, _Dp>::reset(std::__1::unique_ptr<_Tp, _Dp>::pointer) [with _Tp = blink::SQLErrorData; _Dp = std::__1::default_delete<blink::SQLErrorData>; std::__1::unique_ptr<_Tp, _Dp>::pointer = blink::SQLErrorData*]’ ../../buildtools/third_party/libc++/trunk/include/memory:2500:24:   required from ‘std::__1::unique_ptr<_Tp, _Dp>::~unique_ptr() [with _Tp = blink::SQLErrorData; _Dp = std::__1::default_delete<blink::SQLErrorData>]’ ../../third_party/WebKit/Source/modules/webdatabase/SQLStatementBackend.h:45:7:   required from ‘void blink::GarbageCollectedFinalized<T>::FinalizeGarbageCollectedObject() [with T = blink::SQLStatementBackend]’ ../../third_party/WebKit/Source/platform/heap/GCInfo.h:35:5:   required from ‘static void blink::FinalizerTraitImpl<T, true>::Finalize(void*) [with T = blink::SQLStatementBackend]’ ../../third_party/WebKit/Source/platform/heap/GCInfo.h:61:58:   required from ‘static void blink::FinalizerTrait<T>::Finalize(void*) [with T = blink::SQLStatementBackend]’ ../../third_party/WebKit/Source/platform/heap/GCInfo.h:198:5:   [ skipping 8 instantiation contexts, use -ftemplate-backtrace-limit=0 to disable ] ../../third_party/WebKit/Source/platform/heap/Persistent.h:442:36:   required from ‘blink::CrossThreadPersistent<T>::CrossThreadPersistent() [with T = blink::SQLStatementBackend]’ ../../buildtools/third_party/libc++/trunk/include/type_traits:3295:29:   required from ‘struct std::__1::is_trivially_constructible<blink::CrossThreadPersistent<blink::SQLStatementBackend> >’ ../../buildtools/third_party/libc++/trunk/include/type_traits:3422:50:   required from ‘struct std::__1::is_trivially_default_constructible<blink::CrossThreadPersistent<blink::SQLStatementBackend> >’ ../../third_party/WebKit/Source/platform/wtf/VectorTraits.h:36:25:   required from ‘constexpr const bool WTF::VectorTraitsBase<blink::CrossThreadPersistent<blink::SQLStatementBackend> >::kCanInitializeWithMemset’ ../../third_party/WebKit/Source/platform/wtf/Deque.h:152:21:   required from ‘class WTF::Deque<blink::CrossThreadPersistent<blink::SQLStatementBackend> >’ ../../third_party/WebKit/Source/modules/webdatabase/SQLTransactionBackend.h:138:53:   required from here ../../buildtools/third_party/libc++/trunk/include/memory:2229:25: error: invalid application of ‘sizeof’ to incomplete type ‘blink::SQLErrorData’      static_assert(sizeof(_Tp) > 0,                          ^

Did this work before? N/A 

Chrome version: <Copy from: 'about:version'>  Channel: n/a
OS Version: 
Flash Version: Shockwave Flash 24.0 r0

build issue fixed with 

#include "Source/modules/webdatabase/SQLError.h" 
added in chromium/src/third_party/WebKit/Source/modules/webdatabase/SQLStatementBackend.h
 
Components: Build
Labels: Triaged-ET TE-NeedsTriageHelp Needs-Milestone
The issue seems to be out of TE-scope as it is related to build. Hence, adding label TE-NeedsTriageHelp for further investigation from dev team.

Thanks...!!
Project Member

Comment 2 by sheriffbot@chromium.org, Dec 5

Status: Archived (was: Unconfirmed)
Issue has not been modified or commented on in the last 365 days, please re-open or file a new bug if this is still an issue.

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

Sign in to add a comment