Move Android to use new Popup Blocker API.

chrome/chrome_browser_ui.gypi

 # Copyright 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.
 
 {
   'targets': [
     {
       'target_name': 'browser_ui',
       'type': 'static_library',
       'variables': { 'enable_wexit_time_destructors': 1, },
@@ skipping 2797 matching lines @@
             'browser/ui/webui/ntp/new_tab_page_sync_handler.cc',
             'browser/ui/webui/ntp/new_tab_page_sync_handler.h',
             'browser/ui/webui/ntp/ntp_login_handler.cc',
             'browser/ui/webui/ntp/ntp_login_handler.h',
             'browser/ui/webui/sync_setup_handler.cc',
             'browser/ui/webui/sync_setup_handler.h',
 
             # There's no Browser/BrowserList on Android.
             'browser/ui/active_tab_tracker.cc',
             'browser/ui/active_tab_tracker.h',
-            'browser/ui/blocked_content/popup_blocker_tab_helper.cc',
-            'browser/ui/blocked_content/popup_blocker_tab_helper.h',
             'browser/ui/bookmarks/bookmark_bubble_sign_in_delegate.cc',
             'browser/ui/bookmarks/bookmark_bubble_sign_in_delegate.h',
             'browser/ui/bookmarks/bookmark_context_menu_controller.cc',
             'browser/ui/bookmarks/bookmark_prompt_controller.cc',
             'browser/ui/browser.cc',
             'browser/ui/browser_command_controller.cc',
             'browser/ui/browser_finder.cc',
             'browser/ui/browser_iterator.cc',
             'browser/ui/browser_list.cc',
-            'browser/ui/browser_navigator.cc',

[Yaron, 2013/08/21 19:07:58]

I think this is left over from your temporary work. You aren't using it now and
I'd prefer to keep browser_* out of android for consistency. Oh actually, I see
that you're using NavigatePrams from there. Hmm. I think we should probably
split NavigateParams that's profile-dependent out from the browser-dependent
variant or remove the Browser dep from it.

[David Trainor- moved to gerrit, 2013/08/21 20:21:27]

I agree with what you're saying.  I think that's outside the scope of this
particular CL but it should be done.  I can file a bug and track that/throw it
on the pile of refactors I want to do :).

On 2013/08/21 19:07:58, Yaron wrote:

             'browser/ui/browser_otr_state.cc',
             'browser/ui/browser_tab_contents.cc',
             'browser/ui/browser_tab_restore_service_delegate.cc',
             'browser/ui/browser_tabstrip.cc',
             'browser/ui/browser_tabstrip.h',
             'browser/ui/chrome_pages.cc',
             'browser/ui/chrome_pages.h',
             'browser/ui/extensions/extension_install_ui_default.cc',
             'browser/ui/extensions/extension_install_ui_default.h',
             'browser/ui/ntp_background_util.cc',
@@ skipping 271 matching lines @@
             'browser/ui/webui/local_discovery/local_discovery_ui.cc',
             'browser/ui/webui/local_discovery/local_discovery_ui.h',
             'browser/ui/webui/local_discovery/local_discovery_ui_handler.cc',
             'browser/ui/webui/local_discovery/local_discovery_ui_handler.h',
           ],
         }],
       ],
     },
   ],
 }