Commit 8144dae3 authored by Yue Li's avatar Yue Li Committed by Chromium LUCI CQ

Quick Answers: Add page size options for Search request

Since we only use the first result from search request, set the page
size to 1 to reduce the delay.

Bug: b/173239982
Test: None
Change-Id: Ib2abd92e23b430b837f0556c31bcc0d17873256e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2574072Reviewed-by: default avatarXiyuan Xia <xiyuan@chromium.org>
Reviewed-by: default avatarJeroen Dhollander <jeroendh@chromium.org>
Reviewed-by: default avatarXiaohui Chen <xiaohuic@chromium.org>
Commit-Queue: Yue Li <updowndota@chromium.org>
Cr-Commit-Position: refs/heads/master@{#833940}
parent 433889a5
...@@ -30,6 +30,9 @@ using network::mojom::URLLoaderFactory; ...@@ -30,6 +30,9 @@ using network::mojom::URLLoaderFactory;
// "client_id": { // "client_id": {
// "client_type": "EXPERIMENTAL" // "client_type": "EXPERIMENTAL"
// } // }
// "options": {
// "page_size": "1"
// }
// } // }
// //
// Which is: // Which is:
...@@ -38,12 +41,17 @@ using network::mojom::URLLoaderFactory; ...@@ -38,12 +41,17 @@ using network::mojom::URLLoaderFactory;
// "raw_query": STRING // "raw_query": STRING
// "client_id": DICT // "client_id": DICT
// "client_type": STRING // "client_type": STRING
// "options": DICT
// "page_size": STRING
constexpr base::StringPiece kQueryKey = "query"; constexpr base::StringPiece kQueryKey = "query";
constexpr base::StringPiece kRawQueryKey = "rawQuery"; constexpr base::StringPiece kRawQueryKey = "rawQuery";
constexpr base::StringPiece kClientTypeKey = "clientType"; constexpr base::StringPiece kClientTypeKey = "clientType";
constexpr base::StringPiece kClientIdKey = "clientId"; constexpr base::StringPiece kClientIdKey = "clientId";
constexpr base::StringPiece kClientType = "QUICK_ANSWERS_CROS"; constexpr base::StringPiece kClientType = "QUICK_ANSWERS_CROS";
constexpr base::StringPiece kPageSizeKey = "pageSize";
constexpr base::StringPiece kOptionsKey = "options";
constexpr base::StringPiece kPageSize = "1";
std::string BuildSearchRequestPayload(const std::string& selected_text) { std::string BuildSearchRequestPayload(const std::string& selected_text) {
Value payload(Value::Type::DICTIONARY); Value payload(Value::Type::DICTIONARY);
...@@ -57,6 +65,10 @@ std::string BuildSearchRequestPayload(const std::string& selected_text) { ...@@ -57,6 +65,10 @@ std::string BuildSearchRequestPayload(const std::string& selected_text) {
client_id.SetKey(kClientTypeKey, Value(kClientType)); client_id.SetKey(kClientTypeKey, Value(kClientType));
payload.SetKey(kClientIdKey, std::move(client_id)); payload.SetKey(kClientIdKey, std::move(client_id));
Value options(Value::Type::DICTIONARY);
options.SetKey(kPageSizeKey, Value(kPageSize));
payload.SetKey(kOptionsKey, std::move(options));
std::string request_payload_str; std::string request_payload_str;
base::JSONWriter::Write(payload, &request_payload_str); base::JSONWriter::Write(payload, &request_payload_str);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment