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

Issue 662326 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Email to this user bounced
Closed: Feb 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Please finish the Oilpan documentation

Project Member Reported by pwnall@chromium.org, Nov 4 2016

Issue description

Version: master
OS: N/A

What steps will reproduce the problem?
(1) Go to https://chromium.googlesource.com/chromium/src/+/master/third_party/WebKit/Source/platform/heap/BlinkGCAPIReference.md#STACK_ALLOCATED
(2) Notice the empty sections around.

What is the expected output?
I'd like to learn about how Oilpan interacts with STACK_ALLOCATED objects. This seems to be important, as there is an empty section in the document.

What do you see instead?
An empty section.

Please use labels and text to provide additional information.
The slide deck linked from the document also misses information on STACK_ALLOCATED and a few other concepts.

Oilpan permeates most of Blink. Please finish up its documentation, so it can achieve its full potential to improve development speed!
 
Components: -Infra>Client>Oilpan Blink>MemoryAllocator>GarbageCollection
Project Member

Comment 2 by bugdroid1@chromium.org, Feb 6 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/218c408dbc4f309adc8f03a845c214bdc70a33c2

commit 218c408dbc4f309adc8f03a845c214bdc70a33c2
Author: sigbjornf <sigbjornf@opera.com>
Date: Mon Feb 06 12:03:11 2017

Add missing documentation for Allocator.h macros + EAGERLY_FINALIZE().

R=haraken
BUG= 662326 

Review-Url: https://codereview.chromium.org/2672273002
Cr-Commit-Position: refs/heads/master@{#448234}

[modify] https://crrev.com/218c408dbc4f309adc8f03a845c214bdc70a33c2/third_party/WebKit/Source/platform/heap/BlinkGCAPIReference.md

Owner: sigbjo...@opera.com
Status: Fixed (was: Untriaged)
Thank you!

Sign in to add a comment