Aura on Android: Compile extensions gn target.

chrome/browser/ui/browser_navigator_params.h

 // Copyright 2015 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.
 
 #ifndef CHROME_BROWSER_UI_BROWSER_NAVIGATOR_PARAMS_H_
 #define CHROME_BROWSER_UI_BROWSER_NAVIGATOR_PARAMS_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 37 matching lines @@
 // chrome::NavigateParams params(browser, popup_contents);
 // params.source_contents = source_contents;
 // chrome::Navigate(&params);
 //
 // See browser_navigator_browsertest.cc for more examples.
 
 // TODO(thestig): Split or ifdef out more fields that are not used on Android.
 struct NavigateParams {
 #if defined(OS_ANDROID)
   explicit NavigateParams(content::WebContents* a_target_contents);
-#else
+#endif
+#if !defined(OS_ANDROID) || defined(USE_AURA)

[mfomitchev, 2015/11/10 21:27:28]

I don't see any changes to browser_navigator_params.h/cc in auraclank2. Do you
have any idea why we didn't need them in the branch, but we need them now? I
don't see any recent changes in chrome_runtime_api_delegate, which could've
affect this.

[Hadi, 2015/11/12 16:33:36]

I spent some time on this and couldn't figure out why we didn't need this in
auraclank2.

I think somehow chrome_runtime_api_delegate.cc and extension_tab_util.cc don't
get compiled there, but I couldn't figure out what we did it to disable them.

   NavigateParams(Browser* browser,
                  const GURL& a_url,
                  ui::PageTransition a_transition);
   NavigateParams(Browser* browser,
                  content::WebContents* a_target_contents);
 #endif
   NavigateParams(Profile* profile,
                  const GURL& a_url,
                  ui::PageTransition a_transition);
   ~NavigateParams();
@@ skipping 129 matching lines @@
   // What to do with the ref component of the URL for singleton navigations.
   enum RefBehavior {
     // Two URLs with differing refs are same.
     IGNORE_REF,
     // Two URLs with differing refs are different.
     RESPECT_REF,
   };
   // Default is IGNORE.
   RefBehavior ref_behavior;
 
-#if !defined(OS_ANDROID)
+#if !defined(OS_ANDROID) || defined(USE_AURA)
   // [in]  Specifies a Browser object where the navigation could occur or the
   //       tab could be added. Navigate() is not obliged to use this Browser if
   //       it is not compatible with the operation being performed. This can be
   //       NULL, in which case |initiating_profile| must be provided.
   // [out] Specifies the Browser object where the navigation occurred or the
   //       tab was added. Guaranteed non-NULL unless the disposition did not
   //       require a navigation, in which case this is set to NULL
   //       (SUPPRESS_OPEN, SAVE_TO_DISK, IGNORE_ACTION).
   // Note: If |show_window| is set to false and a new Browser is created by
   //       Navigate(), the caller is responsible for showing it so that its
@@ skipping 33 matching lines @@
   NavigateParams();
 };
 
 // Copies fields from |params| struct to |nav_params| struct.
 void FillNavigateParamsFromOpenURLParams(chrome::NavigateParams* nav_params,
                                          const content::OpenURLParams& params);
 
 }  // namespace chrome
 
 #endif  // CHROME_BROWSER_UI_BROWSER_NAVIGATOR_PARAMS_H_