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

Unified Diff: chrome/browser/chromeos/file_manager/app_installer.cc

Issue 391873003: Files.app: random code clean up. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/file_manager/app_installer.cc
diff --git a/chrome/browser/chromeos/file_manager/app_installer.cc b/chrome/browser/chromeos/file_manager/app_installer.cc
index 65265a4ca7ef98c16b72c3b97bef7a2a3c697268..dc1fd4a8e0d8fc77ed73455052ededfaabd5e65b 100644
--- a/chrome/browser/chromeos/file_manager/app_installer.cc
+++ b/chrome/browser/chromeos/file_manager/app_installer.cc
@@ -5,8 +5,7 @@
#include "chrome/browser/chromeos/file_manager/app_installer.h"
#include "content/public/browser/web_contents.h"
-
-class Profile;
+#include "content/public/browser/web_contents_observer.h"
namespace file_manager {
@@ -14,13 +13,9 @@ namespace {
const char kWebContentsDestroyedError[] = "WebContents is destroyed.";
} // namespace
-class AppInstaller::WebContentsObserver
- : public content::WebContentsObserver {
-
+class AppInstaller::WebContentsObserver : public content::WebContentsObserver {
public:
- explicit WebContentsObserver(
- content::WebContents* web_contents,
- AppInstaller* parent)
+ WebContentsObserver(content::WebContents* web_contents, AppInstaller* parent)
: content::WebContentsObserver(web_contents),
parent_(parent) {
}
@@ -37,15 +32,11 @@ class AppInstaller::WebContentsObserver
DISALLOW_IMPLICIT_CONSTRUCTORS(WebContentsObserver);
};
-AppInstaller::AppInstaller(
- content::WebContents* web_contents,
- const std::string& webstore_item_id,
- Profile* profile,
- const Callback& callback)
- : extensions::WebstoreStandaloneInstaller(
- webstore_item_id,
- profile,
- callback),
+AppInstaller::AppInstaller(content::WebContents* web_contents,
+ const std::string& item_id,
+ Profile* profile,
+ const Callback& callback)
+ : extensions::WebstoreStandaloneInstaller(item_id, profile, callback),
callback_(callback),
web_contents_(web_contents),
web_contents_observer_(new WebContentsObserver(web_contents, this)) {
« no previous file with comments | « chrome/browser/chromeos/file_manager/app_installer.h ('k') | chrome/browser/chromeos/file_manager/file_browser_handlers.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698