New issue
Advanced search Search tips

Issue 816716 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 16
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Task



Sign in to add a comment

Update //storage/browser/blob/README.md after mojofication.

Project Member Reported by mek@chromium.org, Feb 27 2018

Issue description

The README still describes the old pre-mojo, pre-onionsoup codepath, I should update it to mention what changed with mojo, and how things work now.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jan 16

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d3f9092b12d8e35707b617c2a0330645338ee3a9

commit d3f9092b12d8e35707b617c2a0330645338ee3a9
Author: Marijn Kruisselbrink <mek@chromium.org>
Date: Wed Jan 16 01:19:13 2019

[Blobs] Update README.md to match the new mojofied world better.

Last year I rewrote a lot of the blob system while converting from IPC
to Mojo, but never updated this readme file. This should at least make
it less out of date, even though there are probably still gaps.

Bug:  816716 
Change-Id: Ibdb30a1b4c464227ed46319585eab20d7a516e7c
Reviewed-on: https://chromium-review.googlesource.com/c/1410503
Commit-Queue: Marijn Kruisselbrink <mek@chromium.org>
Reviewed-by: Daniel Murphy <dmurph@chromium.org>
Cr-Commit-Position: refs/heads/master@{#622964}
[modify] https://crrev.com/d3f9092b12d8e35707b617c2a0330645338ee3a9/storage/browser/blob/README.md

Status: Fixed (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, Jan 16 (6 days ago)

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/16d9472b4dd54604a58de04760b1930fa9679d23

commit 16d9472b4dd54604a58de04760b1930fa9679d23
Author: Darwin Huang <huangdarwin@chromium.org>
Date: Wed Jan 16 18:15:23 2019

Blobs: Small spellfixes for readme.

- I saw Tricium recommend simple spellfixes on https://crrev.com/c/1410503 so
just did them real quick.

Bug:  816716 
Change-Id: I206a2f8300e1054fa03b19c37b587454bf125515
Reviewed-on: https://chromium-review.googlesource.com/c/1413871
Reviewed-by: Marijn Kruisselbrink <mek@chromium.org>
Commit-Queue: Marijn Kruisselbrink <mek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#623296}
[modify] https://crrev.com/16d9472b4dd54604a58de04760b1930fa9679d23/storage/browser/blob/README.md

Sign in to add a comment