Removing prefixMatch attribute of CacheQueryOptions- Blink Side

Removing prefixMatch attribute of CacheQueryOptions.

As this get removed in latest specs, removing it.
spec: https://slightlyoff.github.io/ServiceWorker/spec/service_worker/#cache-query-options-dictionary

PATCH 1: Chromium: https://codereview.chromium.org/941683002/
PATCH 2: Blink: [THIS PATCH]

BUG=462837
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=192462

[Paritosh Kumar, 2015-02-20 04:42:23]

paritosh.in@samsung.com changed reviewers:
+ a.cavalcanti@samsung.com

[Paritosh Kumar, 2015-02-20 04:42:23]

Please have a look.

[Paritosh Kumar, 2015-02-27 09:34:55]

paritosh.in@samsung.com changed reviewers:
+ jsbell@chromium.org, mkwst@chromium.org

[Paritosh Kumar, 2015-02-27 09:34:56]

Please have a look.

[Paritosh Kumar, 2015-02-27 09:36:27]

paritosh.in@samsung.com changed reviewers:
+ horo@chromium.org

[jsbell, 2015-02-27 16:45:24]

lgtm - will need owners approval for public/platform/ change

Chromium side will need to land first.

[Mike West, 2015-02-27 19:24:40]

Please create a bug to tie these two CLs together. It would also be nice to
point to both of them in the CL description (see
https://codereview.chromium.org/960853002 for an example of how that might
look).

LGTM, otherwise.

[horo, 2015-03-02 02:28:40]

Please remove LayoutTests which are using "prefixMatch" in cache-match-worker.js
and cache-delete-worker.js.

[Paritosh Kumar, 2015-03-02 10:38:39]

On 2015/03/02 02:28:40, horo wrote:
> Please remove LayoutTests which are using "prefixMatch" in
cache-match-worker.js
> and cache-delete-worker.js.

Thanks horo

Removed LayoutTests which are using "prefixMatch".

Please have a look.

[horo, 2015-03-03 03:13:35]

lgtm

[Paritosh Kumar, 2015-03-03 05:40:49]

The CQ bit was checked by paritosh.in@samsung.com

[Paritosh Kumar, 2015-03-03 05:40:49]

The patchset sent to the CQ was uploaded after l-g-t-m from mkwst@chromium.org,
jsbell@chromium.org
Link to the patchset: https://codereview.chromium.org/927723003/#ps20001 (title:
" ")

[commit-bot: I haz the power, 2015-03-03 05:41:21]

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

[Paritosh Kumar, 2015-03-03 09:46:16]

The CQ bit was unchecked by paritosh.in@samsung.com

[Paritosh Kumar, 2015-03-03 10:56:32]

On 2015/03/03 05:41:21, I haz the power (commit-bot) wrote:
> CQ is trying da patch. Follow status at
>  https://chromium-cq-status.appspot.com/patch-status/927723003/20001

CQ is failing on cache-add.html and cache-match.html, as I haven't uploaded
the test expectation for these two files after seeing the Presubmit errors.

Now, uploading these files by bypassing nohooks.

Please have a look.

[jsbell, 2015-03-06 17:02:46]

lgtm - what was the presubmit error, though?

[jsbell, 2015-03-06 17:03:05]

The CQ bit was checked by jsbell@chromium.org

[jsbell, 2015-03-06 17:03:05]

The patchset sent to the CQ was uploaded after l-g-t-m from mkwst@chromium.org,
horo@chromium.org
Link to the patchset: https://codereview.chromium.org/927723003/#ps40001 (title:
" ")

[commit-bot: I haz the power, 2015-03-06 17:03:52]

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

[commit-bot: I haz the power, 2015-03-06 17:13:21]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-03-06 17:13:22]

Try jobs failed on following builders:
  mac_blink_compile_dbg on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/mac_blink_compile_dbg/bu...)

[Paritosh Kumar, 2015-03-24 12:33:59]

On 2015/03/06 17:02:46, jsbell wrote:
> lgtm - what was the presubmit error, though?

Thanks jsbell

Apologize for delay in response, actually I was OOO.

The presubmit error was:

** Presubmit ERRORS **
* The following files are passing testharness results, they should be removed:
/src/third_party/WebKit/LayoutTests/http/tests/serviceworker/cache-delete-expected.txt
ERROR: found passing testharness results.

[Paritosh Kumar, 2015-03-24 14:02:53]

Updated the cl after rebase. Please have a look.

[jsbell, 2015-03-24 16:44:26]

still lgtm

[Paritosh Kumar, 2015-03-24 16:46:10]

The CQ bit was checked by paritosh.in@samsung.com

[Paritosh Kumar, 2015-03-24 16:46:11]

The patchset sent to the CQ was uploaded after l-g-t-m from mkwst@chromium.org,
horo@chromium.org
Link to the patchset: https://codereview.chromium.org/927723003/#ps60001 (title:
" After rebase")

[commit-bot: I haz the power, 2015-03-24 16:46:27]

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

[commit-bot: I haz the power, 2015-03-24 19:45:18]

Committed patchset #4 (id:60001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=192462

[tkent, 2015-03-24 22:22:05]

Does this remove a shipped IDL attribute?

[jsbell, 2015-03-24 22:46:23]

On 2015/03/24 22:22:05, tkent wrote:
> Does this remove a shipped IDL attribute?

No - it removes an optional member of a dictionary param, and the member that
was never supported. This change is undetectable from script.