Chromium Code Reviews| Index: extensions/browser/api/async_api_function.cc |
| diff --git a/chrome/browser/extensions/api/api_function.cc b/extensions/browser/api/async_api_function.cc |
| similarity index 59% |
| rename from chrome/browser/extensions/api/api_function.cc |
| rename to extensions/browser/api/async_api_function.cc |
| index 54bf8e32dee53951afd51a8c8fe10beef78065f8..2ed484dc2fc0bfe5af69dc81e53e75d71132a7e9 100644 |
| --- a/chrome/browser/extensions/api/api_function.cc |
| +++ b/extensions/browser/api/async_api_function.cc |
| @@ -1,30 +1,20 @@ |
| -// Copyright (c) 2012 The Chromium Authors. All rights reserved. |
| +// Copyright 2014 The Chromium Authors. All rights reserved. |
|
James Cook
2014/02/26 17:57:21
ditto
Ken Rockot(use gerrit already)
2014/02/26 18:20:44
Done.
|
| // Use of this source code is governed by a BSD-style license that can be |
| // found in the LICENSE file. |
| -#include "chrome/browser/extensions/api/api_function.h" |
| +#include "extensions/browser/api/async_api_function.h" |
| #include "base/bind.h" |
| -#include "chrome/browser/profiles/profile.h" |
| #include "extensions/browser/extension_system.h" |
| using content::BrowserThread; |
| namespace extensions { |
| -ApiFunction::ApiFunction() { |
| -} |
| - |
| -ApiFunction::~ApiFunction() { |
| -} |
| - |
| // AsyncApiFunction |
| -AsyncApiFunction::AsyncApiFunction() |
| - : work_thread_id_(BrowserThread::IO) { |
| -} |
| +AsyncApiFunction::AsyncApiFunction() : work_thread_id_(BrowserThread::IO) {} |
| -AsyncApiFunction::~AsyncApiFunction() { |
| -} |
| +AsyncApiFunction::~AsyncApiFunction() {} |
| bool AsyncApiFunction::RunImpl() { |
| DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
| @@ -33,18 +23,16 @@ bool AsyncApiFunction::RunImpl() { |
| return false; |
| } |
| bool rv = BrowserThread::PostTask( |
| - work_thread_id_, FROM_HERE, |
| + work_thread_id_, |
| + FROM_HERE, |
| base::Bind(&AsyncApiFunction::WorkOnWorkThread, this)); |
| DCHECK(rv); |
| return true; |
| } |
| -bool AsyncApiFunction::PrePrepare() { |
| - return true; |
| -} |
| +bool AsyncApiFunction::PrePrepare() { return true; } |
| -void AsyncApiFunction::Work() { |
| -} |
| +void AsyncApiFunction::Work() {} |
| void AsyncApiFunction::AsyncWorkStart() { |
| Work(); |
| @@ -54,7 +42,8 @@ void AsyncApiFunction::AsyncWorkStart() { |
| void AsyncApiFunction::AsyncWorkCompleted() { |
| if (!BrowserThread::CurrentlyOn(BrowserThread::UI)) { |
| bool rv = BrowserThread::PostTask( |
| - BrowserThread::UI, FROM_HERE, |
| + BrowserThread::UI, |
| + FROM_HERE, |
| base::Bind(&AsyncApiFunction::RespondOnUIThread, this)); |
| DCHECK(rv); |
| } else { |
| @@ -64,10 +53,10 @@ void AsyncApiFunction::AsyncWorkCompleted() { |
| void AsyncApiFunction::WorkOnWorkThread() { |
| DCHECK(BrowserThread::CurrentlyOn(work_thread_id_)); |
| - DLOG_IF(ERROR, (work_thread_id_ == BrowserThread::UI)) << |
| - "You have specified that AsyncApiFunction::Work() should happen on " |
| - "the UI thread. This nullifies the point of this class. Either " |
| - "specify a different thread or derive from a different class."; |
| + DLOG_IF(ERROR, (work_thread_id_ == BrowserThread::UI)) |
| + << "You have specified that AsyncApiFunction::Work() should happen on " |
| + "the UI thread. This nullifies the point of this class. Either " |
| + "specify a different thread or derive from a different class."; |
| AsyncWorkStart(); |
| } |