PlzNavigate: Chrome UI changes for new methods of WebContentsObserver

extensions/browser/extension_navigation_throttle.cc

 // Copyright 2016 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 "extensions/browser/extension_navigation_throttle.h"
 
 #include "components/guest_view/browser/guest_view_base.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/navigation_handle.h"
 #include "content/public/browser/render_frame_host.h"
@@ skipping 72 matching lines @@
         std::string resource_path = url.path();
         bool is_guest = WebViewGuest::GetGuestPartitionConfigForSite(
             navigation_handle()->GetStartingSiteInstance()->GetSiteURL(),
             &partition_domain, &partition_id, &in_memory);
 
         bool allowed = true;
         url_request_util::AllowCrossRendererResourceLoadHelper(
             is_guest, extension, owner_extension, partition_id, resource_path,
             navigation_handle()->GetPageTransition(), &allowed);
         if (!allowed)
-          return content::NavigationThrottle::CANCEL;
+          return content::NavigationThrottle::BLOCK_REQUEST;

[boliu, 2017/02/08 00:10:38]

is this change relevant to this CL? android doesn't support extensions afaik?

[shaktisahu, 2017/02/08 00:51:09]

I will remove this before commit. This is a bug fix from
https://codereview.chromium.org/2657813004/ which was required to pass the CQ
dry run.

       }
     }
 
     return content::NavigationThrottle::PROCEED;
   }
 
   // Now enforce web_accessible_resources for navigations. Top-level navigations
   // should always be allowed.
 
   // If the navigation is not to a chrome-extension:// URL, no need to perform
@@ skipping 50 matching lines @@
 
   if (WebAccessibleResourcesInfo::IsResourceWebAccessible(extension,
                                                           resource_path)) {
     return content::NavigationThrottle::PROCEED;
   }
 
   return content::NavigationThrottle::BLOCK_REQUEST;
 }
 
 }  // namespace extensions