Remove dependency of chrome in GuestViewBase.

chrome/browser/extensions/api/chrome_extensions_api_client.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // 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/chrome_extensions_api_client.h"
 
 #include "base/files/file_path.h"
 #include "chrome/browser/extensions/api/storage/sync_value_store_cache.h"
-#include "chrome/browser/guest_view/app_view/app_view_guest.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/browser_thread.h"
 #include "device/hid/hid_service.h"
 
 #if defined(ENABLE_CONFIGURATION_POLICY)
 #include "chrome/browser/extensions/api/storage/managed_value_store_cache.h"
 #endif
 
+#if defined(ENABLE_EXTENSIONS)
+#include "chrome/browser/guest_view/app_view/app_view_guest.h"
+#include "chrome/browser/guest_view/extension_options/extension_options_guest.h"
+#include "chrome/browser/guest_view/web_view/web_view_guest.h"
+#endif
+
 namespace extensions {
 
 ChromeExtensionsAPIClient::ChromeExtensionsAPIClient() {}
 
 ChromeExtensionsAPIClient::~ChromeExtensionsAPIClient() {}
 
 void ChromeExtensionsAPIClient::AddAdditionalValueStoreCaches(
     content::BrowserContext* context,
     const scoped_refptr<SettingsStorageFactory>& factory,
     const scoped_refptr<ObserverListThreadSafe<SettingsObserver> >& observers,
@@ skipping 32 matching lines @@
 
 device::HidService* ChromeExtensionsAPIClient::GetHidService() {
   if (!hid_service_) {
     hid_service_.reset(device::HidService::Create(
         content::BrowserThread::GetMessageLoopProxyForThread(
             content::BrowserThread::UI)));
   }
   return hid_service_.get();
 }
 
+void ChromeExtensionsAPIClient::RegisterGuestViewType() {
+  AppViewGuest::Register();

[Ken Rockot(use gerrit already), 2014/08/07 22:37:40]

Please also wrap these Register() calls in an #if defined(ENABLE_EXTENSIONS) as
well

[Xi Han, 2014/08/08 14:12:11]

Wait a second. It is safe to add #if defined(ENABLE_EXTENSIONS), but I begin to
think about whether it is necessary in this class (also the #if defined that I
added in the "includes"). This class is called ChromeExtensionsAPIClient, can we
assume that extensions is enable here?

+  ExtensionOptionsGuest::Register();
+  WebViewGuest::Register();
+}
+
 }  // namespace extensions