Chrome OS "open with" picker allowing Web Intents

chrome/test/data/extensions/api_test/filebrowser_component/main.js

 // 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.
 
 /*
 This component extension test does the following:
 
 1. Creates an abc and log file on the local file system with some random text.
 2. Finds a registered task (file item context menu) for abc file and invokes it
    with url of the test file.
@@ skipping 49 matching lines @@
 
   if (!this.file_ || !this.fileText_ || !this.fileVerifierFunction_) {
     callback({message: "Test expectations not set properly."});
     return;
   }
 
   this.fileVerifierFunction_(this.file_, this.fileText_, request,
                              callback);
 };
 
+TestExpectations.prototype.hasVerifyStep = function() {
+  return !!this.fileVerifierFunction_;
+};
+
+// Verifies that list of tasks |tasks| contains tasks specified in
+// expectedTasks_. |successCallback| expects to be passed |tasks|.
+// |errorCallback| expects error object.
+TestExpectations.prototype.verifyTasks = function(tasks,
+                                                  successCallback,
+                                                  errorCallback) {
+  if (tasks.length != Object.keys(this.expectedTasks_).length) {
+    errorCallback({message: 'Wrong number of tasks found.'});
+    return;
+  }
+
+  for (var i = 0; i < tasks.length; ++i) {
+    var taskName = /^.*[|]\d+[|](\S+)$/.exec(tasks[i].taskId)[1];
+    var patterns = tasks[i].patterns;
+    var expectedPatterns = this.expectedTasks_[taskName];
+    if (!expectedPatterns) {
+      errorCallback({message: 'Wrong task from getFileTasks(): ' + taskName});

[tbarzic, 2012/09/11 02:01:40]

this function has been removed when "patterns" were removed from task passed to
the file manager extension..

[thorogood, 2012/09/13 02:34:49]

Cool, that makes sense. Yeah - I wondered about "patterns", since we were only
returning a list of things for your selected files already.

this was just left over from the big merge. Reverted this whole file.

+      return;
+    }
+    patterns = patterns.sort();
+    expectedPatterns = expectedPatterns.sort();
+    for (var j = 0; j < patterns.length; ++j) {
+      var translatedPattern = expectedPatterns[j].replace(
+          /^filesystem:/, "chrome-extension://*/");
+      if (patterns[j] != translatedPattern) {
+        errorCallback({message: 'Wrong patterns set for task ' +
+                                taskName + '. ' +
+                                'Got: ' + patterns +
+                                ' expected: ' + expectedPatterns});
+        return;
+      }
+    }
+  }
+  successCallback(tasks);
+};
+
 // Class that is in charge for running the test.
 var TestRunner = function(expectations) {
   this.expectations_ = expectations;
   this.fileCreator_ = new TestFileCreator("tmp", true /* shouldRandomize */);

[tbarzic, 2012/09/13 04:29:16]

Do
if (expectations.fileVerifierFunction_)
  this.listener_ = ..

[thorogood, 2012/09/13 08:14:04]

Done, although I never make a request like this from my packaged app, as
"chrome.extension.sendRequest" is not available in v2 packaged apps. I just call
success from the app.

   this.listener_ = this.onHandlerRequest_.bind(this);
 };
 
 // Starts the test.
 TestRunner.prototype.runTest = function() {
   // Get local FS, create dir with a file in it.
   console.log('Requesting local file system...');

[tbarzic, 2012/09/13 04:29:16]

if (this.listener_)

[thorogood, 2012/09/13 08:14:04]

Done.

   chrome.extension.onRequestExternal.addListener(this.listener_);
   chrome.fileBrowserPrivate.requestLocalFileSystem(
       this.onFileSystemFetched_.bind(this));
 };
 
 TestRunner.prototype.onFileSystemFetched_ = function(fs) {
   if (!fs) {
     this.errorCallback_(chrome.extensions.lastError);
     return;
   }
 
   this.fileCreator_.init(fs, this.onFileCreatorInit_.bind(this),
                              this.errorCallback_.bind(this));
 };
 
 TestRunner.prototype.onFileCreatorInit_ = function() {
   var ext = this.expectations_.getFileExtension();
   if (!ext) {
     this.errorCallback_({message: "Test file extension not set."});
     return;
   }
-  console.log(this.fileExtension);
   var self = this;
   this.fileCreator_.createFile('.log',
       function(file, text) {
         self.fileCreator_.createFile(ext,
             self.onFileCreated_.bind(self),
             self.errorCallback_.bind(self));
       },
       this.errorCallback_.bind(this));
 };
 
@@ skipping 12 matching lines @@
   console.log(tasks);
   if (!tasks || !tasks.length) {
     this.errorCallback_({message: 'No tasks registered'});
     return;
   }
 
   console.log('DONE fetching ' + tasks.length + ' tasks');
 
   tasks = this.filterTasks_(tasks);
   chrome.fileBrowserPrivate.executeTask(tasks[0].taskId, [fileUrl]);
+
+  if (!this.expectations_.hasVerifyStep()) {

[tbarzic, 2012/09/13 04:29:16]

I think you still need something like this (TestRunner will wait for the
response which will not be sent by the intent handler)

if (!this.listener_)
  this.reportSuccess_();

[thorogood, 2012/09/13 08:14:04]

I'm not sure - my test intent handler does in fact call "chrome.test.succeed",
which works fine. So I've left this out.

[tbarzic, 2012/09/13 18:44:45]

yeah, but the problem is files created during the test are cleaned up in
reportSuccess_/reportFail_ (but again, cleaning up the test dirs in the
extension could be racy in your case, especially if you add file reading in your
test intent handler)

FWIW, it think you would benefit from writing your own test behaviour rather
than using this, see remote.js for example (you would need initTest(),
readFile() and simplified executeReadFile() part). you should create/delete the
test file in C++ code (you could find ScopedTempDir usefull for cleaning up the
file. you can see  SetUp and AddMountPoint in
chrome/browser/ui/views/select_file_dialog_extension_browsertest.cc for
example).
I plan to refactor the rest of the tests to use something like this soon.

+    console.log('Not verifying content, succeed immediately.');
+    this.reportSuccess_();
+  }
 };
 
 TestRunner.prototype.filterTasks_ = function(tasks) {
   var result = [];
   for (var i = 0; i < tasks.length; i++) {
     if (tasks[i].taskId.split('|')[0] != kFileManagerExtensionId) {
       result.push(tasks[i]);
     }
   }
   return result;
@@ skipping 53 matching lines @@
   if (!error) {
     sendResponse({success: true});
     this.fileCreator_.cleanupAndEndTest(this.reportSuccess_.bind(this),
                                         this.reportFail_.bind(this,
                                                               cleanupError));
   } else {
     sendResponse({success: false});
     this.errorCallback_(error);
   }
 };