- 15 Apr, 2015 2 commits
-
-
jochen authored
The embedder can control how many threads it wants to use via the v8::Platform implementation. V8 internally doesn't spin up threads anymore. If the embedder doesn't want to use any threads at all, it's v8::Platform implementation must either run the background jobs on the foreground thread, or the embedder should specify --predictable BUG=none R=yangguo@chromium.org LOG=y Review URL: https://codereview.chromium.org/1064723005 Cr-Commit-Position: refs/heads/master@{#27833}
-
jochen authored
An embedder that wants to avoid the check should use MaybeLocal::ToLocal. BUG=none R=dcarney@chromium.org LOG=y Review URL: https://codereview.chromium.org/1083943002 Cr-Commit-Position: refs/heads/master@{#27832}
-
- 14 Apr, 2015 1 commit
-
-
smcgruer authored
As uClibc defines __GLIBC__ in an attempt to look like glibc, V8_LIBC_GLIBC was true for uClibc as well. Checking for uClibc before glibc fixes this and restores the correct behavior. BUG= Review URL: https://codereview.chromium.org/1066573005 Cr-Commit-Position: refs/heads/master@{#27806}
-
- 13 Apr, 2015 1 commit
-
-
jochen authored
This allows the embedder to decide whether it's worthwhile to copy the contents to avoid materializing a buffer. BUG=v8:3996 R=dslomov@chromium.org,kbr@chromium.org LOG=y Review URL: https://codereview.chromium.org/1084513002 Cr-Commit-Position: refs/heads/master@{#27782}
-
- 10 Apr, 2015 4 commits
-
-
fedor authored
When debugging Handle leaks in io.js we found it very convenient to be able to Seal some specific (root in our case) scope to prevent Handle allocations in it, and easily find leakage. R=yangguo BUG= Review URL: https://codereview.chromium.org/1079713002 Cr-Commit-Position: refs/heads/master@{#27766}
-
verwaest authored
This fixes the arm(64) and mips(64) write barriers BUG= Review URL: https://codereview.chromium.org/1073133002 Cr-Commit-Position: refs/heads/master@{#27751}
-
dcarney authored
BUG= Review URL: https://codereview.chromium.org/1074153002 Cr-Commit-Position: refs/heads/master@{#27750}
-
jochen authored
BUG=none R=hpayer@chromium.org LOG=n Review URL: https://codereview.chromium.org/1075133002 Cr-Commit-Position: refs/heads/master@{#27736}
-
- 09 Apr, 2015 3 commits
-
-
thakis authored
gcc rejects the following snippet, clang rejects it in -std=c++11 mode: namespace A { template<class T> class C {}; } namespace B { template class A::C<int>; } Indeed, the C++ standard says in 14.7.2p2 "An explicit instantiation shall appear in an enclosing namespace of its template", so cl.exe is incorrect to allow this. Just move the instantiation out of the v8 namespace to fix. No intended behavior change. Fixes building with clang-cl on Windows. BUG=chromium:475643 LOG=N TBR=svenpanne@chromium.org Review URL: https://codereview.chromium.org/1073903002 Cr-Commit-Position: refs/heads/master@{#27721}
-
adamk authored
Revert of Merge cellspace into old pointer space (patchset #8 id:180001 of https://codereview.chromium.org/1010803012/) Reason for revert: Causes test failures on ARM bots related to cells and write barriers. Original issue's description: > Merge cellspace into old pointer space > > BUG= > > Committed: https://crrev.com/4e7163ce05f135918205c7855ae60a48e5d46cc5 > Cr-Commit-Position: refs/heads/master@{#27707} TBR=hpayer@chromium.org,balazs.kilvady@imgtec.com,yangguo@chromium.org,verwaest@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1053243003 Cr-Commit-Position: refs/heads/master@{#27711}
-
verwaest authored
BUG= Review URL: https://codereview.chromium.org/1010803012 Cr-Commit-Position: refs/heads/master@{#27707}
-
- 08 Apr, 2015 1 commit
-
-
loislo authored
BUG=chromium:452067 LOG=n Committed: https://crrev.com/baf927ff5115ec62a6dad684b9232ed9d3960e3a Cr-Commit-Position: refs/heads/master@{#27626} Review URL: https://codereview.chromium.org/1045753002 Cr-Commit-Position: refs/heads/master@{#27674}
-
- 07 Apr, 2015 5 commits
-
-
machenbach authored
Revert of CpuProfiler: public API for deopt info in cpu profiler. (patchset #6 id:150001 of https://codereview.chromium.org/1045753002/) Reason for revert: [Sheriff] Breaks compile here: http://build.chromium.org/p/client.v8/builders/V8%20Win32%20-%20nosnap%20-%20shared/builds/6115 Original issue's description: > CpuProfiler: public API for deopt info in cpu profiler. > > BUG=chromium:452067 > LOG=n > > Committed: https://crrev.com/baf927ff5115ec62a6dad684b9232ed9d3960e3a > Cr-Commit-Position: refs/heads/master@{#27626} TBR=alph@chromium.org,jkummerow@chromium.org,svenpanne@chromium.org,yurys@chromium.org,loislo@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:452067 Review URL: https://codereview.chromium.org/1062053004 Cr-Commit-Position: refs/heads/master@{#27628}
-
loislo authored
BUG=chromium:452067 LOG=n Review URL: https://codereview.chromium.org/1045753002 Cr-Commit-Position: refs/heads/master@{#27626}
-
yangguo authored
Instead, please use v8::Isolate::RequestInterrupt to synchronize to the main thread. R=yurys@chromium.org API=Removed v8::Debug::DebugBreakForCommand LOG=Y Review URL: https://codereview.chromium.org/1036863002 Cr-Commit-Position: refs/heads/master@{#27625}
-
hpayer authored
This reverts commit cbfcee55. BUG= Review URL: https://codereview.chromium.org/1051233002 Cr-Commit-Position: refs/heads/master@{#27623}
-
jkummerow authored
The motivation is that we prefer to avoid creating internal properties, and we have a usable field on maps ("transitions", which is not used for prototype maps). This CL also ensures the invariant that prototype maps are never shared, even if they are in dictionary mode. Review URL: https://codereview.chromium.org/1033653002 Cr-Commit-Position: refs/heads/master@{#27617}
-
- 06 Apr, 2015 1 commit
-
-
hablich authored
BUG= TBR=machenbach@chromium.org,vogelheim@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/1059433002 Cr-Commit-Position: refs/heads/master@{#27604}
-
- 02 Apr, 2015 4 commits
-
-
dcarney authored
Revert of make ToLocalCheck crash in release mode (patchset #1 id:1 of https://codereview.chromium.org/1043363005/) Reason for revert: breaks some devtools things Original issue's description: > make ToLocalCheck crash in release mode > > R=svenpanne@chromium.org > BUG= > > Committed: https://crrev.com/ce7cc5119c1e031bf1eb8476fbf5e55bef738f94 > Cr-Commit-Position: refs/heads/master@{#27585} TBR=svenpanne@chromium.org,machenbach@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1057873003 Cr-Commit-Position: refs/heads/master@{#27589}
-
dcarney authored
R=svenpanne@chromium.org BUG= Review URL: https://codereview.chromium.org/1043363005 Cr-Commit-Position: refs/heads/master@{#27585}
-
jochen authored
BUG=v8:3996 LOG=y R=dslomov@chromium.org,kbr@chromium.org,hpayer@chromium.org Review URL: https://codereview.chromium.org/1041403003 Cr-Commit-Position: refs/heads/master@{#27580}
-
jochen authored
BUG=none R=danno@chromium.org LOG=n Review URL: https://codereview.chromium.org/960903003 Cr-Commit-Position: refs/heads/master@{#27579}
-
- 31 Mar, 2015 2 commits
-
-
adamk authored
It triggers once per context that calls observe (or attempts to access any observation metadata, e.g. through Object.getNotifier). Review URL: https://codereview.chromium.org/1048213002 Cr-Commit-Position: refs/heads/master@{#27557}
-
jochen authored
Embedders should use IdleNotificationDeadline() BUG=none R=hpayer@chromium.org LOG=y Review URL: https://codereview.chromium.org/1019793008 Cr-Commit-Position: refs/heads/master@{#27550}
-
- 30 Mar, 2015 2 commits
-
-
erikcorry authored
R=hpayer@chromium.org BUG= Review URL: https://codereview.chromium.org/1040763002 Cr-Commit-Position: refs/heads/master@{#27520}
-
dcarney authored
also some drive-by handlescope fixes in api.cc R=svenpanne@chromium.org BUG= Review URL: https://codereview.chromium.org/1040043002 Cr-Commit-Position: refs/heads/master@{#27510}
-
- 26 Mar, 2015 3 commits
-
-
dcarney authored
R=verwaest@chromium.org BUG=468451 LOG=N Review URL: https://codereview.chromium.org/1036743004 Cr-Commit-Position: refs/heads/master@{#27482}
-
dcarney authored
R=jochen@chromium.org, erikcorry@chromium.org BUG= Review URL: https://codereview.chromium.org/998253006 Cr-Commit-Position: refs/heads/master@{#27475}
-
yurys authored
Before this patch the embedder could assign timestamp to the last interval after calling GetHeapStats. This would be slightly different from the timstamps assigned by v8 internally and written into heap snapshot. This patch allow to avoid this small discrepancy by returning timestamp along with last heap stats update. BUG=chromium:467222 LOG=Y Review URL: https://codereview.chromium.org/1037803002 Cr-Commit-Position: refs/heads/master@{#27466}
-
- 25 Mar, 2015 2 commits
-
-
yurys authored
The method was deprecated a while ago: https://crrev.com/87e4bba31eabfd3b12e42b5886dc9da08d2daf13 LOG=Y BUG=YES API=Remove v8::Isolate::ClearInterrupt Review URL: https://codereview.chromium.org/1032623007 Cr-Commit-Position: refs/heads/master@{#27449}
-
alexandre.rames authored
BUG= Review URL: https://codereview.chromium.org/1003383004 Cr-Commit-Position: refs/heads/master@{#27442}
-
- 24 Mar, 2015 4 commits
-
-
machenbach authored
Revert of Track how many pages trigger fallback strategies in GC (patchset #2 id:20001 of https://codereview.chromium.org/1029323003/) Reason for revert: This seems to cause lots of crashes in layout tests debug: ../../third_party/WebKit/Source/bindings/core/v8/V8PerIsolateData.cpp(67) : void blink::useCounterCallback(v8::Isolate *, v8::Isolate::UseCounte http://build.chromium.org/p/client.v8/builders/V8-Blink%20Linux%2064%20%28dbg%29/builds/2332 Original issue's description: > Track how many pages trigger fallback strategies in GC > > R=hpayer@chromium.org > BUG= > > Committed: https://crrev.com/bb880058f6499510cff12d98dc7d524d35d769cb > Cr-Commit-Position: refs/heads/master@{#27421} TBR=hpayer@chromium.org,erikcorry@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1000523003 Cr-Commit-Position: refs/heads/master@{#27427}
-
erikcorry authored
R=hpayer@chromium.org BUG= Review URL: https://codereview.chromium.org/1029323003 Cr-Commit-Position: refs/heads/master@{#27421}
-
dcarney authored
additionally, add a drive by fix to WeakCallbackInfo R=jochen@chromium.org, erikcorry@chromium.org BUG= Review URL: https://codereview.chromium.org/1026283004 Cr-Commit-Position: refs/heads/master@{#27407}
-
jochen authored
All users should use IdleNotificationDeadline() instead BUG=none R=hpayer@chromium.org LOG=y Review URL: https://codereview.chromium.org/1028163003 Cr-Commit-Position: refs/heads/master@{#27386}
-
- 23 Mar, 2015 1 commit
-
-
verwaest authored
BUG= Review URL: https://codereview.chromium.org/1009123003 Cr-Commit-Position: refs/heads/master@{#27371}
-
- 19 Mar, 2015 4 commits
-
-
hpayer authored
TBR=verwaest@chromium.org,ulan@chromium.org,ishell@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/1027463002 Cr-Commit-Position: refs/heads/master@{#27323}
-
dslomov authored
Only adds a counter id, does not count anything. R=jochen@chromium.org BUG=v8:3942 LOG=N Review URL: https://codereview.chromium.org/1020883002 Cr-Commit-Position: refs/heads/master@{#27313}
-
jochen authored
If the Maybe is nothing or the MaybeLocal is empty, it means that the API call either threw an exception or an exception was already pending. In that case, the embedder needs to handle the exception or otherwise react to the failed API call. BUG=v8:3929 R=svenpanne@chromium.org LOG=y Review URL: https://codereview.chromium.org/1022803002 Cr-Commit-Position: refs/heads/master@{#27298}
-
yurys authored
It's value is hardwired in gin[1] and is referenced in Blink[2]. Since it is treated specially by v8 debugger it should be defined in v8 API and referenced in gin and blink.. [1] https://src.chromium.org/viewvc/chrome/trunk/src/gin/public/context_holder.h?r1=239099&r2=239098&pathrev=239099 [2] https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit/Source/bindings/core/v8/V8PerContextData.cpp&q=kDebugIdIndex%20f:V8PerContextData&sq=package:chromium&type=cs BUG=chromium:466631 LOG=Y Review URL: https://codereview.chromium.org/1020453002 Cr-Commit-Position: refs/heads/master@{#27290}
-