Skip to content

Commit

Permalink
Mark DOMStorageHost as non-copyable
Browse files Browse the repository at this point in the history
It contains a std::map that would be expensive to copy, and the default
copy/assignment members would be generated inline, bloating the binary
size.

Add GitHub to AUTHORS

The CLA was signed back in June.

R=mal@chromium.org, marja@chromium.org, michaeln@chromium.org
BUG=

Review URL: https://codereview.chromium.org/155933003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@250501 0039d316-1c4b-4281-b951-d872f2087c98
  • Loading branch information
adam@github.com committed Feb 11, 2014
1 parent 3882aae commit 7ee61d3
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 0 deletions.
1 change: 1 addition & 0 deletions AUTHORS
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
Aaron Jacobs <samusaaron3@gmail.com>
Aaron Leventhal <aaronlevbugs@gmail.com>
Aaron Randolph <aaron.randolph@gmail.com>
Adam Roben <adam@github.com>
Adam Treat <adam.treat@samsung.com>
Adenilson Cavalcanti <a.cavalcanti@partner.samsung.com>
Aditya Bhargava <heuristicist@gmail.com>
Expand Down
2 changes: 2 additions & 0 deletions content/browser/dom_storage/dom_storage_host.h
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,8 @@ class CONTENT_EXPORT DOMStorageHost {
scoped_refptr<DOMStorageContextImpl> context_;
AreaMap connections_;
int render_process_id_;

DISALLOW_COPY_AND_ASSIGN(DOMStorageHost);
};

} // namespace content
Expand Down

0 comments on commit 7ee61d3

Please sign in to comment.