[SCM] WebKit Debian packaging branch, debian/experimental, updated. upstream/1.3.3-9427-gc2be6fc
podivilov at chromium.org
podivilov at chromium.org
Wed Dec 22 16:32:29 UTC 2010
The following commit has been merged in the debian/experimental branch:
commit 869ec66746f4ca35572d4e5b43dde2af55fbf403
Author: podivilov at chromium.org <podivilov at chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date: Thu Nov 25 13:56:43 2010 +0000
2010-11-24 Pavel Podivilov <podivilov at chromium.org>
Reviewed by Yury Semikhatsky.
Web Inspector: set DOM breakpoints by path.
https://bugs.webkit.org/show_bug.cgi?id=50017
This change is needed to be able to push DOM breakpoints for all URL's to backend
at frontend load.
* inspector/InspectorController.cpp:
(WebCore::InspectorController::setNativeBreakpoint):
* inspector/InspectorDOMAgent.cpp:
(WebCore::InspectorDOMAgent::InspectorDOMAgent):
(WebCore::InspectorDOMAgent::discardBindings):
(WebCore::InspectorDOMAgent::setDOMBreakpoint):
(WebCore::InspectorDOMAgent::removeDOMBreakpoint):
(WebCore::InspectorDOMAgent::descriptionForDOMEvent):
(WebCore::InspectorDOMAgent::removeBreakpointsForNode):
* inspector/InspectorDOMAgent.h:
* inspector/front-end/BreakpointManager.js:
(WebInspector.BreakpointManager.prototype.reset):
(WebInspector.BreakpointManager.prototype.createDOMBreakpoint):
(WebInspector.BreakpointManager.prototype._updateNativeBreakpointsInSettings):
(WebInspector.BreakpointManager.prototype.restoreBreakpoints):
(WebInspector.DOMBreakpoint):
(WebInspector.DOMBreakpoint.prototype._setNodeId):
(WebInspector.DOMBreakpoint.prototype.populateLabelElement):
(WebInspector.EventListenerBreakpoint):
(WebInspector.XHRBreakpoint):
* inspector/front-end/BreakpointsSidebarPane.js:
(WebInspector.BreakpointsSidebarPane):
(WebInspector.BreakpointsSidebarPane.prototype._reset):
(WebInspector.EventListenerBreakpointsSidebarPane):
(WebInspector.EventListenerBreakpointsSidebarPane.prototype._reset):
* inspector/front-end/DOMAgent.js:
(WebInspector.DOMAgent.prototype._setDocument):
* inspector/front-end/ElementsPanel.js:
(WebInspector.ElementsPanel.prototype.reset):
* inspector/front-end/ElementsTreeOutline.js:
(WebInspector.ElementsTreeElement.prototype._populateTagContextMenu.handlerFunction):
(WebInspector.ElementsTreeElement.prototype._populateTagContextMenu):
* inspector/front-end/ScriptsPanel.js:
(WebInspector.ScriptsPanel.prototype.reset):
* inspector/front-end/inspector.js:
(WebInspector.reset):
(WebInspector.didCommitLoad):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@72727 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 425e30a..4c7e92a 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,51 @@
+2010-11-24 Pavel Podivilov <podivilov at chromium.org>
+
+ Reviewed by Yury Semikhatsky.
+
+ Web Inspector: set DOM breakpoints by path.
+ https://bugs.webkit.org/show_bug.cgi?id=50017
+
+ This change is needed to be able to push DOM breakpoints for all URL's to backend
+ at frontend load.
+
+ * inspector/InspectorController.cpp:
+ (WebCore::InspectorController::setNativeBreakpoint):
+ * inspector/InspectorDOMAgent.cpp:
+ (WebCore::InspectorDOMAgent::InspectorDOMAgent):
+ (WebCore::InspectorDOMAgent::discardBindings):
+ (WebCore::InspectorDOMAgent::setDOMBreakpoint):
+ (WebCore::InspectorDOMAgent::removeDOMBreakpoint):
+ (WebCore::InspectorDOMAgent::descriptionForDOMEvent):
+ (WebCore::InspectorDOMAgent::removeBreakpointsForNode):
+ * inspector/InspectorDOMAgent.h:
+ * inspector/front-end/BreakpointManager.js:
+ (WebInspector.BreakpointManager.prototype.reset):
+ (WebInspector.BreakpointManager.prototype.createDOMBreakpoint):
+ (WebInspector.BreakpointManager.prototype._updateNativeBreakpointsInSettings):
+ (WebInspector.BreakpointManager.prototype.restoreBreakpoints):
+ (WebInspector.DOMBreakpoint):
+ (WebInspector.DOMBreakpoint.prototype._setNodeId):
+ (WebInspector.DOMBreakpoint.prototype.populateLabelElement):
+ (WebInspector.EventListenerBreakpoint):
+ (WebInspector.XHRBreakpoint):
+ * inspector/front-end/BreakpointsSidebarPane.js:
+ (WebInspector.BreakpointsSidebarPane):
+ (WebInspector.BreakpointsSidebarPane.prototype._reset):
+ (WebInspector.EventListenerBreakpointsSidebarPane):
+ (WebInspector.EventListenerBreakpointsSidebarPane.prototype._reset):
+ * inspector/front-end/DOMAgent.js:
+ (WebInspector.DOMAgent.prototype._setDocument):
+ * inspector/front-end/ElementsPanel.js:
+ (WebInspector.ElementsPanel.prototype.reset):
+ * inspector/front-end/ElementsTreeOutline.js:
+ (WebInspector.ElementsTreeElement.prototype._populateTagContextMenu.handlerFunction):
+ (WebInspector.ElementsTreeElement.prototype._populateTagContextMenu):
+ * inspector/front-end/ScriptsPanel.js:
+ (WebInspector.ScriptsPanel.prototype.reset):
+ * inspector/front-end/inspector.js:
+ (WebInspector.reset):
+ (WebInspector.didCommitLoad):
+
2010-11-25 Pavel Feldman <pfeldman at chromium.org>
Reviewed by Yury Semikhatsky.
diff --git a/WebCore/inspector/InspectorController.cpp b/WebCore/inspector/InspectorController.cpp
index 243b7ac..224477d 100644
--- a/WebCore/inspector/InspectorController.cpp
+++ b/WebCore/inspector/InspectorController.cpp
@@ -1426,15 +1426,13 @@ void InspectorController::setNativeBreakpoint(PassRefPtr<InspectorObject> breakp
} else if (type == domNativeBreakpointType) {
if (!m_domAgent)
return;
- double nodeIdNumber;
- if (!condition->getNumber("nodeId", &nodeIdNumber))
+ String path;
+ if (!condition->getString("path", &path))
return;
- double domBreakpointTypeNumber;
- if (!condition->getNumber("type", &domBreakpointTypeNumber))
+ double domBreakpointType;
+ if (!condition->getNumber("type", &domBreakpointType))
return;
- long nodeId = (long) nodeIdNumber;
- long domBreakpointType = (long) domBreakpointTypeNumber;
- *breakpointId = m_domAgent->setDOMBreakpoint(nodeId, domBreakpointType);
+ *breakpointId = m_domAgent->setDOMBreakpoint(path, static_cast<long>(domBreakpointType));
if (!breakpointId->isEmpty())
m_nativeBreakpoints.set(*breakpointId, type);
}
diff --git a/WebCore/inspector/InspectorDOMAgent.cpp b/WebCore/inspector/InspectorDOMAgent.cpp
index 894e4b2..41eb90a 100644
--- a/WebCore/inspector/InspectorDOMAgent.cpp
+++ b/WebCore/inspector/InspectorDOMAgent.cpp
@@ -223,6 +223,7 @@ InspectorDOMAgent::InspectorDOMAgent(InspectorCSSStore* cssStore, InspectorFront
, m_domListener(0)
, m_lastNodeId(1)
, m_matchJobsTimer(this, &InspectorDOMAgent::onMatchJobsTimer)
+ , m_lastBreakpointId(0)
{
}
@@ -398,6 +399,8 @@ void InspectorDOMAgent::discardBindings()
m_childrenRequested.clear();
m_inspectedNodes.clear();
m_breakpoints.clear();
+ m_idToBreakpoint.clear();
+ m_breakpointToId.clear();
}
Node* InspectorDOMAgent::nodeForId(long id)
@@ -762,36 +765,40 @@ void InspectorDOMAgent::searchCanceled()
m_searchResults.clear();
}
-String InspectorDOMAgent::setDOMBreakpoint(long nodeId, long type)
+String InspectorDOMAgent::setDOMBreakpoint(const String& path, long type)
{
- Node* node = nodeForId(nodeId);
+ Node* node = nodeForPath(path);
if (!node)
return "";
- String breakpointId = createBreakpointId(nodeId, type);
- if (m_idToBreakpoint.contains(breakpointId))
+ uint32_t rootBit = 1 << type;
+ uint32_t mask = m_breakpoints.get(node);
+ if (mask & rootBit)
return "";
- m_idToBreakpoint.set(breakpointId, std::make_pair(nodeId, type));
-
- uint32_t rootBit = 1 << type;
- m_breakpoints.set(node, m_breakpoints.get(node) | rootBit);
+ m_breakpoints.set(node, mask | rootBit);
if (rootBit & inheritableDOMBreakpointTypesMask) {
for (Node* child = innerFirstChild(node); child; child = innerNextSibling(child))
updateSubtreeBreakpoints(child, rootBit, true);
}
+ String breakpointId = makeString("dom:", String::number(++m_lastBreakpointId));
+ Breakpoint breakpoint(node, type);
+ m_idToBreakpoint.set(breakpointId, breakpoint);
+ m_breakpointToId.set(breakpoint, breakpointId);
return breakpointId;
}
void InspectorDOMAgent::removeDOMBreakpoint(const String& breakpointId)
{
- Breakpoint breakpoint = m_idToBreakpoint.take(breakpointId);
-
- Node* node = nodeForId(breakpoint.first);
- if (!node)
+ HashMap<String, Breakpoint>::iterator it = m_idToBreakpoint.find(breakpointId);
+ if (it == m_idToBreakpoint.end())
return;
+ Breakpoint breakpoint = it->second;
+ m_idToBreakpoint.remove(it);
+ m_breakpointToId.remove(breakpoint);
+ Node* node = breakpoint.first;
uint32_t rootBit = 1 << breakpoint.second;
uint32_t mask = m_breakpoints.get(node) & ~rootBit;
if (mask)
@@ -862,9 +869,8 @@ PassRefPtr<InspectorValue> InspectorDOMAgent::descriptionForDOMEvent(Node* targe
description->setBoolean("insertion", insertion);
}
- long breakpointOwnerNodeId = m_documentNodeToIdMap.get(breakpointOwner);
- ASSERT(breakpointOwnerNodeId);
- description->setString("breakpointId", createBreakpointId(breakpointOwnerNodeId, breakpointType));
+ String breakpointId = m_breakpointToId.get(Breakpoint(breakpointOwner, breakpointType));
+ description->setString("breakpointId", breakpointId);
return description;
}
@@ -1187,20 +1193,15 @@ void InspectorDOMAgent::removeBreakpointsForNode(Node* node)
uint32_t mask = m_breakpoints.take(node);
if (!mask)
return;
- long nodeId = m_documentNodeToIdMap.get(node);
- if (!nodeId)
- return;
for (int type = 0; type < DOMBreakpointTypesCount; ++type) {
- if (mask && (1 << type))
- m_idToBreakpoint.remove(createBreakpointId(nodeId, type));
+ if (mask & (1 << type)) {
+ Breakpoint breakpoint(node, type);
+ String breakpointId = m_breakpointToId.take(breakpoint);
+ m_idToBreakpoint.remove(breakpointId);
+ }
}
}
-String InspectorDOMAgent::createBreakpointId(long nodeId, long type)
-{
- return makeString("dom:", String::number(nodeId), ':', String::number(type));
-}
-
void InspectorDOMAgent::getStyles(long nodeId, bool authorOnly, RefPtr<InspectorValue>* styles)
{
Node* node = nodeForId(nodeId);
diff --git a/WebCore/inspector/InspectorDOMAgent.h b/WebCore/inspector/InspectorDOMAgent.h
index 84ca027..6655476 100644
--- a/WebCore/inspector/InspectorDOMAgent.h
+++ b/WebCore/inspector/InspectorDOMAgent.h
@@ -158,7 +158,7 @@ namespace WebCore {
String documentURLString(Document* document) const;
- String setDOMBreakpoint(long nodeId, long type);
+ String setDOMBreakpoint(const String& path, long type);
void removeDOMBreakpoint(const String& breakpointId);
private:
@@ -178,7 +178,6 @@ namespace WebCore {
void updateSubtreeBreakpoints(Node* root, uint32_t rootMask, bool value);
void removeBreakpointsForNode(Node* node);
PassRefPtr<InspectorValue> descriptionForDOMEvent(Node* target, long breakpointType, bool insertion);
- String createBreakpointId(long nodeId, long type);
PassRefPtr<InspectorObject> buildObjectForAttributeStyles(Element* element);
PassRefPtr<InspectorArray> buildArrayForCSSRules(Document* ownerDocument, CSSRuleList*);
@@ -233,8 +232,10 @@ namespace WebCore {
HashSet<RefPtr<Node> > m_searchResults;
Vector<long> m_inspectedNodes;
HashMap<Node*, uint32_t> m_breakpoints;
- typedef pair<long, long> Breakpoint;
+ typedef pair<Node*, long> Breakpoint;
HashMap<String, Breakpoint> m_idToBreakpoint;
+ HashMap<Breakpoint, String> m_breakpointToId;
+ unsigned int m_lastBreakpointId;
};
#endif
diff --git a/WebCore/inspector/front-end/BreakpointManager.js b/WebCore/inspector/front-end/BreakpointManager.js
index 3d51092..d120033 100644
--- a/WebCore/inspector/front-end/BreakpointManager.js
+++ b/WebCore/inspector/front-end/BreakpointManager.js
@@ -88,6 +88,7 @@ WebInspector.BreakpointManager.prototype = {
this._breakpoints = {};
delete this._oneTimeBreakpoint;
this._nativeBreakpoints = {};
+ this.dispatchEventToListeners("reset");
},
_setBreakpoint: function(sourceID, url, line, enabled, condition)
@@ -128,16 +129,17 @@ WebInspector.BreakpointManager.prototype = {
InspectorBackend.setBreakpoint(breakpoint.sourceID, breakpoint.line, breakpoint.enabled, breakpoint.condition, didSetBreakpoint.bind(this));
},
- createDOMBreakpoint: function(nodeId, domEventType, disabled)
+ createDOMBreakpoint: function(path, domEventType, disabled)
{
- var frontendId = "dom:" + nodeId + ":" + domEventType;
+ var frontendId = "dom:" + path + ":" + domEventType;
if (frontendId in this._nativeBreakpoints)
return;
- var breakpoint = new WebInspector.DOMBreakpoint(this, frontendId, nodeId, domEventType);
+ var breakpoint = new WebInspector.DOMBreakpoint(this, frontendId, path, domEventType);
this._nativeBreakpoints[frontendId] = breakpoint;
this.dispatchEventToListeners("dom-breakpoint-added", breakpoint);
breakpoint.enabled = !disabled;
+ InspectorBackend.pushNodeByPathToFrontend(path, breakpoint.setNodeId.bind(breakpoint));
return breakpoint;
},
@@ -222,8 +224,7 @@ WebInspector.BreakpointManager.prototype = {
var persistentBreakpoints = [];
for (var id in this._nativeBreakpoints) {
var breakpoint = this._nativeBreakpoints[id];
- if (breakpoint._persistentCondition)
- persistentBreakpoints.push({ type: breakpoint._type, enabled: breakpoint.enabled, condition: breakpoint._persistentCondition });
+ persistentBreakpoints.push({ type: breakpoint._type, enabled: breakpoint.enabled, condition: breakpoint._condition });
}
WebInspector.settings.nativeBreakpoints = persistentBreakpoints;
},
@@ -264,41 +265,14 @@ WebInspector.BreakpointManager.prototype = {
{
var breakpoints = this._persistentBreakpoints();
for (var i = 0; i < breakpoints.length; ++i) {
- if (breakpoints[i].type === "EventListener")
- this.createEventListenerBreakpoint(breakpoints[i].condition.eventName);
- else if (breakpoints[i].type === "XHR")
- this.createXHRBreakpoint(breakpoints[i].condition.url, !breakpoints[i].enabled);
- }
- },
-
- restoreDOMBreakpoints: function()
- {
- function didPushNodeByPathToFrontend(path, nodeId)
- {
- pathToNodeId[path] = nodeId;
- pendingCalls -= 1;
- if (pendingCalls)
- return;
- for (var i = 0; i < breakpoints.length; ++i) {
- var breakpoint = breakpoints[i];
- var nodeId = pathToNodeId[breakpoint.condition.path];
- if (nodeId)
- this.createDOMBreakpoint(nodeId, breakpoint.condition.type, !breakpoint.enabled);
- }
- }
-
- var breakpoints = this._persistentBreakpoints();
- var pathToNodeId = {};
- var pendingCalls = 0;
- for (var i = 0; i < breakpoints.length; ++i) {
- if (breakpoints[i].type !== "DOM")
- continue;
- var path = breakpoints[i].condition.path;
- if (path in pathToNodeId)
- continue;
- pathToNodeId[path] = 0;
- pendingCalls += 1;
- InspectorBackend.pushNodeByPathToFrontend(path, didPushNodeByPathToFrontend.bind(this, path));
+ var type = breakpoints[i].type;
+ var condition = breakpoints[i].condition;
+ if (type === "DOM")
+ this.createDOMBreakpoint(condition.path, condition.type, !breakpoints[i].enabled);
+ else if (type === "EventListener")
+ this.createEventListenerBreakpoint(condition.eventName);
+ else if (type === "XHR")
+ this.createXHRBreakpoint(condition.url, !breakpoints[i].enabled);
}
},
@@ -469,21 +443,26 @@ WebInspector.NativeBreakpoint.prototype = {
WebInspector.NativeBreakpoint.prototype.__proto__ = WebInspector.Object.prototype;
-WebInspector.DOMBreakpoint = function(manager, frontendId, nodeId, domEventType)
+WebInspector.DOMBreakpoint = function(manager, frontendId, path, domEventType)
{
WebInspector.NativeBreakpoint.call(this, manager, frontendId, "DOM");
- this._nodeId = nodeId;
+ this._path = path;
this._domEventType = domEventType;
- this._condition = { nodeId: this._nodeId, type: this._domEventType };
-
- var node = WebInspector.domAgent.nodeForId(this._nodeId);
- if (node) {
- node.breakpoints[this._domEventType] = this;
- this._persistentCondition = { path: node.path(), type: this._domEventType };
- }
+ this._condition = { path: this._path, type: this._domEventType };
}
WebInspector.DOMBreakpoint.prototype = {
+ setNodeId: function(nodeId)
+ {
+ this._nodeId = nodeId;
+ var node = WebInspector.domAgent.nodeForId(this._nodeId);
+ if (node) {
+ node.breakpoints[this._domEventType] = this;
+ this.dispatchEventToListeners("label-changed");
+ } else
+ this.remove();
+ },
+
compareTo: function(other)
{
return this._compare(this._domEventType, other._domEventType);
@@ -492,6 +471,9 @@ WebInspector.DOMBreakpoint.prototype = {
populateLabelElement: function(element)
{
// FIXME: this should belong to the view, not the manager.
+ if (!this._nodeId)
+ return;
+
var linkifiedNode = WebInspector.panels.elements.linkifyNodeById(this._nodeId);
linkifiedNode.addStyleClass("monospace");
element.appendChild(linkifiedNode);
@@ -544,7 +526,6 @@ WebInspector.EventListenerBreakpoint = function(manager, frontendId, eventName)
WebInspector.NativeBreakpoint.call(this, manager, frontendId, "EventListener");
this._eventName = eventName;
this._condition = { eventName: this._eventName };
- this._persistentCondition = this._condition;
}
WebInspector.EventListenerBreakpoint.eventNameForUI = function(eventName)
@@ -589,7 +570,6 @@ WebInspector.XHRBreakpoint = function(manager, frontendId, url)
WebInspector.NativeBreakpoint.call(this, manager, frontendId, "XHR");
this._url = url;
this._condition = { url: this._url };
- this._persistentCondition = this._condition;
}
WebInspector.XHRBreakpoint.prototype = {
diff --git a/WebCore/inspector/front-end/BreakpointsSidebarPane.js b/WebCore/inspector/front-end/BreakpointsSidebarPane.js
index 47194da..552fa88 100644
--- a/WebCore/inspector/front-end/BreakpointsSidebarPane.js
+++ b/WebCore/inspector/front-end/BreakpointsSidebarPane.js
@@ -35,10 +35,12 @@ WebInspector.BreakpointsSidebarPane = function(title)
this.emptyElement.textContent = WebInspector.UIString("No Breakpoints");
this.bodyElement.appendChild(this.emptyElement);
+
+ WebInspector.breakpointManager.addEventListener("reset", this._reset, this);
}
WebInspector.BreakpointsSidebarPane.prototype = {
- reset: function()
+ _reset: function()
{
this.listElement.removeChildren();
if (this.listElement.parentElement) {
@@ -257,6 +259,7 @@ WebInspector.EventListenerBreakpointsSidebarPane = function()
this.bodyElement.appendChild(this.categoriesElement);
WebInspector.breakpointManager.addEventListener("event-listener-breakpoint-added", this._breakpointAdded, this);
+ WebInspector.breakpointManager.addEventListener("reset", this._reset, this);
this._breakpointItems = {};
this._createCategory("Keyboard", "listener", ["keydown", "keyup", "keypress", "textInput"]);
@@ -383,7 +386,7 @@ WebInspector.EventListenerBreakpointsSidebarPane.prototype = {
categoryItem.checkbox.indeterminate = hasEnabled && hasDisabled;
},
- reset: function()
+ _reset: function()
{
for (var eventName in this._breakpointItems) {
var breakpointItem = this._breakpointItems[eventName];
diff --git a/WebCore/inspector/front-end/DOMAgent.js b/WebCore/inspector/front-end/DOMAgent.js
index 37bf549..ad1eb94 100644
--- a/WebCore/inspector/front-end/DOMAgent.js
+++ b/WebCore/inspector/front-end/DOMAgent.js
@@ -379,7 +379,6 @@ WebInspector.DOMAgent.prototype = {
this.document = new WebInspector.DOMDocument(this, this._window, payload);
this._idToDOMNode[payload.id] = this.document;
this._bindNodes(this.document.children);
- WebInspector.breakpointManager.restoreDOMBreakpoints();
} else
this.document = null;
WebInspector.panels.elements.setDocument(this.document);
diff --git a/WebCore/inspector/front-end/ElementsPanel.js b/WebCore/inspector/front-end/ElementsPanel.js
index 61ba33e..b1cd24c 100644
--- a/WebCore/inspector/front-end/ElementsPanel.js
+++ b/WebCore/inspector/front-end/ElementsPanel.js
@@ -169,9 +169,6 @@ WebInspector.ElementsPanel.prototype = {
this.recentlyModifiedNodes = [];
delete this.currentQuery;
-
- if (Preferences.nativeInstrumentationEnabled)
- this.sidebarPanes.domBreakpoints.reset();
},
setDocument: function(inspectedRootDocument)
diff --git a/WebCore/inspector/front-end/ElementsTreeOutline.js b/WebCore/inspector/front-end/ElementsTreeOutline.js
index f893ca0..20b3ac5 100644
--- a/WebCore/inspector/front-end/ElementsTreeOutline.js
+++ b/WebCore/inspector/front-end/ElementsTreeOutline.js
@@ -765,9 +765,9 @@ WebInspector.ElementsTreeElement.prototype = {
// Add debbuging-related actions
contextMenu.appendSeparator();
- function handlerFunction(nodeId, breakType)
+ function handlerFunction(path, breakType)
{
- WebInspector.breakpointManager.createDOMBreakpoint(nodeId, breakType);
+ WebInspector.breakpointManager.createDOMBreakpoint(path, breakType);
WebInspector.panels.elements.sidebarPanes.domBreakpoints.expand();
}
var node = this.representedObject;
@@ -776,7 +776,7 @@ WebInspector.ElementsTreeElement.prototype = {
var label = WebInspector.domBreakpointTypeContextMenuLabel(type);
var breakpoint = node.breakpoints[type];
if (!breakpoint)
- var handler = handlerFunction.bind(this, node.id, type);
+ var handler = handlerFunction.bind(this, node.path(), type);
else
var handler = breakpoint.remove.bind(breakpoint);
contextMenu.appendCheckboxItem(label, handler, !!breakpoint);
diff --git a/WebCore/inspector/front-end/ScriptsPanel.js b/WebCore/inspector/front-end/ScriptsPanel.js
index 0a3c7a9..e018a2f 100644
--- a/WebCore/inspector/front-end/ScriptsPanel.js
+++ b/WebCore/inspector/front-end/ScriptsPanel.js
@@ -456,15 +456,8 @@ WebInspector.ScriptsPanel.prototype = {
this._sourceIDMap = {};
this.sidebarPanes.watchExpressions.refreshExpressions();
- if (!preserveItems) {
- this.sidebarPanes.jsBreakpoints.reset();
- if (Preferences.nativeInstrumentationEnabled) {
- this.sidebarPanes.domBreakpoints.reset();
- this.sidebarPanes.xhrBreakpoints.reset();
- this.sidebarPanes.eventListenerBreakpoints.reset();
- }
+ if (!preserveItems)
this.sidebarPanes.workers.reset();
- }
},
get visibleView()
diff --git a/WebCore/inspector/front-end/inspector.js b/WebCore/inspector/front-end/inspector.js
index 78592ff..0356dbb 100644
--- a/WebCore/inspector/front-end/inspector.js
+++ b/WebCore/inspector/front-end/inspector.js
@@ -1375,8 +1375,6 @@ WebInspector.reset = function()
this.console.clearMessages();
this.extensionServer.notifyInspectorReset();
-
- this.breakpointManager.restoreBreakpoints();
}
WebInspector.resetProfilesPanel = function()
@@ -1405,6 +1403,7 @@ WebInspector.didCommitLoad = function()
{
// Cleanup elements panel early on inspected page refresh.
WebInspector.setDocument(null);
+ this.breakpointManager.restoreBreakpoints();
}
WebInspector.updateConsoleMessageExpiredCount = function(count)
--
WebKit Debian packaging
More information about the Pkg-webkit-commits
mailing list