Popup windows launched in a new window in ChromeFrame on IE8 would open behin...

chrome_frame/chrome_active_document.cc

 // Copyright (c) 2010 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.
 
 // Implementation of ChromeActiveDocument
 #include "chrome_frame/chrome_active_document.h"
 
 #include <hlink.h>
 #include <htiface.h>
 #include <initguid.h>
@@ skipping 124 matching lines @@
                                           LPMSG msg,
                                           IOleClientSite* active_site,
                                           LONG index,
                                           HWND parent_window,
                                           LPCRECT pos) {
   // IE will try and in-place activate us in some cases. This happens when
   // the user opens a new IE window with a URL that has us as the DocObject.
   // Here we refuse to be activated in-place and we will force IE to UIActivate
   // us.
   if (OLEIVERB_INPLACEACTIVATE == verb)
-    return E_NOTIMPL;
+    return OLEOBJ_E_INVALIDVERB;
   // Check if we should activate as a docobject or not
   // (client supports IOleDocumentSite)
   if (doc_site_) {
     switch (verb) {
     case OLEIVERB_SHOW: {
       ScopedComPtr<IDocHostUIHandler> doc_host_handler;
       doc_host_handler.QueryFrom(doc_site_);
       if (doc_host_handler.get())
         doc_host_handler->ShowUI(DOCHOSTUITYPE_BROWSE, this, this, NULL, NULL);
     }
@@ skipping 1176 matching lines @@
 
   if (new_navigation_info.url != navigation_info_.url)
     return true;
 
   if (new_navigation_info.referrer != navigation_info_.referrer)
     return true;
 
   return false;
 }