chrome/browser/ui/intents/web_intent_picker_controller_browsertest.cc - Issue 9763001: [Web Intents] Inline installation of extensions in web intents picker.

Unified Diff: chrome/browser/ui/intents/web_intent_picker_controller_browsertest.cc

Issue 9763001: [Web Intents] Inline installation of extensions in web intents picker. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: mac fix 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 side-by-side diff with in-line comments

Download patch

« no previous file with comments | « chrome/browser/ui/intents/web_intent_picker_controller.cc ('k') | chrome/browser/ui/intents/web_intent_picker_delegate.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: chrome/browser/ui/intents/web_intent_picker_controller_browsertest.cc

diff --git a/chrome/browser/ui/intents/web_intent_picker_controller_browsertest.cc b/chrome/browser/ui/intents/web_intent_picker_controller_browsertest.cc

index 9dab1d2993f0a1fdc1d0a635a922a45bf55dca59..28d89733f9bf6e76f46ebe66d322e76263ef197a 100644

--- a/chrome/browser/ui/intents/web_intent_picker_controller_browsertest.cc

+++ b/chrome/browser/ui/intents/web_intent_picker_controller_browsertest.cc

@@ -20,6 +20,7 @@

#include "chrome/browser/ui/intents/web_intent_picker_model_observer.h"

#include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h"

#include "chrome/browser/webdata/web_data_service.h"

+#include "chrome/common/chrome_switches.h"

#include "chrome/common/url_constants.h"

#include "chrome/test/base/in_process_browser_test.h"

#include "chrome/test/base/ui_test_utils.h"

@@ -27,6 +28,7 @@

#include "content/public/browser/web_intents_dispatcher.h"

#include "content/test/test_url_fetcher_factory.h"

#include "net/base/escape.h"

+#include "net/base/mock_host_resolver.h"

#include "testing/gtest/include/gtest/gtest.h"

#include "ui/gfx/image/image_unittest_util.h"

#include "ui/gfx/image/image_util.h"

@@ -34,11 +36,13 @@

namespace {

-const string16 kAction1(ASCIIToUTF16("http://www.example.com/share"));

+const string16 kAction1(ASCIIToUTF16("http://webintents.org/share"));

const string16 kAction2(ASCIIToUTF16("http://www.example.com/foobar"));

-const string16 kType(ASCIIToUTF16("image/png"));

+const string16 kType1(ASCIIToUTF16("image/png"));

+const string16 kType2(ASCIIToUTF16("text/*"));

const GURL kServiceURL1("http://www.google.com");

const GURL kServiceURL2("http://www.chromium.org");

+const char kDummyExtensionId[] = "bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb";

const char kCWSResponseEmpty[] =

"{\"kind\":\"chromewebstore#itemList\",\"total_items\":0,\"start_index\":0,"

"\"items\":[]}";

@@ -49,34 +53,20 @@ const char kCWSResponseResultFormat[] =

"\"start_index\":0,"

"\"items\":[{"

"\"kind\":\"chromewebstore#item\","

- "\"id\":\"nhkckhebbbncbkefhcpcgepcgfaclehe\","

+ "\"id\":\"%s\","

"\"type\":\"APPLICATION\","

"\"num_ratings\":0,"

"\"average_rating\":0.0,"

"\"manifest\": \"{\\n"

- "\\\"update_url\\\":\\"

- "\"http://0.tbhome_staging.dserver.download-qa.td.borg.google.com/"

- "service/update2/crx\\\",\\n "

- "\\\"name\\\": \\\"Sidd's Intent App\\\",\\n "

- "\\\"description\\\": \\\"Do stuff\\\",\\n "

- "\\\"version\\\": \\\"1.2.19\\\",\\n "

- "\\\"app\\\": {\\n "

- "\\\"urls\\\": [ \\n ],\\n "

- "\\\"launch\\\": {\\n "

- "\\\"web_url\\\": \\\"http://siddharthasaha.net/\\\"\\n "

- "}\\n "

- "},\\n "

- "\\\"icons\\\": {\\n \\\"128\\\": \\\"icon128.png\\\"\\n },\\n "

- "\\\"permissions\\\":" " [\\n "

- "\\\"unlimitedStorage\\\",\\n \\\"notifications\\\"\\n "

- "],\\n"

- " \\\"intents\\\": {\\n "

- "\\\"%s\\\" : {\\n "

- "\\\"type\\\" : [\\\"%s\\\"],\\n "

- "\\\"path\\\" : \\\"//services/edit\\\",\\n "

- "\\\"title\\\" : \\\"Sample Editing Intent\\\",\\n "

- "\\\"disposition\\\" : \\\"inline\\\"\\n "

- "}\\n "

+ "\\\"name\\\": \\\"Dummy Share\\\",\\n"

+ "\\\"version\\\": \\\"1.0.0.0\\\",\\n"

+ "\\\"intents\\\": {\\n"

+ "\\\"%s\\\" : {\\n"

+ "\\\"type\\\" : [\\\"%s\\\"],\\n"

+ "\\\"path\\\" : \\\"share.html\\\",\\n"

+ "\\\"title\\\" : \\\"Dummy share!\\\",\\n"

+ "\\\"disposition\\\": \\\"inline\\\"\\n"

+ "}\\n"

"}\\n"

"}\\n\","

"\"family_safe\":true,"

@@ -107,6 +97,7 @@ class WebIntentPickerMock : public WebIntentPicker,

: num_installed_services_(0),

num_icons_changed_(0),

num_extension_icons_changed_(0),

+ num_extensions_installed_(0),

message_loop_started_(false),

pending_async_completed_(false) {

}

@@ -130,23 +121,35 @@ class WebIntentPickerMock : public WebIntentPicker,

WebIntentPickerModel* model, const GURL& url) OVERRIDE {}

