Fix warnings when compiling with clang

android_webview/lib/aw_browser_dependency_factory_impl.cc

 // Copyright (c) 2012 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 "android_webview/lib/aw_browser_dependency_factory_impl.h"
 
 // TODO(joth): Componentize or remove chrome/... dependencies.
 #include "android_webview/browser/net/aw_network_delegate.h"
 #include "android_webview/native/aw_contents_container.h"
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/lazy_instance.h"
 #include "base/memory/ref_counted.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/ui/app_modal_dialogs/javascript_dialog_creator.h"

[joth, 2012/09/24 23:20:02]

can this include be removed now?

[michaelbai, 2012/09/24 23:25:55]

Done.

 #include "content/public/browser/web_contents.h"
 #include "ipc/ipc_message.h"
 #include "net/url_request/url_request_context.h"
 #include "net/url_request/url_request_context_getter.h"
 
 using content::BrowserContext;
 using content::WebContents;
 
 namespace android_webview {
 
@@ skipping 67 matching lines @@
 WebContents* AwBrowserDependencyFactoryImpl::CreateWebContents(bool incognito) {
   return content::WebContents::Create(
       GetBrowserContext(incognito), 0, MSG_ROUTING_NONE, 0);
 }
 
 AwContentsContainer* AwBrowserDependencyFactoryImpl::CreateContentsContainer(
     content::WebContents* contents) {
   return new WebContentsWrapper(contents);
 }
 
-content::JavaScriptDialogCreator*
-    AwBrowserDependencyFactoryImpl::GetJavaScriptDialogCreator() {
-  return GetJavaScriptDialogCreatorInstance();
-}
-
 }  // namespace android_webview