Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(107)

Side by Side Diff: Source/devtools/front_end/main/Main.js

Issue 402873002: DevTools: Build function details' raw location in DebuggerModel (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Comment addressed Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2006, 2007, 2008 Apple Inc. All rights reserved. 2 * Copyright (C) 2006, 2007, 2008 Apple Inc. All rights reserved.
3 * Copyright (C) 2007 Matt Lilek (pewtermoose@gmail.com). 3 * Copyright (C) 2007 Matt Lilek (pewtermoose@gmail.com).
4 * Copyright (C) 2009 Joseph Pecoraro 4 * Copyright (C) 2009 Joseph Pecoraro
5 * 5 *
6 * Redistribution and use in source and binary forms, with or without 6 * Redistribution and use in source and binary forms, with or without
7 * modification, are permitted provided that the following conditions 7 * modification, are permitted provided that the following conditions
8 * are met: 8 * are met:
9 * 9 *
10 * 1. Redistributions of source code must retain the above copyright 10 * 1. Redistributions of source code must retain the above copyright
(...skipping 570 matching lines...) Expand 10 before | Expand all | Expand 10 after
581 { 581 {
582 elementsPanel.stopOmittingDefaultSelection(); 582 elementsPanel.stopOmittingDefaultSelection();
583 WebInspector.Revealer.reveal(node); 583 WebInspector.Revealer.reveal(node);
584 if (!WebInspector.inspectorView.drawerVisible() && !WebInspector._no tFirstInspectElement) 584 if (!WebInspector.inspectorView.drawerVisible() && !WebInspector._no tFirstInspectElement)
585 InspectorFrontendHost.inspectElementCompleted(); 585 InspectorFrontendHost.inspectElementCompleted();
586 WebInspector._notFirstInspectElement = true; 586 WebInspector._notFirstInspectElement = true;
587 object.release(); 587 object.release();
588 } 588 }
589 589
590 if (object.type === "function") { 590 if (object.type === "function") {
591 /**
592 * @param {?Protocol.Error} error
593 * @param {!DebuggerAgent.FunctionDetails} response
594 */
595 object.functionDetails(didGetDetails); 591 object.functionDetails(didGetDetails);
596 return; 592 return;
597 } 593 }
598 594
599 /** 595 /**
600 * @param {?DebuggerAgent.FunctionDetails} response 596 * @param {?WebInspector.DebuggerModel.FunctionDetails} response
601 */ 597 */
602 function didGetDetails(response) 598 function didGetDetails(response)
603 { 599 {
604 object.release(); 600 object.release();
605 601
606 if (!response) 602 if (!response || !response.location)
607 return; 603 return;
608 604
609 WebInspector.Revealer.reveal(WebInspector.DebuggerModel.Location.fro mPayload(object.target(), response.location).toUILocation()); 605 WebInspector.Revealer.reveal(response.location.toUILocation());
610 } 606 }
611 607
612 if (hints.copyToClipboard) 608 if (hints.copyToClipboard)
613 InspectorFrontendHost.copyText(object.value); 609 InspectorFrontendHost.copyText(object.value);
614 object.release(); 610 object.release();
615 }, 611 },
616 612
617 /** 613 /**
618 * @override 614 * @override
619 * @param {string} reason 615 * @param {string} reason
(...skipping 324 matching lines...) Expand 10 before | Expand all | Expand 10 after
944 }, 940 },
945 941
946 /** 942 /**
947 * @param {!WebInspector.Event} event 943 * @param {!WebInspector.Event} event
948 */ 944 */
949 _inspectNode: function(event) 945 _inspectNode: function(event)
950 { 946 {
951 WebInspector.Revealer.reveal(/** @type {!WebInspector.DOMNode} */ (event .data)); 947 WebInspector.Revealer.reveal(/** @type {!WebInspector.DOMNode} */ (event .data));
952 } 948 }
953 } 949 }
OLDNEW
« no previous file with comments | « Source/devtools/front_end/console/ConsoleView.js ('k') | Source/devtools/front_end/sdk/DebuggerModel.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698