diff options
author | pvalenzuela <pvalenzuela@chromium.org> | 2015-06-26 11:38:59 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-06-26 18:39:30 +0000 |
commit | 4f4dc3834e1aa811300703074f824685d2cdeabc (patch) | |
tree | 30b55c2c21696f6bbe29175956400f0921ca41f6 /chromeos/chromeos_switches.cc | |
parent | ac53d3da52014db50e638e96fd9f52817072fd64 (diff) | |
download | chromium_src-4f4dc3834e1aa811300703074f824685d2cdeabc.zip chromium_src-4f4dc3834e1aa811300703074f824685d2cdeabc.tar.gz chromium_src-4f4dc3834e1aa811300703074f824685d2cdeabc.tar.bz2 |
Sync: store FakeServer as WeakPtr
This CL changes the FakeServerNetworkResources stack to store FakeServer
as a WeakPtr. This is done because FakeServerHttpPostProvider posts a
task to another thread and FakeServer may be deleted before the task
executes.
This CL adds a ThreadChecker to FakeServer to ensure that all actions
(especially AsWeakPtr() and destruction) happen on the same thread. As
a result of this thread restriction, FakeServerHelper has been changed
to call all FakeServer methods on the UI thread.
FakeServerHttpPostProvider's task-posting has also been reconfigured to
be cleaner. This class now calls PostTask (instead of unnecessarily
calling PostNonNestableTask previously) and has removed the possibility
of FakeServer::HandleCommand (UI thread) calling a method on
FakeServerHttpPostProvider (Sync thread).
BUG=481192
Review URL: https://codereview.chromium.org/1149723007
Cr-Commit-Position: refs/heads/master@{#336422}
Diffstat (limited to 'chromeos/chromeos_switches.cc')
0 files changed, 0 insertions, 0 deletions