virtual void Close() OVERRIDE {}

- virtual void OnPendingAsyncCompleted() OVERRIDE {

- pending_async_completed_ = true;

+ virtual void OnExtensionInstallSuccess(const std::string& id) OVERRIDE {

+ num_extensions_installed_++;

+ }

- if (message_loop_started_)

- MessageLoop::current()->Quit();

+ virtual void OnExtensionInstallFailure(const std::string& id) OVERRIDE {

+ }

+ virtual void OnPendingAsyncCompleted() OVERRIDE {

+ StopWaiting();

}

- void WaitForPendingAsync() {

+ void Wait() {

if (!pending_async_completed_) {

message_loop_started_ = true;

ui_test_utils::RunMessageLoop();

+ pending_async_completed_ = false;

}

+ void StopWaiting() {

+ pending_async_completed_ = true;

+ if (message_loop_started_)

+ MessageLoop::current()->Quit();

+ }

int num_installed_services_;

int num_icons_changed_;

int num_extension_icons_changed_;

+ int num_extensions_installed_;

bool message_loop_started_;

bool pending_async_completed_;

};

@@ -183,6 +186,27 @@ class WebIntentPickerControllerBrowserTest : public InProcessBrowserTest {

WebIntentPickerControllerBrowserTest() {}

+ virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {

+ // We start the test server now instead of in

+ // SetUpInProcessBrowserTestFixture so that we can get its port number.

+ ASSERT_TRUE(test_server()->Start());

+ InProcessBrowserTest::SetUpCommandLine(command_line);

+ net::HostPortPair host_port = test_server()->host_port_pair();

+ command_line->AppendSwitchASCII(

+ switches::kAppsGalleryDownloadURL,

+ base::StringPrintf(

+ "http://www.example.com:%d/files/extensions/intents/%%s.crx",

+ host_port.port()));

+ command_line->AppendSwitchASCII(

+ switches::kAppsGalleryInstallAutoConfirmForTests, "accept");

+ }

+ virtual void SetUpInProcessBrowserTestFixture() OVERRIDE {

+ host_resolver()->AddRule("www.example.com", "127.0.0.1");

+ }

virtual void SetUpOnMainThread() OVERRIDE {

// The FakeURLFetcherFactory will return a NULL URLFetcher if a request is

// created for a URL it doesn't know and there is no default factory.

@@ -207,33 +231,36 @@ class WebIntentPickerControllerBrowserTest : public InProcessBrowserTest {

void AddWebIntentService(const string16& action, const GURL& service_url) {

webkit_glue::WebIntentServiceData service;

service.action = action;

- service.type = kType;

+ service.type = kType1;

service.service_url = service_url;

web_data_service_->AddWebIntentService(service);

}

void AddCWSExtensionServiceEmpty(const string16& action) {

- GURL cws_query_url = CWSIntentsRegistry::BuildQueryURL(action, kType);

+ GURL cws_query_url = CWSIntentsRegistry::BuildQueryURL(action, kType1);

fake_url_fetcher_factory_->SetFakeResponse(cws_query_url.spec(),

- kCWSResponseEmpty, true);

+ kCWSResponseEmpty, true);

}

- void AddCWSExtensionServiceWithResult(const string16& action) {

- GURL cws_query_url = CWSIntentsRegistry::BuildQueryURL(action, kType);

+ void AddCWSExtensionServiceWithResult(const std::string& extension_id,

+ const string16& action,

+ const string16& type) {

+ GURL cws_query_url = CWSIntentsRegistry::BuildQueryURL(action, type);

std::string icon_url;

std::string escaped_action = net::EscapePath(UTF16ToUTF8(action));

base::SStringPrintf(&icon_url, kCWSFakeIconURLFormat,

- escaped_action.c_str());

+ escaped_action.c_str());

std::string response;

base::SStringPrintf(&response, kCWSResponseResultFormat,

- UTF16ToUTF8(action).c_str(), UTF16ToUTF8(kType).c_str(),

- icon_url.c_str());

+ extension_id.c_str(),

+ UTF16ToUTF8(action).c_str(), UTF16ToUTF8(type).c_str(),

+ icon_url.c_str());

fake_url_fetcher_factory_->SetFakeResponse(cws_query_url.spec(), response,

- true);

+ true);

fake_url_fetcher_factory_->SetFakeResponse(icon_url, icon_response_,

- true);

+ true);

}

void OnSendReturnMessage(

@@ -249,6 +276,10 @@ class WebIntentPickerControllerBrowserTest : public InProcessBrowserTest {

controller_->OnCancelled();

}

+ void OnExtensionInstallRequested(const std::string& extension_id) {

+ controller_->OnExtensionInstallRequested(extension_id);

+ }

void CreateFakeIcon() {

gfx::Image image(gfx::test::CreateImage());

std::vector<unsigned char> image_data;

@@ -256,7 +287,7 @@ class WebIntentPickerControllerBrowserTest : public InProcessBrowserTest {

DCHECK(result);

std::copy(image_data.begin(), image_data.end(),

- std::back_inserter(icon_response_));

+ std::back_inserter(icon_response_));

}

WebIntentPickerMock picker_;

@@ -273,8 +304,8 @@ IN_PROC_BROWSER_TEST_F(WebIntentPickerControllerBrowserTest, ChooseService) {

AddWebIntentService(kAction1, kServiceURL2);

AddCWSExtensionServiceEmpty(kAction1);

- controller_->ShowDialog(browser(), kAction1, kType);

- picker_.WaitForPendingAsync();

+ controller_->ShowDialog(browser(), kAction1, kType1);

+ picker_.Wait();

EXPECT_EQ(2, picker_.num_installed_services_);

EXPECT_EQ(0, picker_.num_icons_changed_);

@@ -299,10 +330,10 @@ IN_PROC_BROWSER_TEST_F(WebIntentPickerControllerBrowserTest,

FetchExtensionIcon) {

AddWebIntentService(kAction1, kServiceURL1);

AddWebIntentService(kAction1, kServiceURL2);

- AddCWSExtensionServiceWithResult(kAction1);

+ AddCWSExtensionServiceWithResult(kDummyExtensionId, kAction1, kType1);

- controller_->ShowDialog(browser(), kAction1, kType);

- picker_.WaitForPendingAsync();

+ controller_->ShowDialog(browser(), kAction1, kType1);

+ picker_.Wait();

EXPECT_EQ(2, picker_.num_installed_services_);

EXPECT_EQ(0, picker_.num_icons_changed_);

EXPECT_EQ(1, picker_.num_extension_icons_changed_);

@@ -313,11 +344,11 @@ IN_PROC_BROWSER_TEST_F(WebIntentPickerControllerBrowserTest, OpenCancelOpen) {

AddWebIntentService(kAction1, kServiceURL2);

AddCWSExtensionServiceEmpty(kAction1);

- controller_->ShowDialog(browser(), kAction1, kType);

- picker_.WaitForPendingAsync();

+ controller_->ShowDialog(browser(), kAction1, kType1);

+ picker_.Wait();

OnCancelled();

- controller_->ShowDialog(browser(), kAction1, kType);

+ controller_->ShowDialog(browser(), kAction1, kType1);

OnCancelled();

}

@@ -335,8 +366,8 @@ IN_PROC_BROWSER_TEST_F(WebIntentPickerControllerBrowserTest,

ASSERT_EQ(2, browser()->tab_count());

EXPECT_EQ(original, browser()->GetSelectedWebContents()->GetURL());

- controller_->ShowDialog(browser(), kAction1, kType);

- picker_.WaitForPendingAsync();

+ controller_->ShowDialog(browser(), kAction1, kType1);

+ picker_.Wait();

EXPECT_EQ(1, picker_.num_installed_services_);

webkit_glue::WebIntentData intent;

@@ -356,3 +387,29 @@ IN_PROC_BROWSER_TEST_F(WebIntentPickerControllerBrowserTest,

ASSERT_EQ(2, browser()->tab_count());

EXPECT_EQ(original, browser()->GetSelectedWebContents()->GetURL());

}

+IN_PROC_BROWSER_TEST_F(WebIntentPickerControllerBrowserTest,

+ ExtensionInstallSuccess) {

+ const char extension_id[] = "ooodacpbmglpoagccnepcbfhfhpdgddn";

+ AddCWSExtensionServiceWithResult(extension_id, kAction1, kType2);

+ controller_->ShowDialog(browser(), kAction1, kType2);

+ picker_.Wait();

+ webkit_glue::WebIntentData intent;

+ intent.action = kAction1;

+ intent.type = kType2;

+ IntentsDispatcherMock dispatcher(intent);

+ controller_->SetIntentsDispatcher(&dispatcher);

+ OnExtensionInstallRequested(extension_id);

+ picker_.Wait();

+ EXPECT_EQ(1, picker_.num_extensions_installed_);

+ const Extension* extension = browser()->profile()->GetExtensionService()->

+ GetExtensionById(extension_id, false);

+ EXPECT_TRUE(extension);

+ // Installing an extension should also choose it. Since this extension uses

+ // window disposition, it will create a new tab.

+ ASSERT_EQ(2, browser()->tab_count());