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

Side by Side Diff: Source/devtools/front_end/elements/MetricsSidebarPane.js

Issue 1104163003: Devtools: [ElementsPanel] Add dom listeners in sidebars (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@move-force-state
Patch Set: Address offline comments Created 5 years, 7 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
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2007 Apple Inc. All rights reserved. 2 * Copyright (C) 2007 Apple Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions 5 * modification, are permitted provided that the following conditions
6 * are met: 6 * are met:
7 * 7 *
8 * 1. Redistributions of source code must retain the above copyright 8 * 1. Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * 2. Redistributions in binary form must reproduce the above copyright 10 * 2. Redistributions in binary form must reproduce the above copyright
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
79 { 79 {
80 if (style && this.node() === node) 80 if (style && this.node() === node)
81 this.inlineStyle = style; 81 this.inlineStyle = style;
82 finishedCallback(); 82 finishedCallback();
83 } 83 }
84 cssModel.getInlineStylesAsync(node.id, inlineStyleCallback.bind(this)); 84 cssModel.getInlineStylesAsync(node.id, inlineStyleCallback.bind(this));
85 }, 85 },
86 86
87 /** 87 /**
88 * @override 88 * @override
89 * @param {!WebInspector.DOMNode} changedNode
90 */ 89 */
91 onDOMNodeChanged: function(changedNode) 90 onDOMModelChanged: function()
92 { 91 {
93 if (this.node() !== changedNode)
lushnikov 2015/05/26 18:24:30 yay!
94 return;
95
96 this.update(); 92 this.update();
97 }, 93 },
98 94
99 /** 95 /**
100 * @override 96 * @override
101 */ 97 */
102 onCSSModelChanged: function() 98 onCSSModelChanged: function()
103 { 99 {
104 this.update(); 100 this.update();
105 }, 101 },
(...skipping 354 matching lines...) Expand 10 before | Expand all | Expand 10 after
460 }, 456 },
461 457
462 editingCommitted: function(element, userInput, previousContent, context) 458 editingCommitted: function(element, userInput, previousContent, context)
463 { 459 {
464 this.editingEnded(element, context); 460 this.editingEnded(element, context);
465 this._applyUserInput(element, userInput, previousContent, context, true) ; 461 this._applyUserInput(element, userInput, previousContent, context, true) ;
466 }, 462 },
467 463
468 __proto__: WebInspector.ElementsSidebarPane.prototype 464 __proto__: WebInspector.ElementsSidebarPane.prototype
469 } 465 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698