Fix the browser test DownloadExtensionTest_FileIcon_Active crash which started occurring after my c…

chrome/browser/download/download_completion_observer_win.cc

 // Copyright (c) 2012 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/download/download_completion_observer_win.h"
 
 #include <string>
 
 #include "base/logging.h"
 #include "base/stl_util.h"
@@ skipping 26 matching lines @@
   }
 }
 
 void DownloadCompletionObserver::ManagerGoingDown(DownloadManager* manager) {
   ClearDownloadItems();
   manager->RemoveObserver(this);
   delete this;
 }
 
 void DownloadCompletionObserver::OnDownloadUpdated(DownloadItem* download) {
-  if (!base::win::IsMetroProcess())
-    return;
-
   switch (download->GetState()) {
     case DownloadItem::COMPLETE: {
-      if (!download->GetOpenWhenComplete() &&
+      if (base::win::IsMetroProcess() &&
+          !download->GetOpenWhenComplete() &&
           !download->ShouldOpenFileBasedOnExtension() &&
           !download->IsTemporary() &&
           !download->GetAutoOpened()) {
         // In Windows 8 metro mode display a metro style notification which
         // informs the user that the download is complete.
         HMODULE metro = base::win::GetMetroModule();
         base::win::MetroNotification display_notification =
             reinterpret_cast< base::win::MetroNotification>(
                 ::GetProcAddress(metro, "DisplayNotification"));
         DCHECK(display_notification);
@@ skipping 36 matching lines @@
   }
 }
 
 void DownloadCompletionObserver::ClearDownloadItems() {
   for (std::set<DownloadItem*>::iterator it = download_items_.begin();
        it != download_items_.end(); ++it) {
     (*it)->RemoveObserver(this);
   }
   download_items_.clear();
 }