Give the ChromeVox Panel an extension view type.

extensions/browser/process_manager.cc

 // Copyright 2013 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/process_manager.h"
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/logging.h"
@@ skipping 150 matching lines @@
   ExtensionRenderFrameData()
       : view_type(VIEW_TYPE_INVALID), has_keepalive(false) {}
 
   // Returns whether the view can keep the lazy background page alive or not.
   bool CanKeepalive() const {
     switch (view_type) {
       case VIEW_TYPE_APP_WINDOW:
       case VIEW_TYPE_BACKGROUND_CONTENTS:
       case VIEW_TYPE_EXTENSION_DIALOG:
       case VIEW_TYPE_EXTENSION_GUEST:
+      case VIEW_TYPE_EXTENSION_OTHER:

[Devlin, 2016/01/04 18:29:14]

Having a pseudo-dummy tag have keepalive makes me a bit nervous.  If ChromeVox
doesn't need this, I'd rather default to it not.

[dmazzoni, 2016/01/04 18:42:13]

ChromeVox doesn't need this, so moved below.

       case VIEW_TYPE_EXTENSION_POPUP:
       case VIEW_TYPE_LAUNCHER_PAGE:
       case VIEW_TYPE_PANEL:
       case VIEW_TYPE_TAB_CONTENTS:
       case VIEW_TYPE_VIRTUAL_KEYBOARD:
         return true;
 
       case VIEW_TYPE_INVALID:
       case VIEW_TYPE_EXTENSION_BACKGROUND_PAGE:
         return false;
@@ skipping 796 matching lines @@
   if (extension && !IncognitoInfo::IsSplitMode(extension)) {
     BrowserContext* original_context =
         ExtensionsBrowserClient::Get()->GetOriginalContext(browser_context());
     return ProcessManager::Get(original_context)->GetSiteInstanceForURL(url);
   }
 
   return ProcessManager::GetSiteInstanceForURL(url);
 }
 
 }  // namespace extensions