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

Issue 2726713002: Allow creation of CRWSessionStorage outside of serialisation API. (Closed)

Created:
3 years, 9 months ago by sdefresne
Modified:
3 years, 9 months ago
Reviewers:
kkhorimoto
CC:
chromium-reviews, marq+watch_chromium.org, Eugene But (OOO till 7-30), pkl (ping after 24h if needed), noyau+watch_chromium.org, sdefresne+watch_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Allow creation of CRWSessionStorage outside of serialisation API. For recreating fake session during unit tests, allow creation of CRWSessionStorage instance outside of the serialization code (this required supporting null in AddSerializableUserData method). Fix typos in serializable_user_data_manager_impl.mm, convert the code to use base::MakeUnique<> and to pass std::unique_ptr<> to base::SupportsUserData::SetUserData method. BUG=None Review-Url: https://codereview.chromium.org/2726713002 Cr-Commit-Position: refs/heads/master@{#454180} Committed: https://chromium.googlesource.com/chromium/src/+/32a45a277461ac61b4556e773cb3a16d4e213fa9

Patch Set 1 #

Patch Set 2 : Rebase. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+22 lines, -22 lines) Patch
M ios/chrome/browser/tabs/tab_model_unittest.mm View 2 chunks +2 lines, -1 line 0 comments Download
M ios/web/navigation/serializable_user_data_manager_impl.mm View 5 chunks +20 lines, -21 lines 0 comments Download

Depends on Patchset:

Dependent Patchsets:

Messages

Total messages: 17 (13 generated)
sdefresne
Please take a look.
3 years, 9 months ago (2017-02-28 23:12:25 UTC) #3
kkhorimoto
lgtm
3 years, 9 months ago (2017-02-28 23:53:22 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2726713002/20001
3 years, 9 months ago (2017-03-02 03:27:51 UTC) #14
commit-bot: I haz the power
3 years, 9 months ago (2017-03-02 04:39:17 UTC) #17
Message was sent while issue was closed.
Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/32a45a277461ac61b4556e773cb3...

Powered by Google App Engine
This is Rietveld 408576698