Navigations from WebUSB notifications are not renderer initiated

chrome/browser/chrome_webusb_browser_client.cc

 // 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.
 
 #include "chrome/browser/chrome_webusb_browser_client.h"
 
 #include <utility>
 
 #include "base/macros.h"
 #include "base/metrics/histogram_macros.h"
@@ skipping 42 matching lines @@
 }
 
 Browser* GetBrowser() {
   chrome::ScopedTabbedBrowserDisplayer browser_displayer(
       ProfileManager::GetActiveUserProfile());
   DCHECK(browser_displayer.browser());
   return browser_displayer.browser();
 }
 
 void OpenURL(const GURL& url) {
-  GetBrowser()->OpenURL(
-      content::OpenURLParams(url, content::Referrer(), NEW_FOREGROUND_TAB,
-                             ui::PAGE_TRANSITION_AUTO_TOPLEVEL, true));
+  GetBrowser()->OpenURL(content::OpenURLParams(
+      url, content::Referrer(), NEW_FOREGROUND_TAB,
+      ui::PAGE_TRANSITION_AUTO_TOPLEVEL, /*is_renderer_initiated=*/false));

[Reilly Grant (use Gerrit), 2016/02/18 18:36:45]

nit: The usual way to annotate this is:

false /* is_renderer_initialized */

[juncai, 2016/02/18 20:17:50]

Done.

 }
 
 // Delegate for webusb notification
 class WebUsbNotificationDelegate : public message_center::NotificationDelegate {
  public:
   WebUsbNotificationDelegate(const GURL& landing_page,
                              const std::string& notification_id)
       : landing_page_(landing_page), notification_id_(notification_id) {}
 
   void Click() override {
@@ skipping 60 matching lines @@
 }
 
 void ChromeWebUsbBrowserClient::OnDeviceRemoved(
     const std::string& notification_id) {
   message_center::MessageCenter* message_center =
       message_center::MessageCenter::Get();
   if (message_center->FindVisibleNotificationById(notification_id)) {
     message_center->RemoveNotification(notification_id, false /* by_user */);
   }
 }