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

Issue 211012: DevTools: don't duplicate eval scripts on each panel switch. (Closed)

Created:
11 years, 3 months ago by yurys
Modified:
9 years, 7 months ago
Reviewers:
apavlov
CC:
chromium-reviews_googlegroups.com, darin (slow to review), pam+watch_chromium.org
Visibility:
Public.

Description

DevTools: don't duplicate eval scripts on each panel switch. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=26562

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+60 lines, -8 lines) Patch
M chrome/browser/debugger/devtools_sanity_unittest.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M webkit/glue/devtools/js/debugger_agent.js View 2 1 chunk +0 lines, -8 lines 0 comments Download
M webkit/glue/devtools/js/tests.js View 1 chunk +54 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
yurys
11 years, 3 months ago (2009-09-17 15:37:12 UTC) #1
yurys
Added a unit test. Please review.
11 years, 3 months ago (2009-09-18 13:39:12 UTC) #2
apavlov
11 years, 3 months ago (2009-09-18 13:42:55 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld 408576698