Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-06-19 14:48:59 +01:00
Code Issues Projects Releases Packages Wiki Activity

auto merge of #3765 : pcwalton/servo/profiler-cleanups, r=cgaebel

Browse source 
The only real user-visible change this effects is to trim URLs to 30 characters so they don't make huge lines on the terminal.

r? @cgaebel
This commit is contained in:
bors-servo 2014-10-22 14:45:35 -06:00
commit a6f0159cb8
3 changed files with 29 additions and 58 deletions
Download patch file Download diff file Expand all files Collapse all files

17
components/util/time.rs
View file

@ -27,9 +27,9 @@ impl TimeProfilerChan {
#[deriving(PartialEq, Clone, PartialOrd, Eq, Ord)]
pub struct TimerMetadata {
url: String,
iframe: bool,
first_reflow: bool,
url: String,
iframe: bool,
incremental: bool,
}
pub trait Formatable {
@ -42,9 +42,14 @@ impl Formatable for Option<TimerMetadata> {
// TODO(cgaebel): Center-align in the format strings as soon as rustc supports it.
&Some(ref meta) => {
let url = meta.url.as_slice();
let first_reflow = if meta.first_reflow { " yes" } else { " no " };
let url = if url.len() > 30 {
url.slice_to(30)
} else {
url
};
let incremental = if meta.incremental { " yes" } else { " no " };
let iframe = if meta.iframe { " yes" } else { " no " };
format!(" {:14} {:9} {:30}", first_reflow, iframe, url)
format!(" {:14} {:9} {:30}", incremental, iframe, url)
},
&None =>
format!(" {:14} {:9} {:30}", " N/A", " N/A", " N/A")
@ -256,7 +261,7 @@ pub fn profile<T>(category: TimeProfilerCategory,
TimerMetadata {
url: url.serialize(),
iframe: iframe,
first_reflow: first_reflow,
incremental: !first_reflow,
});
time_profiler_chan.send(TimeMsg((category, meta), ms));
return val;