Introduce concept of provisional destinations to print preview

Introduce concept of provisional destination to print preview

These are destinations that can be listed in the UI, but cannot be
set as a selected destination in destination store. When user selects them in
destination search UI, extra steps are taken in order to resolve these destinations.
When resolved, provisional destinations are replaced by the resolved version,
which is then selected.

This is used for selecting USB printers that can be handled by an
extension destination provider (as decleared in the extension manifest),
but for whose USB devices the extension does not yet have permission.

These destinations cannot be directly handled by the extension, as it is
not avare of their presence. These destinations are provided by printer
provider API backend. When attempting to select those, user is asked to
allow the extension to access the USB device in question. If the user
agrees, the extension is given permission to access the USB device and
requested printer description for the USB printer. When the printer
description is returned, the provisional destinaion is removed from
destination store. The new destination is added to the store and selected.

BUG=468955
Committed: https://crrev.com/fd8d83b5e6fec748d71a6dff7ead345ca89d784e
Cr-Commit-Position: refs/heads/master@{#333888}

[tbarzic, 2015-05-27 17:10:11]

tbarzic@chromium.org changed reviewers:
+ alekseys@chromium.org, reillyg@chromium.org, vitalybuka@chromium.org

[tbarzic, 2015-05-27 17:10:11]

Built on top of https://codereview.chromium.org/1148383002/#ps20001

This is not quite ready for detailed review (for one I expect C++ code on which
this relies to change somewhat, but I'll update the cl as needed; also needs
i18n and css that better matches mocks), but I would like to get your opinion on
a general approach.

https://codereview.chromium.org/1144983002/diff/40001/chrome/browser/resource...
File chrome/browser/resources/print_preview/print_preview_focus_manager.js
(right):

https://codereview.chromium.org/1144983002/diff/40001/chrome/browser/resource...
chrome/browser/resources/print_preview/print_preview_focus_manager.js:24: var el
= document.body;
needed to handle showing an overlay (USB permission warning dialog) on top of
another overlay (destination search UI).

[Vitaly Buka (NO REVIEWS), 2015-06-01 17:29:17]

Does this need rebase?

[tbarzic, 2015-06-02 17:03:27]

On 2015/06/01 17:29:17, Vitaly Buka wrote:
> Does this need rebase?

yes, it does. I'll wait until Reilly's cl land, and then ping you when this is
ready to be reviewed.

[tbarzic, 2015-06-04 20:57:06]

I think this should be ready for review now.
Please take a look.

[Aleksey Shlyapnikov, 2015-06-08 23:00:54]

lgtm

https://codereview.chromium.org/1144983002/diff/80001/chrome/browser/resource...
File chrome/browser/resources/print_preview/data/destination_store.js (right):

https://codereview.chromium.org/1144983002/diff/80001/chrome/browser/resource...
chrome/browser/resources/print_preview/data/destination_store.js:666:
this.destinationMap_[this.getKey_(provisionalDestination)] = undefined;
delete this.destinationMap_[this.getKey_(provisionalDestination)];

https://codereview.chromium.org/1144983002/diff/80001/chrome/browser/resource...
File chrome/browser/resources/print_preview/search/destination_search.js
(right):

https://codereview.chromium.org/1144983002/diff/80001/chrome/browser/resource...
chrome/browser/resources/print_preview/search/destination_search.js:618:
lastFocusedElement = null;
Why do you need to reset this local var?

[tbarzic, 2015-06-09 18:21:59]

https://codereview.chromium.org/1144983002/diff/80001/chrome/browser/resource...
File chrome/browser/resources/print_preview/data/destination_store.js (right):

https://codereview.chromium.org/1144983002/diff/80001/chrome/browser/resource...
chrome/browser/resources/print_preview/data/destination_store.js:666:
this.destinationMap_[this.getKey_(provisionalDestination)] = undefined;
On 2015/06/08 23:00:54, Aleksey Shlyapnikov wrote:
> delete this.destinationMap_[this.getKey_(provisionalDestination)];

Done.

https://codereview.chromium.org/1144983002/diff/80001/chrome/browser/resource...
File chrome/browser/resources/print_preview/search/destination_search.js
(right):

https://codereview.chromium.org/1144983002/diff/80001/chrome/browser/resource...
chrome/browser/resources/print_preview/search/destination_search.js:618:
lastFocusedElement = null;
On 2015/06/08 23:00:54, Aleksey Shlyapnikov wrote:
> Why do you need to reset this local var?

removed

[tbarzic, 2015-06-11 00:53:33]

The CQ bit was checked by tbarzic@chromium.org

[tbarzic, 2015-06-11 00:53:33]

The patchset sent to the CQ was uploaded after l-g-t-m from
alekseys@chromium.org
Link to the patchset: https://codereview.chromium.org/1144983002/#ps100001
(title: ".")

[commit-bot: I haz the power, 2015-06-11 00:54:18]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1144983002/100001

[commit-bot: I haz the power, 2015-06-11 03:53:38]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2015-06-11 03:54:42]

Patchset 6 (id:??) landed as
https://crrev.com/fd8d83b5e6fec748d71a6dff7ead345ca89d784e
Cr-Commit-Position: refs/heads/master@{#333888}