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

Issue 7563005: Merge 91858 - Web Inspector: Disable cache option should only clear memory cache, not disable it. (Closed)

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

Description

Merge 91858 - Web Inspector: Disable cache option should only clear memory cache, not disable it. https://bugs.webkit.org/show_bug.cgi?id=65184 Reviewed by Pavel Feldman. Source/WebCore: * inspector/InspectorClient.h: * inspector/InspectorResourceAgent.cpp: (WebCore::InspectorResourceAgent::enable): (WebCore::InspectorResourceAgent::disable): (WebCore::InspectorResourceAgent::setCacheDisabled): (WebCore::InspectorResourceAgent::mainFrameNavigated): Source/WebKit/chromium: * src/InspectorClientImpl.cpp: * src/InspectorClientImpl.h: * src/WebDevToolsAgentImpl.cpp: * src/WebDevToolsAgentImpl.h: LayoutTests: * http/tests/inspector/network/network-disable-cache-memory-expected.txt: * http/tests/inspector/network/network-disable-cache-memory.html: * http/tests/inspector/network/resources/random-script.php: Added. TBR=pfeldman@chromium.org BUG=90757 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=92270

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+58 lines, -73 lines) Patch
M LayoutTests/http/tests/inspector/network/network-disable-cache-memory.html View 1 chunk +50 lines, -51 lines 0 comments Download
M LayoutTests/http/tests/inspector/network/network-disable-cache-memory-expected.txt View 1 chunk +5 lines, -5 lines 0 comments Download
A + LayoutTests/http/tests/inspector/network/resources/random-script.php View 0 chunks +-1 lines, --1 lines 0 comments Download
M Source/WebCore/inspector/InspectorClient.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/WebCore/inspector/InspectorResourceAgent.cpp View 3 chunks +4 lines, -4 lines 0 comments Download
M Source/WebKit/chromium/src/InspectorClientImpl.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/WebKit/chromium/src/InspectorClientImpl.cpp View 1 chunk +0 lines, -6 lines 0 comments Download
M Source/WebKit/chromium/src/WebDevToolsAgentImpl.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/WebKit/chromium/src/WebDevToolsAgentImpl.cpp View 1 chunk +0 lines, -5 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
vsevik
9 years, 4 months ago (2011-08-03 10:49:42 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698