Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(106)

Side by Side Diff: chrome/browser/extensions/platform_app_browsertest.cc

Issue 9845003: Pass command line arguments onto platform apps which provide the right intent. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Include files Created 8 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/command_line.h" 5 #include "base/command_line.h"
6 #include "base/stringprintf.h" 6 #include "base/stringprintf.h"
7 #include "base/utf_string_conversions.h" 7 #include "base/utf_string_conversions.h"
8 #include "chrome/browser/extensions/extension_apitest.h" 8 #include "chrome/browser/extensions/extension_apitest.h"
9 #include "chrome/browser/extensions/extension_browsertest.h" 9 #include "chrome/browser/extensions/extension_browsertest.h"
10 #include "chrome/browser/extensions/extension_host.h" 10 #include "chrome/browser/extensions/extension_host.h"
(...skipping 178 matching lines...) Expand 10 before | Expand all | Expand 10 after
189 #if defined(USE_AURA) 189 #if defined(USE_AURA)
190 // On Aura, this currently fails because the window width is returned as 256 190 // On Aura, this currently fails because the window width is returned as 256
191 // instead of 250. See http://crbug.com/119410. 191 // instead of 250. See http://crbug.com/119410.
192 #define MAYBE_WindowsApi FAILS_WindowsApi 192 #define MAYBE_WindowsApi FAILS_WindowsApi
193 #else 193 #else
194 #define MAYBE_WindowsApi WindowsApi 194 #define MAYBE_WindowsApi WindowsApi
195 #endif 195 #endif
196 IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, MAYBE_WindowsApi) { 196 IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, MAYBE_WindowsApi) {
197 ASSERT_TRUE(RunPlatformAppTest("platform_apps/windows_api")) << message_; 197 ASSERT_TRUE(RunPlatformAppTest("platform_apps/windows_api")) << message_;
198 } 198 }
199
200 // Tests that command line parameters get passed through to platform apps
201 // via window.intent correctly, and that the file names passed through can be
202 // read successfully.
203 IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, DocumentEditor) {
204 CommandLine* command_line = CommandLine::ForCurrentProcess();
205 FilePath test_doc(test_data_dir_.AppendASCII(
206 "platform_apps/document_editor/test.txt"));
207 command_line->AppendArgPath(test_doc);
208 ASSERT_TRUE(RunPlatformAppTest("platform_apps/document_editor")) << message_;
209 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698