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

Issue 10911336: Merge 127902 - Web Inspector: [REGRESSION] Content is not available for dynamically loaded script s… (Closed)

Created:
8 years, 3 months ago by vsevik
Modified:
8 years, 3 months ago
Reviewers:
vsevik
CC:
chromium-reviews
Base URL:
http://svn.webkit.org/repository/webkit/branches/chromium/1229/
Visibility:
Public.

Description

Merge 127902 - Web Inspector: [REGRESSION] Content is not available for dynamically loaded script sometimes. https://bugs.webkit.org/show_bug.cgi?id=95954 Reviewed by Yury Semikhatsky. Source/WebCore: Resource now loads content from request when it is available. Content was loaded from PageAgent before where it might be not available if the resource was already GCed. Test: http/tests/inspector/resource-tree/resource-request-content-after-loading-and-clearing-cache.html * inspector/front-end/Resource.js: (WebInspector.Resource.prototype._innerRequestContent.contentLoaded): (WebInspector.Resource.prototype._innerRequestContent.resourceContentLoaded): (WebInspector.Resource.prototype._innerRequestContent): LayoutTests: * http/tests/inspector/resource-tree/resource-request-content-after-loading-and-clearing-cache-expected.txt: Added. * http/tests/inspector/resource-tree/resource-request-content-after-loading-and-clearing-cache.html: Added. * http/tests/inspector/resource-tree/resources/dynamic-script.js: Added. (foo): * http/tests/inspector/resources-test.js: TBR=vsevik@chromium.org BUG=146823 Committed: https://trac.webkit.org/changeset/128736

Patch Set 1 #

Messages

Total messages: 1 (0 generated)
vsevik
8 years, 3 months ago (2012-09-17 09:55:35 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698