Chromium Code Reviews
|
|
Chromium Code Reviews
Issue
569173004:
adb_profile_chrome: --trace-scheduler should also trace blink's scheduler (Closed)
Descriptionadb_profile_chrome: --trace-scheduler should also trace blink's scheduler
BUG=391005
Committed: https://crrev.com/7a5c6ba96def019bd77667858a40bf8fcfa025fe
Cr-Commit-Position: refs/heads/master@{#294828}
Patch Set 1 #
Messages
Total messages: 14 (5 generated)
skyostil@chromium.org changed reviewers: + picksi@chromium.org
lgtm!
The CQ bit was checked by skyostil@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patchset/569173004/1
The CQ bit was unchecked by commit-bot@chromium.org
No LGTM from a valid reviewer yet. Only full committers are accepted. Even if an LGTM may have been provided, it was from a non-committer or a provisional committer, _not_ a full super star committer. See http://www.chromium.org/getting-involved/become-a-committer Note that this has nothing to do with OWNERS files.
skyostil@chromium.org changed reviewers: + andrewhayden@chromium.org
Andrew, could you give me your Seal of Quality?
On 2014/09/15 13:51:31, Sami wrote:
> Andrew, could you give me your Seal of Quality?
LGTM based on a reading of the bug.
## ##
####### ###
/#### ##
/ ### ###
/ ### ##
/ ## ##
/## ## ##
## # ##
/ ## ## ##
### ##
/ ## ### /### ### ## ### /### /## ## ### ####
/### ## ### /## /### ##
/ ## ###/ #### / ######### ###/ #### / / ### ## ### ### /
/ #### / ### ### / ### / ### / ##
/ ## ## ###/ ## #### ## ###/ / ### ## ### ###/
## ###/ ### ### / ### / ###/ ##
/ ## ## ## ## ## ## ## ### ## ## ##
#### ### /## ## ### ## ## ##
/######## ## ## ## ## ## ######## ## ## ##
### #/ /## ######## ## ## ##
/ ## ## ## ## ## ## ####### ## ## ##
### #/ ## ####### ## ## ##
# ## ## ## ## ## ## ## ## ## ##
### # / ## ## ## ##
/#### ## ## ## ## /# ## #### / ## /# /
/### ## /## / #### / ## /# ##
/ #### ## / ### ### ####/ ### ######/ ######/ ######/ /
#### / / ########/ ######/ ####/ ## ### /
/ ## #/ ### ### ### ### ##### ##### #####
###/ / ##### ##### ### ## ##/
#
|
##
\)
# ### /## # ### ###
/ /### #/ ### / /### ### #
/ / ### ## ### / / ### ## ###
#
/ ## ### ## / ## ### ## #
##
/ ### ### ## / ### ### ##
##
## ## ## ###### ## ## ## ## #### /### ## ###
######## ## ####
## ## ## ##### ## ## ## ## ### / / ### / ## ###
######## ## ### /
## ## ## ## ## ## ## ## ###/ / ###/ ## ##
## ## ###/
## ## ## ## ## ## ## ## ## ## ## ## ##
## ## ##
## ## ## ## ## ## ## ## ## ## ## ## ##
## ## ##
## ## ## ## ## ## ### ## ## ## ## ## ## ##
## ## ##
## # / ## ## # #### ## ## ## ## ## ##
## ## ##
### / ## ### /## ## /# ## /# ## ##
## ## ##
######/ ## ######/ ## ######/ ## ####/ ## ### / ###
/ ## #########
### ## ### ## ##### ## ### ## ##/
##/ ## #### ###
##
###
/
##### ###
/
/####### /#
/
/ ###/
The CQ bit was checked by skyostil@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patchset/569173004/1
Message was sent while issue was closed.
Committed patchset #1 (id:1) as 4a1052b54f42bbed991973788c661a0fe1dbc654
Message was sent while issue was closed.
Patchset 1 (id:??) landed as https://crrev.com/7a5c6ba96def019bd77667858a40bf8fcfa025fe Cr-Commit-Position: refs/heads/master@{#294828} |
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
