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

Issue 25447002: Move JSONValues.cpp/.h to Source/platform/ (Closed)

Created:
7 years, 2 months ago by pilgrim_google
Modified:
7 years, 2 months ago
Reviewers:
abarth-chromium
CC:
blink-reviews, apavlov+blink_chromium.org, adamk+blink_chromium.org, aandrey+blink_chromium.org, do-not-use, Nils Barth (inactive), caseq+blink_chromium.org, Nate Chapin, yurys+blink_chromium.org, abarth-chromium, marja+watch_chromium.org, dglazkov+blink, mkwst+watchlist_chromium.org, devtools-reviews_chromium.org, loislo+blink_chromium.org, lushnikov+blink_chromium.org, eustas+blink_chromium.org, paulirish+reviews_chromium.org, haraken, kojih, jsbell+bindings_chromium.org, alph+blink_chromium.org, vsevik+blink_chromium.org, pfeldman+blink_chromium.org, jeez, darin (slow to review)
Visibility:
Public.

Description

Move JSONValues.cpp/.h to Source/platform/ BUG=297477 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=158607

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+51 lines, -829 lines) Patch
M Source/bindings/v8/ScriptCallStackFactory.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/bindings/v8/ScriptHeapSnapshot.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/bindings/v8/ScriptValue.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/bindings/v8/custom/V8InjectedScriptHostCustom.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/core.gypi View 1 chunk +0 lines, -2 lines 0 comments Download
M Source/core/html/parser/XSSAuditor.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/html/parser/XSSAuditorDelegate.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/inspector/CodeGeneratorInspectorStrings.py View 4 chunks +4 lines, -4 lines 0 comments Download
M Source/core/inspector/InjectedScript.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/inspector/InjectedScriptBase.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/inspector/InjectedScriptHost.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/inspector/InjectedScriptManager.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/inspector/InspectorAgent.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/inspector/InspectorCSSAgent.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/inspector/InspectorCSSAgent.cpp View 2 chunks +1 line, -1 line 0 comments Download
M Source/core/inspector/InspectorDOMAgent.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/inspector/InspectorDOMDebuggerAgent.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/inspector/InspectorDOMStorageAgent.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/inspector/InspectorDatabaseAgent.cpp View 2 chunks +1 line, -1 line 0 comments Download
M Source/core/inspector/InspectorDebuggerAgent.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/inspector/InspectorFrontendHost.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/inspector/InspectorIndexedDBAgent.cpp View 2 chunks +1 line, -1 line 0 comments Download
M Source/core/inspector/InspectorInputAgent.cpp View 2 chunks +1 line, -1 line 0 comments Download
M Source/core/inspector/InspectorOverlay.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/inspector/InspectorPageAgent.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/inspector/InspectorResourceAgent.cpp View 2 chunks +1 line, -1 line 0 comments Download
M Source/core/inspector/InspectorRuntimeAgent.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/inspector/InspectorState.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/inspector/InspectorStyleSheet.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/inspector/InspectorStyleSheet.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/inspector/InspectorTimelineAgent.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/inspector/InspectorWorkerAgent.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/inspector/JSONParser.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/inspector/TimelineRecordFactory.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/inspector/TimelineRecordFactory.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/inspector/TimelineTraceEventProcessor.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/page/ContentSecurityPolicy.cpp View 1 chunk +1 line, -1 line 0 comments Download
D Source/core/platform/JSONValues.h View 1 chunk +0 lines, -393 lines 0 comments Download
D Source/core/platform/JSONValues.cpp View 1 chunk +0 lines, -386 lines 0 comments Download
A + Source/platform/JSONValues.h View 8 chunks +8 lines, -7 lines 0 comments Download
A + Source/platform/JSONValues.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/platform/blink_platform.gypi View 1 chunk +2 lines, -0 lines 0 comments Download
M Source/web/WebDevToolsAgentImpl.cpp View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
pilgrim_google
7 years, 2 months ago (2013-10-01 04:17:41 UTC) #1
abarth-chromium
lgtm
7 years, 2 months ago (2013-10-01 04:26:27 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/pilgrim@chromium.org/25447002/1
7 years, 2 months ago (2013-10-01 04:26:37 UTC) #3
commit-bot: I haz the power
7 years, 2 months ago (2013-10-01 08:07:46 UTC) #4
Message was sent while issue was closed.
Change committed as 158607

Powered by Google App Engine
This is Rietveld 408576698