auto merge of #5229 : hsvalava/servo/m1503, r=jdm

Fixing ConsoleMsg for console.log messages in the Developer Tools web console.
This commit is contained in:
bors-servo 2015-03-19 21:48:48 -06:00
commit ab8d43910c
3 changed files with 17 additions and 9 deletions

View file

@ -67,9 +67,12 @@ struct ConsoleAPICall {
#[derive(RustcEncodable)]
struct ConsoleMsg {
logLevel: u32,
timestamp: u64,
message: String,
level: String,
timeStamp: u64,
arguments: Vec<String>,
filename: String,
lineNumber: u32,
columnNumber: u32,
}
/// Spin up a devtools server that listens for connections on the specified port.
@ -192,14 +195,17 @@ fn run_server(receiver: Receiver<DevtoolsControlMsg>, port: u16) {
let actors = actors.lock().unwrap();
let console_actor = actors.find::<ConsoleActor>(console_actor_name.as_slice());
match console_message {
ConsoleMessage::LogMessage(message) => {
ConsoleMessage::LogMessage(message, filename, lineNumber, columnNumber) => {
let msg = ConsoleAPICall {
from: console_actor.name.clone(),
__type__: "consoleAPICall".to_string(),
message: ConsoleMsg {
logLevel: 0,
timestamp: precise_time_ns(),
message: message,
level: "log".to_string(),
timeStamp: precise_time_ns(),
arguments: vec!(message),
filename: filename,
lineNumber: lineNumber,
columnNumber: columnNumber,
},
};
for stream in console_actor.streams.borrow_mut().iter_mut() {

View file

@ -122,6 +122,7 @@ impl Decodable for Modification {
//TODO: Include options for Warn, Debug, Info, Error messages from Console
#[derive(Clone)]
pub enum ConsoleMessage {
LogMessage(String),
// Log: message, filename, line number, column number
LogMessage(String, String, u32, u32),
//WarnMessage(String),
}

View file

@ -33,7 +33,8 @@ impl Console {
impl<'a> ConsoleMethods for JSRef<'a, Console> {
fn Log(self, message: DOMString) {
println!("{}", message);
propagate_console_msg(&self, ConsoleMessage::LogMessage(message));
//TODO: Sending fake values for filename, lineNumber and columnNumber in LogMessage; adjust later
propagate_console_msg(&self, ConsoleMessage::LogMessage(message, String::from_str("test"), 1, 1));
}
fn Debug(self, message: DOMString) {