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

Unified Diff: third_party/WebKit/Source/core/inspector/InspectorResourceAgent.cpp

Issue 1825033002: Revert of DevTools: Select max resource size in NetworkResourcesData depending on device capabilities. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/inspector/InspectorResourceAgent.cpp
diff --git a/third_party/WebKit/Source/core/inspector/InspectorResourceAgent.cpp b/third_party/WebKit/Source/core/inspector/InspectorResourceAgent.cpp
index bf2cc3575ed7460bbdb9c7fd61005d1f81799c2a..e5e64e6934c590a08b5062ed940cb67954766ad9 100644
--- a/third_party/WebKit/Source/core/inspector/InspectorResourceAgent.cpp
+++ b/third_party/WebKit/Source/core/inspector/InspectorResourceAgent.cpp
@@ -86,16 +86,6 @@
static const char userAgentOverride[] = "userAgentOverride";
static const char monitoringXHR[] = "monitoringXHR";
static const char blockedURLs[] = "blockedURLs";
-static const char totalBufferSize[] = "totalBufferSize";
-static const char resourceBufferSize[] = "resourceBufferSize";
-}
-
-namespace {
-// 100MB
-static size_t maximumTotalBufferSize = 100 * 1000 * 1000;
-
-// 10MB
-static size_t maximumResourceBufferSize = 10 * 1000 * 1000;
}
namespace {
@@ -257,11 +247,8 @@
void InspectorResourceAgent::restore()
{
- if (m_state->booleanProperty(ResourceAgentState::resourceAgentEnabled, false)) {
- enable(
- m_state->numberProperty(ResourceAgentState::totalBufferSize, maximumTotalBufferSize),
- m_state->numberProperty(ResourceAgentState::resourceBufferSize, maximumResourceBufferSize));
- }
+ if (m_state->booleanProperty(ResourceAgentState::resourceAgentEnabled, false))
+ enable();
}
static PassOwnPtr<protocol::Network::ResourceTiming> buildObjectForTiming(const ResourceLoadTiming& timing)
@@ -883,19 +870,16 @@
frontend()->webSocketFrameError(IdentifiersFactory::requestId(identifier), monotonicallyIncreasingTime(), errorMessage);
}
-void InspectorResourceAgent::enable(ErrorString*, const Maybe<int>& totalBufferSize, const Maybe<int>& resourceBufferSize)
-{
- enable(totalBufferSize.fromMaybe(maximumTotalBufferSize), resourceBufferSize.fromMaybe(maximumResourceBufferSize));
-}
-
-void InspectorResourceAgent::enable(int totalBufferSize, int resourceBufferSize)
+void InspectorResourceAgent::enable(ErrorString*)
+{
+ enable();
+}
+
+void InspectorResourceAgent::enable()
{
if (!frontend())
return;
- m_resourcesData->setResourcesDataSizeLimits(totalBufferSize, resourceBufferSize);
m_state->setBoolean(ResourceAgentState::resourceAgentEnabled, true);
- m_state->setNumber(ResourceAgentState::totalBufferSize, totalBufferSize);
- m_state->setNumber(ResourceAgentState::resourceBufferSize, resourceBufferSize);
m_instrumentingAgents->setInspectorResourceAgent(this);
}
@@ -1121,7 +1105,7 @@
InspectorResourceAgent::InspectorResourceAgent(InspectedFrames* inspectedFrames)
: InspectorBaseAgent<InspectorResourceAgent, protocol::Frontend::Network>("Network")
, m_inspectedFrames(inspectedFrames)
- , m_resourcesData(NetworkResourcesData::create(maximumTotalBufferSize, maximumResourceBufferSize))
+ , m_resourcesData(NetworkResourcesData::create())
, m_pendingRequest(nullptr)
, m_isRecalculatingStyle(false)
, m_removeFinishedReplayXHRTimer(this, &InspectorResourceAgent::removeFinishedReplayXHRFired)

Powered by Google App Engine
This is Rietveld 408576698