- 11 Aug, 2016 2 commits
-
-
bgeron authored
BUG= Review-Url: https://codereview.chromium.org/2230313002 Cr-Commit-Position: refs/heads/master@{#38588}
-
dgozman authored
This target introduces a dependency on WebKit/platform which is not the way we want it. Keeping the build files in src/inspector, as they are not used and will be useful later on. BUG=none NOPRESUBMIT=true Review-Url: https://codereview.chromium.org/2239603002 Cr-Commit-Position: refs/heads/master@{#38587}
-
- 10 Aug, 2016 5 commits
-
-
bgeron authored
BUG= R=danno Review-Url: https://codereview.chromium.org/2232933002 Cr-Commit-Position: refs/heads/master@{#38562}
-
bgeron authored
BUG= R=danno Review-Url: https://codereview.chromium.org/2230783005 Cr-Commit-Position: refs/heads/master@{#38561}
-
epertoso authored
NOTRY=true NOPRESUBMIT=true NOTREECHECKS=true Review-Url: https://codereview.chromium.org/2231083002 Cr-Commit-Position: refs/heads/master@{#38546}
-
danno authored
Also convert CodeView to a class and fix a host of selection bugs in the process, as well as move the logic and data to "enrich" location with one type of location data with location data known globally to the whole graph in the selection broker. Review-Url: https://codereview.chromium.org/2230083004 Cr-Commit-Position: refs/heads/master@{#38544}
-
danno authored
Review-Url: https://codereview.chromium.org/2234523002 Cr-Commit-Position: refs/heads/master@{#38520}
-
- 09 Aug, 2016 3 commits
-
-
bgeron authored
R=danno,jarin BUG= Review-Url: https://codereview.chromium.org/2226293002 Cr-Commit-Position: refs/heads/master@{#38502}
-
bgeron authored
BUG= Review-Url: https://codereview.chromium.org/2225683009 Cr-Commit-Position: refs/heads/master@{#38500}
-
mstarzinger authored
R=machenbach@chromium.org Review-Url: https://codereview.chromium.org/2228853002 Cr-Commit-Position: refs/heads/master@{#38494}
-
- 08 Aug, 2016 3 commits
-
-
bgeron authored
R=danno BUG= Review-Url: https://codereview.chromium.org/2227643002 Cr-Commit-Position: refs/heads/master@{#38442}
-
machenbach authored
This removes the test-download method for all but promises-aplus. They all only contain legacy code for deleting old archives. Only test262 needs to prepare sources on swarming, which is moved to a new method, called unconditionally. All references to --download-data and --download-data-only in the infrastructure can be removed after this. BUG= Review-Url: https://codereview.chromium.org/2227613002 Cr-Commit-Position: refs/heads/master@{#38439}
-
machenbach authored
Remove --ignition-turbofan after: https://codereview.chromium.org/2222913002/ BUG=v8:5238 TBR=rmcilroy@chromium.org, mstarzinger@chromium.org NOTRY=true Review-Url: https://codereview.chromium.org/2220383002 Cr-Commit-Position: refs/heads/master@{#38425}
-
- 05 Aug, 2016 3 commits
-
-
machenbach authored
BUG=v8:5238 NOTREECHECKS=true Review-Url: https://codereview.chromium.org/2214283003 Cr-Commit-Position: refs/heads/master@{#38381}
-
machenbach authored
This makes test outcomes immutable. This has two benefits: 1) It's more robust. Using the |= operator by mistake wouldn't lead to unwanted mutation. 2) It's faster as now the set reference can be copied again when copying test cases. Test case copying happens an order of magnitude more often than rules do apply. BUG=v8:5238 Review-Url: https://codereview.chromium.org/2212333002 Cr-Commit-Position: refs/heads/master@{#38373}
-
machenbach authored
BUG=v8:5238 Review-Url: https://codereview.chromium.org/2216293002 Cr-Commit-Position: refs/heads/master@{#38362}
-
- 04 Aug, 2016 2 commits
-
-
machenbach authored
This adds the possibility to address test cases in the status file with the variant under which the test is running. This is only allowed in top-level sections. Example: [{ 'test-case': [PASS, SLOW], }] ['variant == foo', { 'test-case': [FAIL], }] The test case "test-case" is marked as slow in all variants. Additionally, in variant foo, it'll be expected to fail. This CL also exemplifies the new feature with test cases running under the ignition_turbofan variant. The corresponding legacy flag is deprecated. BUG=v8:5238 Review-Url: https://codereview.chromium.org/2203013002 Cr-Commit-Position: refs/heads/master@{#38342}
-
hablich authored
Ignition benchmarks will be switched to --ignition-staging. TBR=machenbach@chromium.org NOTRY=true NOTREECHECKS=true NOPRESUBMIT=true Review-Url: https://codereview.chromium.org/2210873002 Cr-Commit-Position: refs/heads/master@{#38322}
-
- 03 Aug, 2016 1 commit
-
-
jochen authored
This will allow for the background parser to parse inner functions BUG=v8:5215 R=marja@chromium.org,verwaest@chromium.org Review-Url: https://codereview.chromium.org/2198043002 Cr-Commit-Position: refs/heads/master@{#38291}
-
- 02 Aug, 2016 5 commits
-
-
machenbach authored
This prepares for adding the ignition_staging variant to the default variant set. BUG=v8:5238 Review-Url: https://codereview.chromium.org/2205883002 Cr-Commit-Position: refs/heads/master@{#38257}
-
machenbach authored
This is not referenced anymore by the infrastructure after: https://codereview.chromium.org/2202113003/ BUG=v8:5238 Review-Url: https://codereview.chromium.org/2203023003 Cr-Commit-Position: refs/heads/master@{#38254}
-
machenbach authored
It should be enough to specify --variants=... with the desired set of variants. The stress/nostress shortcuts might get confusing soon, as we intend to move the stress option out of the developer default set. BUG=v8:5238 Review-Url: https://codereview.chromium.org/2195373002 Cr-Commit-Position: refs/heads/master@{#38248}
-
machenbach authored
Without this fix, two rules [PASS, MODIFIER] and [FAIL] would be joined as [PASS, MODIFIER, FAIL], undermining the intention of the second rule to mark a test as failing. BUG=v8:4680,v8:4900 Review-Url: https://codereview.chromium.org/2201033002 Cr-Commit-Position: refs/heads/master@{#38238}
-
Michael Achenbach authored
Cr-Commit-Position: refs/heads/master@{#38225}
-
- 01 Aug, 2016 4 commits
-
-
machenbach authored
This introduces aliases for testing variants that can be combined with other variant names. E.g. --variants=dev,foo would run the three developer default variants and variant foo. We'll have three stages: "dev" for variants to be run by default on developer workstations, "more" for additional variants, executed on all bots, "extra" for additional variants executed on a subset of bots (e.g. not on very slow or otherwise resource-limited bots). BUG=v8:5238 NOTRY=true Review-Url: https://codereview.chromium.org/2196223002 Cr-Commit-Position: refs/heads/master@{#38214}
-
machenbach authored
The flag was introduced for ignition development. It can only be used when running ignition tests in isolation on the bots. The bots only use ignition_turbo in isolation since a while and don't pass the --ignition flag anymore. BUG=v8:5238 Review-Url: https://codereview.chromium.org/2197123002 Cr-Commit-Position: refs/heads/master@{#38206}
-
machenbach authored
First step of deprecating the dedicated ignition alias. Next it will be changed on the bots to use the bot_default suite. Then we'll delete it. BUG=v8:5238 Review-Url: https://codereview.chromium.org/2194153002 Cr-Commit-Position: refs/heads/master@{#38202}
-
yangguo authored
BUG=chromium:580337 R=machenbach@chromium.org Review-Url: https://codereview.chromium.org/2189023002 Cr-Commit-Position: refs/heads/master@{#38198}
-
- 29 Jul, 2016 2 commits
-
-
bgeron authored
NOTRY=true Review-Url: https://codereview.chromium.org/2197593002 Cr-Commit-Position: refs/heads/master@{#38178}
-
machenbach authored
Required for https://codereview.chromium.org/2194823002/ BUG=chromium:631979 NOTRY=true Review-Url: https://codereview.chromium.org/2189323002 Cr-Commit-Position: refs/heads/master@{#38161}
-
- 28 Jul, 2016 5 commits
-
-
ishell authored
[gdb] Define print functions used by gdb macros in the top level namespace to make them always be available. This should fix the following errors that sometimes appear when using V8 gdb macros: Cannot evaluate function -- may be inlined or No symbol "xxx" in namespace "v8::internal". Review-Url: https://codereview.chromium.org/2186293002 Cr-Commit-Position: refs/heads/master@{#38145}
-
Alexander.Gilday2 authored
Change was meant to be in previous patch but was accidentally reverted. README.md updated with new filename, also. BUG= Review-Url: https://codereview.chromium.org/2184223005 Cr-Commit-Position: refs/heads/master@{#38134}
-
machenbach authored
Splitt of CL: https://codereview.chromium.org/2188693002/#ps100001 Switching to GN is blocked on test errors. These files are required for compilation and allow looking into the test errors locally. BUG=chromium:474921 NOTRY=true Review-Url: https://codereview.chromium.org/2190863003 Cr-Commit-Position: refs/heads/master@{#38130}
-
Alexander.Gilday2 authored
perf-turbo.py merges a perf data file and a turbofan trace file into a single json object which can then be piped to a file and uploaded to turbolizer to display the profiling data in the disassembly. With the changes, turbolizer now shows the event counts for instruction in percentage form and with heatmap-stype colouring. Multiple different events can be recorded at once with a new drop-down menu to select which event to view the counts of. The documentation has been updated with instructions. Using the script is optional and turbolizer retains previous functionality if a trace without profiling data is uploaded. BUG=None Review-Url: https://codereview.chromium.org/2174803002 Cr-Commit-Position: refs/heads/master@{#38124}
-
jochen authored
Use it like this: Program received signal SIGTRAP, Trace/breakpoint trap. [Switching to Thread 0x7f50d02c4700 (LWP 8152)] base::debug::(anonymous namespace)::DebugBreak () at ../../base/debug/debugger_posix.cc:230 230 } (gdb) bt at ../../base/debug/debugger_posix.cc:230 ... args_object=0x7f50d02bfe00, isolate=0x2f001c9e2020) at ../../v8/src/builtins/builtins-api.cc:123 ... (gdb) jss (gdb) bt isolate=0x2f001c9e2020, is_construct=false, target=..., receiver=..., argc=1, args=0x7f50d02c0518, new_target=...) at ../../v8/src/execution.cc:111 isolate=0x2f001c9e2020, callable=..., receiver=..., argc=1, argv=0x7f50d02c0518) at ../../v8/src/execution.cc:168 ... R=yangguo@chromium.org BUG= Review-Url: https://codereview.chromium.org/2186013002 Cr-Commit-Position: refs/heads/master@{#38121}
-
- 27 Jul, 2016 2 commits
-
-
qyearsley authored
This is intended to be submitted just after http://crrev.com/2174553002, which renames these builders. BUG=590036 NOTRY=true Review-Url: https://codereview.chromium.org/2181133002 Cr-Commit-Position: refs/heads/master@{#38105}
-
yangguo authored
Review-Url: https://codereview.chromium.org/2175193003 Cr-Commit-Position: refs/heads/master@{#38081}
-
- 26 Jul, 2016 3 commits
-
-
tandrii authored
Revert of [release] Change blink trybot name on v8 roll CLs (patchset #1 id:1 of https://codereview.chromium.org/2185513002/ ) Reason for revert: Revert, because blink tryserver bot rename is reverted. BUG=chromium:631448 Original issue's description: > [release] Change blink trybot name on v8 roll CLs > > BUG=chromium:590036 > NOTRY=true > > Committed: https://crrev.com/a5fae1039409864295b42a6f33cef85ca9396bda > Cr-Commit-Position: refs/heads/master@{#38041} TBR=hablich@chromium.org,machenbach@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:590036 Review-Url: https://codereview.chromium.org/2186593003 Cr-Commit-Position: refs/heads/master@{#38062}
-
bgeron authored
BUG= R=danno Review-Url: https://codereview.chromium.org/2171543004 Cr-Commit-Position: refs/heads/master@{#38045}
-
machenbach authored
BUG=chromium:590036 NOTRY=true Review-Url: https://codereview.chromium.org/2185513002 Cr-Commit-Position: refs/heads/master@{#38041}
-