Changeset 56799 in webkit

Timestamp:

Mar 30, 2010 10:42:12 AM (14 years ago)

Author:

pfeldman@chromium.org

Message:

2010-03-30 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Copying from the Content tab in Resources includes line numbers.

​https://bugs.webkit.org/show_bug.cgi?id=35104

• inspector/front-end/TextViewer.js: (WebInspector.TextViewer): (WebInspector.TextViewer.prototype._beforeCopy): (WebInspector.TextViewer.prototype._copy):

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• inspector/front-end/TextViewer.js (modified) (2 diffs)

trunk/WebCore/ChangeLog

@@ +1 @@
+2010-03-30  Pavel Feldman  <pfeldman@chromium.org>
+
+        Reviewed by Timothy Hatcher.
+
+        Web Inspector: Copying from the Content tab in Resources includes line numbers.
+
+        https://bugs.webkit.org/show_bug.cgi?id=35104
+
+        * inspector/front-end/TextViewer.js:
+        (WebInspector.TextViewer):
+        (WebInspector.TextViewer.prototype._beforeCopy):
+        (WebInspector.TextViewer.prototype._copy):
+
 2010-03-30  Pavel Feldman  <pfeldman@chromium.org>
 

trunk/WebCore/inspector/front-end/TextViewer.js

@@ -42 +42 @@
     this.element.addEventListener("scroll", this._scroll.bind(this), false);
     this.element.addEventListener("keydown", this._handleKeyDown.bind(this), false);
+    this.element.addEventListener("beforecopy", this._beforeCopy.bind(this), false);
+    this.element.addEventListener("copy", this._copy.bind(this), false);
 
     this._url = url;
@@ -235 +237 @@
             this.element.scrollLeft += scrollValue;
         }
+    },
+
+    _beforeCopy: function(e)
+    {
+        e.preventDefault();
+    },
+
+    _copy: function(e)
+    {
+        var range = this._getSelection();
+        var text = this._textModel.copyRange(range);
+        InspectorFrontendHost.copyText(text);
+        e.preventDefault();
     },