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

Issue 941363003: InlinedVisitor: Migrate filesystem to use inlined tracing (Closed)

Created:
5 years, 10 months ago by kouhei (in TOK)
Modified:
5 years, 10 months ago
CC:
blink-reviews, kinuko+fileapi, nhiroki, tzik
Target Ref:
refs/heads/master
Project:
blink
Visibility:
Public.

Description

InlinedVisitor: Migrate filesystem to use inlined tracing Updates trace methods as of https://groups.google.com/a/chromium.org/d/msg/blink-dev/oi4Snt9jNx8/XcES55c7QaQJ R=oilpan-reviews BUG=455524 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=190722

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+50 lines, -50 lines) Patch
M Source/modules/filesystem/DOMFileSystem.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/DOMFileSystem.cpp View 2 chunks +2 lines, -2 lines 0 comments Download
M Source/modules/filesystem/DOMFileSystemBase.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/DOMFileSystemSync.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/DirectoryEntry.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/DirectoryEntry.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/DirectoryEntrySync.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/DirectoryEntrySync.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/DirectoryReader.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/DirectoryReader.cpp View 3 chunks +3 lines, -3 lines 0 comments Download
M Source/modules/filesystem/DirectoryReaderBase.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/DirectoryReaderSync.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/DirectoryReaderSync.cpp View 3 chunks +3 lines, -3 lines 0 comments Download
M Source/modules/filesystem/DraggedIsolatedFileSystem.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/DraggedIsolatedFileSystem.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/EntriesCallback.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/Entry.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/Entry.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/EntryBase.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/EntryBase.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/EntryCallback.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/EntrySync.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/EntrySync.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/ErrorCallback.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/FileCallback.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/FileEntry.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/FileEntry.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/FileEntrySync.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/FileEntrySync.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/FileSystemCallback.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/FileWriter.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/FileWriter.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/FileWriterBase.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/FileWriterBaseCallback.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/FileWriterCallback.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/FileWriterSync.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/FileWriterSync.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/InspectorFileSystemAgent.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/InspectorFileSystemAgent.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/LocalFileSystem.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/LocalFileSystem.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/Metadata.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/MetadataCallback.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/filesystem/SyncCallbackHelper.h View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 9 (3 generated)
kouhei (in TOK)
5 years, 10 months ago (2015-02-24 06:56:19 UTC) #2
tkent
lgtm
5 years, 10 months ago (2015-02-24 07:38:55 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/941363003/1
5 years, 10 months ago (2015-02-24 07:39:50 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1) as https://src.chromium.org/viewvc/blink?view=rev&revision=190722
5 years, 10 months ago (2015-02-24 08:23:17 UTC) #7
haraken
LGTM
5 years, 10 months ago (2015-02-24 08:33:57 UTC) #8
kinuko
5 years, 10 months ago (2015-02-24 08:47:59 UTC) #9
Message was sent while issue was closed.
rs/lgtm, thanks

Powered by Google App Engine
This is Rietveld 408576698